>In NewLayerPropertiesPlugIn#setInfoProjection, you changed
>ProjUtils.getSRSInfoFromLayerSource
>to
>ProjUtils.getSRSInfoFromLayerStyleOrSource
Correct. I did that changes in order to read SRIDStyle info. What I noted
was that SRIDStyle definition on loading file was not working so "Change
SRID" plugin was always displaying "0" value. Doing a small changes on
getSRSInfoFromLayerSource
and using ProjUtils.getSRSInfoFromLayerStyleOrSource seemed to work well
>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
For me it is OK to go on 1.11. No problem to switch to tiff name, what is
important is to omlogate RasterImageLayer and Layer info dispaly
Peppe
2017-04-02 20:06 GMT+02:00 Michaël Michaud <m.michael.mich...@orange.fr>:
> 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>:
>
>> 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>:
>>
>>> 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 a écrit :
>>> > On 01.04.2017 13 <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
>>> > 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
> listJump-pilot-devel@lists.sourceforge.nethttps://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
