Hi Michaël,

I agree that ability to transform several layers at a time is a good reason for 
having CTS in the context menu.  I am not sure about the Layers menu. SetSRID 
is of course related but there are no other real editing tools in there.  Even 
SetSRID is only touching the layer metadata but not the geometries. I was 
thinking that coordinate transformation tool is are somehow similar to Geometry 
conversion tool that is in the Edit Geometry menu but I do not have strong 
opinions about this.

-Jukka Rahkonen-



-----Alkuperäinen viesti-----
Lähettäjä: Michaël Michaud [mailto:m.michael.mich...@orange.fr] 
Lähetetty: 23. lokakuuta 2017 9:08
Vastaanottaja: Rahkonen Jukka (MML) <jukka.rahko...@maanmittauslaitos.fi>; jump 
devel <jump-pilot-devel@lists.sourceforge.net>
Aihe: Re: [JPP-Devel] Fwd: Re: CTS plugin

Hi Jukka,

I agree that there is no more reason to have it in plugins menu and we can 
discuss about a better place,

Edit geometry is a good candidate but not the only one :

- coordinate transformation is tightly related to setSrid tool which is 
currently in Layer menu (just a remark, I don't say it is a better place to put 
it here)

- having it in the context menu of the layer name panel is an easy way to 
transform several layers at a time (I would let it here regardless of the place 
we find it elsewhere)

Michaël


Le 22/10/2017 à 22:15, Rahkonen Jukka (MML) a écrit :
> Hi,
>
> In r5522 the Coordinate Transformation tool is both in the Plugins menu and 
> in the mouse right click menu that opens through the table of contents. The 
> first location feels like old remaining before adding CTS plugin into the 
> core, and the second one is not the most logical to me. I somehow believed 
> that it would be found from  Tools-Edit Geometry. What do others think?
>
> -Jukka Rahkonen-
>
> ________________________________________
> Lähettäjä: edgar.sol...@web.de <edgar.sol...@web.de>
> Lähetetty: 22. lokakuuta 2017 18:26
> Vastaanottaja: Michaël Michaud; jump devel
> Aihe: Re: [JPP-Devel] Fwd: Re:  CTS plugin
>
> Mike,
>
> i took the freedom to fixup the CTSPlugIn (r5522). where did you get the 
> icon, do we need to credit somebody in readme.txt for it?
>
> ..ede
>
> On 10/22/2017 15:03, edgar.sol...@web.de wrote:
>> one other thing. you seem to have forgotten to commit the CTS icon 
>> 'world.png' . in case the icon comes from famfam/fugue please copy it into 
>> the approriate folder!
>>
>> ..ede
>>
>> [INFO] 14:58:00.491 Loading Plugin 
>> org.openjump.core.ui.plugin.raster.RasterQueryPlugIn took 0.01s  at 
>> com.vividsolutions.jump.workbench.plugin.PlugInManager.loadPlugIns(Pl
>> ugInManager.java:305) {main-menu={menupath=MenuNames.PLUGINS, 
>> icon=world.png, install=true, order_id=333}, order_id=332} [ERROR] 
>> 14:58:00.522 parameter url must not be null. at 
>> com.vividsolutions.jump.workbench.ui.WorkbenchFrame.handleThrowable(W
>> orkbenchFrame.java:1242)
>> java.security.InvalidParameterException: parameter url must not be null.
>>        at 
>> com.vividsolutions.jump.workbench.ui.images.IconLoader.getIcon(IconLoader.java:78)
>>        at 
>> com.vividsolutions.jump.workbench.ui.images.IconLoader.icon(IconLoader.java:55)
>>        at 
>> com.vividsolutions.jump.workbench.ui.plugin.FeatureInstaller.addMenuPluginItem(FeatureInstaller.java:435)
>>        at 
>> com.vividsolutions.jump.workbench.ui.plugin.FeatureInstaller.addMenuPlugin(FeatureInstaller.java:401)
>>        at 
>> com.vividsolutions.jump.workbench.plugin.PlugInManager.loadPlugIns(PlugInManager.java:296)
>>        at 
>> com.vividsolutions.jump.workbench.plugin.PlugInManager.load(PlugInManager.java:177)
>>        at 
>> com.vividsolutions.jump.workbench.JUMPWorkbench.main(JUMPWorkbench.java:457)
>>        at 
>> com.vividsolutions.jump.workbench.JUMPWorkbench.main(JUMPWorkbench.ja
>> va:386)
>>
>> On 10/18/2017 23:57, Michaël Michaud wrote:
>>> Hi Ede,
>>>
>>> Sorry, just removed the CTSPLugin
>>>
>>> I'll check the other problem tomorrow.
>>>
>>> Hopefully, it was an incompatibility between the old plugin and the new 
>>> version 1.5.1 of cts lib.
>>>
>>> Michaël
>>>
>>>
>>> Le 18/10/2017 à 13:31, edgar.sol...@web.de a écrit :
>>>> Mike,
>>>>
>>>> did you see the issue below? ..ede
>>>>
>>>>
>>>> -------- Forwarded Message --------
>>>> Subject: Re: [JPP-Devel] CTS plugin
>>>> Date: Mon, 9 Oct 2017 14:28:33 +0200
>>>> From: edgar.sol...@web.de
>>>> To: jump-pilot-devel@lists.sourceforge.net
>>>>
>>>> hey Mike,
>>>>
>>>> i get a startup error now (when run from eclipse)
>>>>
>>>> "
>>>> java.lang.NoClassDefFoundError: org/cts/registry/RegistryException
>>>>      at java.lang.Class.getDeclaredConstructors0(Native Method)
>>>>      at java.lang.Class.privateGetDeclaredConstructors(Class.java:2671)
>>>>      at java.lang.Class.getConstructor0(Class.java:3075)
>>>>      at java.lang.Class.newInstance(Class.java:412)
>>>>      at 
>>>> com.vividsolutions.jump.workbench.plugin.PlugInManager.loadPlugIns(PlugInManager.java:250)
>>>>      at 
>>>> com.vividsolutions.jump.workbench.plugin.PlugInManager.load(PlugInManager.java:177)
>>>>      at 
>>>> com.vividsolutions.jump.workbench.JUMPWorkbench.main(JUMPWorkbench.java:457)
>>>>      at 
>>>> com.vividsolutions.jump.workbench.JUMPWorkbench.main(JUMPWorkbench.
>>>> java:386)
>>>> "
>>>>
>>>> additionally you seem to have forgotten to remove lib/plus/CTSPLugin-*.jar 
>>>> .
>>>>
>>>> ..ede
>>>>
>>>> On 08.10.2017 08:21, giuseppe.aruta wrote:
>>>>> Hi Michael
>>>>> Thanks for having moved CTS plug-in into OJ core.
>>>>> Peppe
>>>>>
>>>>>
>>>>>
>>>>> Inviato da smartphone Samsung Galaxy.
>>>>>
>>>>>
>>>>> ------------------------------------------------------------------
>>>>> ------------ Check out the vibrant tech community on one of the 
>>>>> world's most engaging tech sites, Slashdot.org! 
>>>>> http://sdm.link/slashdot
>>>>>
>>>>>
>>>>>
>>>>> _______________________________________________
>>>>> Jump-pilot-devel mailing list
>>>>> Jump-pilot-devel@lists.sourceforge.net
>>>>> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
>>>>>
>
> ----------------------------------------------------------------------
> -------- Check out the vibrant tech community on one of the world's 
> most engaging tech sites, Slashdot.org! http://sdm.link/slashdot 
> _______________________________________________
> Jump-pilot-devel mailing list
> Jump-pilot-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
>

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel

Reply via email to