Ticket #365 (closed task: fixed)

TODO : find a testbed - find some option in IJ2 that when changed currently DOES NOT work correctly because it calls an IJ1 method that uses IJ1's preferences.

This should work as a test bed:

ImageCalculator, float images, divide one image by another image that is all zero. IJ1 looks up a pref for the divide by zero value. IJ2 tracks this itself. If you set DBZ to something in IJ2 Edit Options it will not be applied when the above image calculation is done.

Since I've replaced ImageCalculator the only way to test this with my test bed is to unblacklist the IJ1 ImageCalculator, set the DBZ value in IJ2 prefs, run IJ1 ImageCalculator as specified above, and see that IJ2 DBZ value actually used.

Another item to be added is Options events. Make each OptionsPlugin extend an abstract options plugin of some type. The abstract class has the single method run() which publishes an OptionChanged event (only if a setting actually changed or just everytime?).

ij-legacy layer subscribes to OptionChanged events and sets the corresponding IJ1 pref to keep them in sync.

Some work completed on this ticket in c3110d966d0d8359060d8a0ace8714364fd30499. Added an OptionsPlugin base class whose run() method throws an OptionsChangedEvent. Changed all options plugins to inherit this behavior. Included new Rounded Rect Tool plugin.

Still need to do:

• listen in legacy layer for OptionsChangedEvents. When fired, get current values from Prefs and set IJ1's relevant global settings or Prefs.

Since the legacy layer is being modified by someone else right now this can wait.

Made code for setting IJ1 preferences (as much as feasible) when IJ2 options are changed. Present in cd09b8c05346c9ed7ada1d755a657c0cfbff0ce8.

Still TODO:

An initial call to OptionsSynchronizer::update() has to take place when the IJ2 Prefs are loaded from disk. Determine initialization order.

Some preferences cannot be synced with IJ1 and need IJ2 side effects to be implemented. Those are outlined as TODOs in OptionsSynchronizer. Each IJ2 Options dialog could do the side effects code as needed.

Initial call to update() happens in LegacyService::initialize().

Have not determined how to set all IJ1 preferences from IJ2. Might need to use reflection on IJ1 plugins that contain the nonpublic settings.

Have not put hooks in the legacy layer that notices when an IJ1 setting is updated by an IJ1 command which then updates IJ2's settings.

Plan:

• make sure IJ2 can set all IJ1 options fields (use ClassUtils methods such as setValue() and one Curtis hatched for getting a Field). Use these to set instance variables of IJ1 options classes as needed.

• hook IJ1's Prefs.set() in the legacy layer and update the appropriate IJ2 option. Have a map that takes IJ1 pref key id strings and gets the IJ2 settings key. Set the value as specified.

LegacyInjector now captures all getting and setting of ij.Prefs with methods to IJ2's Prefs. There is one batch of settings. Changes in IJ1 to settings should affect IJ2 values.

All that is left is to:

1) Make sure we are setting IJ1 options values from IJ2 (might need to use reflection)

2) Make sure we are using default values on IJ2 prefs get() calls so that the initial values are assigned correctly.

Reflection comment

use ClassUtils.setValue() and ClassUtils.getField(class,fieldName)

Much code changed by Curtis and Barry. Major commit in 0c803bfcb23b34910d2221e1e7192b5c0e6a2761. OptionsPlugins now manage initialization of fields from preferences falling back to compile time defaults.