Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 311019 - Unable to customize preferences when Subversive is embedded in an eclipse product
Summary: Unable to customize preferences when Subversive is embedded in an eclipse pro...
Status: RESOLVED FIXED
Alias: None
Product: Subversive
Classification: Technology
Component: Core (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Igor Burilo CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-04-29 09:26 EDT by Yann Andenmatten CLA
Modified: 2010-09-29 13:18 EDT (History)
1 user (show)

See Also:


Attachments
Patch with the correction (1.93 KB, patch)
2010-04-29 09:46 EDT, Yann Andenmatten CLA
igor.burilo: iplog+
Details | Diff
Eclipse product for test (11.55 KB, application/octet-stream)
2010-04-29 10:07 EDT, Yann Andenmatten CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Yann Andenmatten CLA 2010-04-29 09:26:29 EDT
Build Identifier: 20090920-1017

When I use the standard plugin_customization.ini file that allow to override the preference default defined by plugins, it is ignored by subversive plugin.
I analyzed the Subversive code and found out why and produce a patch.

Reproducible: Always

Steps to Reproduce:
1. Create an eclipse product (I can post one)
2. Add a plugin_customization.ini at the root of the product plugin with the following line:
org.eclipse.team.svn.ui/preference.autopropertiesautoproperties=MQ\=\=;YWJj;YWJj
3. Run the product
4. Look at Window > Preference > Team > SVN > Properties Configuration
5. It remains empty (it should have an automatic property checked with abc as a File Name and abc as Properties
Comment 1 Yann Andenmatten CLA 2010-04-29 09:46:50 EDT
Created attachment 166482 [details]
Patch with the correction

The issue is due to the initilization of the default prefrences from the SVNTeamUIPlugin's start method directly. Such initilization should be done using the org.eclipse.core.runtime.preferences initializer extension point.

When done this way the preferenceCustomization is taken in account after the Subversive default properties are set. Otherwise the preferenceCustomization will be taken in account firstly, then overridden during org.eclipse.team.svn.ui plugin startup (the default preference initilisation is called here).
Comment 2 Yann Andenmatten CLA 2010-04-29 10:07:06 EDT
Created attachment 166485 [details]
Eclipse product for test

Here is an Eclipse test product to do the validation. 

Launch the "Lauch Eclipise Test Product.launch" in a workspace with the subversive plugins checked out and with the patch applied. (Futher target plateform might have to be defined.)
Comment 3 Yann Andenmatten CLA 2010-05-26 04:20:30 EDT
I don't mean to be rude or pushy, but do you have any idea when you could simply review the patch. It's a quite simple modification (as many look like, I agree).
Thanks
Yann
Comment 4 Igor Burilo CLA 2010-05-27 03:29:37 EDT
Yann, thanks a lot for provided patch, I'll review it when I finish with some important tasks I'm doing now.
Comment 5 Igor Burilo CLA 2010-05-28 06:29:33 EDT
Yann, I applied patch. Thanks a lot.
Comment 6 Yann Andenmatten CLA 2010-05-28 07:04:56 EDT
Thanks to you, Igor.
Comment 7 Yann Andenmatten CLA 2010-09-28 12:44:11 EDT
Hi Igor, I couldn't found my correction in org.eclipse.team.svn.ui_0.7.9.I20100512-1900.jar. In which version will it be released ?
Comment 8 Yann Andenmatten CLA 2010-09-29 13:18:43 EDT
Sorry, I just see now your comment is was on 2010-05-27, and the last release from 
2010-05-12. So we'll just wait a bit more...