Hi Peppe,
I think that using URI is a more general solution.
About c) I think it has been broken by a change you made last week.
Problem is I agree with your change.
In NewLayerPropertiesPlugIn#setInfoProjection, you changed
ProjUtils.getSRSInfoFromLayerSource
to
ProjUtils.getSRSInfoFromLayerStyleOrSource
I think it is OK, especially if the user changed the SRID, but by using
SRIDStyle info, we loose the SRS source file info, which is consistent.
What to do ?
I did not check d), but I would say it is not really a bug and we can
improve it after 1.11 release. What you want is "Geographic metadata"
instead of the tiff file name when SRS information is taken from geotiff
tags. Can you confirm ?
Michaël
Le 02/04/2017 à 18:54, Giuseppe Aruta a écrit :
correct b) on 5122 <I don't like the solution and it is temporary>
what do you think about point c) and d)?
2017-04-02 18:10 GMT+02:00 Giuseppe Aruta <giuseppe.ar...@gmail.com
<mailto:giuseppe.ar...@gmail.com>>:
Hi Michael, Ede
I did some quick tests.
This is a brief observation (some of them are already discussed,
this is only a resume)
a) Regression with Unit detection is solved, thanks Michael
b) a found a java.lang.StringIndexOutOfBoundsException (when I try
to open Layer property plugin and only with rasters) if the file
is located into a folder with a withe space in the middle, like
C:\documents\my folder\test.png.
It is possible to solve it on ProjUtils.getSRSInfoFromLayerSource
method
c) (Only for Layer.class Layer Property plugIn) Still
Projection>Source Path box shows "null" even if there is a source
projection file (.prj or .xml)
d) (only for GeoTIFF) Layer Property plugIn>Projection>Source Path
shows file path while Raster Layer Info>Projection>Source Path
shows the string "Geographic metadata"
My proposal
b) I can solve it right now
c) and d) we can leave as they are and try to solve later OR we
can deactivate Projection>Source Path on both Layer and
RasterLayer plugin
Peppe
2017-04-02 13:08 GMT+02:00 Michaël Michaud
<m.michael.mich...@orange.fr <mailto:m.michael.mich...@orange.fr>>:
Hi,
It is now OK from my side.
Peppe, can you confirm that the regressions I introduced are
fixed ?
Do you think there is still something to be fixed/improved
before 1.11 ?
Michaël
Le 02/04/2017 à 12:54, edgar.sol...@web.de
<mailto:edgar.sol...@web.de> a écrit :
> On 01.04.2017 13 <tel:01.04.2017%2013>:00, Giuseppe Aruta wrote:
>> if Ede gives us few days more :(
> hey guys,
>
> just give me a go at some point :).. ede
>
>
------------------------------------------------------------------------------
> 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
<mailto:Jump-pilot-devel@lists.sourceforge.net>
>
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
<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
<mailto:Jump-pilot-devel@lists.sourceforge.net>
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
<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
