tweezy23 / anyedittools

Automatically exported from code.google.com/p/anyedittools
2 stars 0 forks source link

Auto-convert project specific settings #52

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Disable "Remove trailing whitespace" checkbox in Workspace Settings
2. Enable project specific settings
3. Enable "Remove trailing whitespace" checkbox in project specific settings

When I do it the other way round: enabling trailing whitespace removal in 
workspace and disabling it in a project it works as expected.

I was expecting that this configuration will allow me to enable auto-convert on 
a project based settings. This is the best approach when you work on multiple 
projects and only in some of them you are want to auto-convert.

AnyEdit 2.3.3, Eclipse Helios SR1, Linux (but I suppose it's not OS specific).

Original issue reported on code.google.com by pite...@gmail.com on 27 Nov 2010 at 12:28

GoogleCodeExporter commented 9 years ago
"When I do it the other way round: enabling trailing whitespace removal in 
workspace and disabling it in a project *it* works as expected."

WHAT works as expected???

And if it works, than it is not clear for me what is the issue?
Configuration allows already project specific settings. What else do you need?

Regards,
Andrei

Original comment by iloveeclipse on 27 Nov 2010 at 12:34

GoogleCodeExporter commented 9 years ago
To put it simple: the project specific settings work only one way.

First scenario (working): you set "Remove trailing whitespace" checkbox in 
workspace configuration (Window -> Preferences) and then you unset "Remove 
trailing whitespace" checkbox in project configuration.
The result is as expected -> "Remove trailing whitespace" is not enabled for 
the selected project

Second scenario (not working): you unset "Remove trailing whitespace" checkbox 
in workspace configuration (Window -> Preferences) and then you set "Remove 
trailing whitespace" checkbox in project configuration.

The result I expected would be that "Remove trailing whitespace" is not enabled 
for all other but selected project.
The result I observe is that "Remove trailing whitespace" is not enabled for 
all projects (even the one that has the checkbox with this feature set).

Is it clear now?

Regards,
Piotr

Original comment by pite...@gmail.com on 27 Nov 2010 at 2:27

GoogleCodeExporter commented 9 years ago
Got it. Good catch :-)
Do you want to have snapshot to test?
CU
Andrei

Original comment by iloveeclipse on 8 Dec 2010 at 10:19

GoogleCodeExporter commented 9 years ago
If you provide one I will test it but tonight I start holidays until 18th of 
December.

Regards,
Piotr

Original comment by pite...@gmail.com on 9 Dec 2010 at 5:16

GoogleCodeExporter commented 9 years ago
Attached is the the plugin snapshot (not tested at all). 
You have to uninstall old Anyedit first and then drop the file to the 
/eclipse/dropins directory.

Original comment by iloveeclipse on 9 Dec 2010 at 9:35

Attachments:

GoogleCodeExporter commented 9 years ago
I checked it only for discussed issue and it is fixed now.

Scenarios checked:
1. Enabled in global preferences, project preferences not set
   expected: trailing spaces removed
   result: as expected

2. Enabled in global preferences, disabled in project preferences
   expected: trailing spaces not removed
   result: as expected

3. Disabled in global preferences, enabled in project preferences
   expected: trailing spaces removed
   result: as expected

Thank you!

Regards,
Piotr

Original comment by pite...@gmail.com on 9 Dec 2010 at 10:06

GoogleCodeExporter commented 9 years ago

Original comment by iloveeclipse on 17 Dec 2010 at 5:32