Opened 7 years ago

Closed 7 years ago

#129 closed defect (fixed)

WORKPLACE_PROCESS check removes all "open" options should only remove WPS opts

Reported by: gyoung Owned by: stevenhl,gyoung
Priority: minor Milestone: Release_3.7
Component: fm/2 base Version: 3.07
Keywords: Cc:

Description


Change History (4)

comment:1 Changed 7 years ago by stevenhl

  • Milestone set to Release_3.7
  • Owner set to stevenhl,gyoung
  • Type changed from enhancement to defect
  • Version set to 3.07

comment:2 Changed 7 years ago by stevenhl

Based on Ulrich Möller work in XWorkPlace, it appears that the values XWORKPLACE_PROCESS=YES and XWORKPLACEPROCESS=NO are the correct values to check for. The change applied by r718 to init.c tests the variable appropriately. It is assumed is some other app defines itself as the Workplace process, it will properly control the environment for applications that it might start.

What remains to do for this ticket is to ensure that the menu enable/disables are appropriate for when fm/2 is the Workplace process.

comment:3 Changed 7 years ago by gyoung

Some of the menus that are disabled don't need to be.

Open FM/2 Window is but shouldn't be. I think the best way to fix this is by moving this item off the open sub menu to the main menu and renaming "open" WPS open.

The partitioning menu is also removed. Why?

This is also used to guess if the print spooler is running.
if (!fWorkPlace) Print.c 50
return TRUE;

Is there a more reliable way to check this?

comment:4 Changed 7 years ago by gyoung

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.