Hei Michael (Brunig),

A few month ago, I changed a method signature in OpenJUMP LayerManager's 
class to solve a  problem with "cloned windows", but it has broken one 
(or more) pirol's plugin :

The change :
com.vividsolutions.jump.workbench.model.LayerManager.dispose(Layerable 
layerable)
-->
com.vividsolutions.jump.workbench.model.LayerManager.dispose(WorkbenchFrame 
frame, Layerable layerable)

The reason :
when one removes a layer from a task, I want this layer to be removed 
from all the cloned windows in this task,
so I needed a reference to the frame.

The break :
in 
de.fhOsnabrueck.jump.pirol.plugIns.PirolRasterImage.RemoveSelectedRasterImageLayersPlugIn.remove(Layerable[]
 
selectedLayers)
the code use the old method in the following line
selectedLayers[i].getLayerManager().dispose(selectedLayers[i]);

It's quite straight to change it to use the new method (you have also to 
pass the PlugInContext to the method to have a reference to WorkbenchFrame)

If you cannot do it, or don't want for any reason, I'll add the old 
method back to LayerManager, with a strong advice against its use.

Sincerly,

Michael (the french one)




-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >>  http://get.splunk.com/
_______________________________________________
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel

Reply via email to