I can see that Coordinate Transformation plugin points (as target
coordinate system) to Project SRS metadata (Edit>Project properties): this
is fine form me. I agree in this case to leave on Layer context menu.

It would be fine to have 1) it to work on several layers, 2) an access
point on Layer menu, but in this case we need a EnableCheck that activate
the plugin only if all the layers have the same SRS.

By the way: On "Project properties" I used *SRS* (Spatial reference system)
while on Coordinate Transformation Plugin Michael used "Source (and
Target) *Coordinate
System*" : should we uniform the terminology? In this case what is your
idea? SRS (Spatial reference syetem) or Coordinate System?
Peppe

2017-10-23 11:13 GMT+02:00 <edgar.sol...@web.de>:

> let's see what others say.
>
> i'd say Layer menu fit's best.
>
> ..ede
>
> On 23.10.2017 10:14, Rahkonen Jukka (MML) wrote:
> > 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
> >
>
>
> ------------------------------------------------------------
> ------------------
> 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