Bug 19494 - [Preferences] Preferences not working with import/export
Summary: [Preferences] Preferences not working with import/export
Status: CLOSED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 2.0   Edit
Hardware: Other other
: P3 normal (vote)
Target Milestone: 2.1   Edit
Assignee: Tod Creasey CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 21971 (view as bug list)
Depends on: 19541 19687 19688 19691 19937 21971 21977
Blocks:
  Show dependency tree
 
Reported: 2002-06-06 10:36 EDT by Nick Edgar CLA
Modified: 2005-05-10 14:54 EDT (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Nick Edgar CLA 2002-06-06 10:36:24 EDT
Build F2

The following preferences were not transfered from one workspace to another 
using the export/import preferences feature.

Workbench
 - Perform build automatically on resource startup
Workbench>Editors
 - Key bindings
Workbench>File Associations
 - Added/Removed types and editors
 - Default
Workbench>Fonts
 - Banner font
 - Header font
 - Text font
Workbench>Label Decorators
 - Selection of decorators
Workbench>Local History
 - Days to keep files
 - Entries per file
 - Max file size
Workbench>Perspectives
 - Default perspective
Workbench>Search
 - Reuse editors
 - Emphasize inexact
 - Foreground color
Workbench>Startup
 - not tested
Build Order
 - Use default
 - order 
Debug>Console
 - Text font (see bug 19485)
External Tools>Ant
 -Added classpath variables, tasks and types
Help
 - Current web browser adapter not tested as there was only one
Java>Classpath Variables
 - Added/Removed classpath variables
Java>Debug
 - Debugger timeout
Java>JUnit
 - Show the Junit results
 - Removed stack filter patterns
Java>Templates
 - New, Edited, Removed
Plug-In Devlopment>Source Code Location
 - Added, Removed
Plug-In Devlopment>Target Platform
 - another compatible application
 - Application Location
 - Selection
Team
 - Use Incoming/Outgoing
Team>CVS>Console
 - Console fon setting (see bug 19483)
Team>File Content
 - Add, Remove, Change
Team>Ignored Resources
 - Add, Remove, Selected
Comment 1 Nick Edgar CLA 2002-06-06 10:38:56 EDT
Many of these are not stored in the preference store.
E.g. autobuild is just in the workspace description.

Others clearly should be, e.g. Debugger Timeout.

Need to go through them all and consider whether they should be in the 
preference store.  Mirroring state kept elsewhere (like autobuild) is a 
possibility, but there is a risk of them getting out of sync.

Comment 2 Darin Swanson CLA 2002-06-06 16:10:44 EDT
Java Debugger timeout preferences problem logged as bug 19541
Comment 3 Nick Edgar CLA 2002-06-07 15:47:15 EDT
Bumping up priority.  Please file PRs against the other components and mark 
them as blocking this one.
Comment 4 Nick Edgar CLA 2002-06-11 00:07:19 EDT
For F3, can only address ones that are straightforward.
Should avoid mirroring state that is kept elsewhere (e.g. local history 
settings).
Comment 5 Tod Creasey CLA 2002-06-11 14:01:48 EDT
Resolved for those in the Platform UI that are not too dangerous to change. 
Here is the breakdown.

Works in build 20020610
Workbench>Fonts
 - Banner font
 - Header font
 - Text font

Fixed and verified by RG
Workbench>Editors
 - Key bindings
Workbench>Label Decorators
 - Selection of decorators

Workbench>Perspectives
 - Default perspective


High risk fix - not done

Workbench
 - Perform build automatically on resource startup
Workbench>File Associations
 - Added/Removed types and editors
 - Default

Workbench>Local History
 - Days to keep files
 - Entries per file
 - Max file size

Build Order
 - Use default
 - order 
Comment 6 Tod Creasey CLA 2002-06-11 15:25:56 EDT
Added Bug 19937 as a dependancy as this prevents import working for the Ant 
preference page.
Comment 7 Tod Creasey CLA 2002-06-11 15:30:32 EDT
Marking as later as all work for F3 (not including dependant PRs) has been done.
Comment 8 Tod Creasey CLA 2002-07-25 09:19:16 EDT
Should re-investigate for 2.1
Comment 9 Nick Edgar CLA 2002-07-25 13:49:32 EDT
Rather than the workbench attempting to mirror core settings as preferences, it 
would be better for core to move to using preferences for these settings (auto 
build, local history settings, etc.).
Comment 10 Nick Edgar CLA 2002-07-26 15:49:12 EDT
Filed bug 21977 for core to consider using preferences for workspace settings.
Comment 11 Nick Edgar CLA 2002-12-03 14:25:26 EST
Now that Core is using the preference store for its own settings like 
autobuild, we should not have to mirror these ourselves.
Comment 12 Tod Creasey CLA 2002-12-04 13:09:49 EST
Upping proirity to P3 and marking as normal.
Comment 13 Tod Creasey CLA 2002-12-04 13:15:02 EST
Once we have access to the constants from Core we will also have to start 
listening to the build order preference and generating an auto build from that.

I would suggest splitting WorkbenchPreferenceListener up and adding a 
CorePreferenceListener that generates global build actions for auto build and 
build order.
Comment 14 Nick Edgar CLA 2002-12-04 13:19:58 EST
We should not be triggering builds ourselves.
If the user changes the autobuild preference, that should change the setting in 
Core as it does now.  Core should now set this in their preference store.  
Listeners on Core's preferences will be notified.  In our own implementation, 
we will not have to fake out a preference notification for this as we do 
currently.  We can just hook Core's.
Comment 15 Tod Creasey CLA 2002-12-04 13:26:20 EST
We currently do this in two cases: when updating build order and when updating 
auto build. DJ if this OK I will remove both of our build actions that we 
create.

This means our only concern will be adding and removing the build action 
button from the window when auto build is toggled.
Comment 16 Tod Creasey CLA 2002-12-04 13:27:47 EST
*** Bug 21971 has been marked as a duplicate of this bug. ***
Comment 17 Tod Creasey CLA 2002-12-04 13:47:59 EST
We will leave the build action behavior as it is as we do not want to trigger 
extra actions when the preferences are changed outside of the UI.
Comment 18 Tod Creasey CLA 2002-12-05 11:43:05 EST
Closing PR as we have added all that we can for the UI.
Comment 19 Tod Creasey CLA 2005-05-10 14:54:13 EDT
Marking as closed.