This is an upgrade of OJ behaviour when it is closed "before saving the
project file", *in italics my comments*:

a) Open OJ and *load a vector* layer. Than close OJ.
OJ shuts down with no warning to save the project.
*This is a bit ambiguous as an user wants maybe only to loads vectors with
no need do perform analysis. But only to have a project already set*. I*n
my opinion OJ should anyhow warns users if save the project*

b) Open OJ and *create a vector* file, Than close OJ.
There are two types of warning according to what happens:
- if the new layer is only created OJ warns that an automatically generated
layer is not saved
- if the new file is created and modified (ex. adding a feature), OJ warns
that one (or more) datasets has been modified.
*This is OK. We probably can extend both warning dialog adding a list of
the name of unsaved/modified layers*

c) Open OJ and l*oad a Sextante Raster* Layer. Than close OJ
See OJ behaviour and comments at point a)

d) Open OJ, load a Sextante Raster Layer, than *create a new Sextante
Raster* layer using Sextante Toolbox or other OJ tools. Than Close OJ.
OJ shuts down with no no warning to save the project.
*This is fatal for users who are not aware about it, as they can loose all
their work. We should add a warning message, like in point b) to add users
that 1d) new raster layers have been created, 2d) these layers are
temporary layers (they are saved into OS temp folder, users can lost them
if they shut down the computer)*

e) Open OJ and *load a WMS* Layer. Than close OJ
See OJ behaviour and comments at point a)


My conclusion:
1) I think we need anyhow a warning message that ask user to save the
project (see point a, c and e)
2) extend the warning message for the presence of "temporary raster layer"
and to save them (point d)
3) possibly add to this warning message a list of a) modified vector
layers, b) new vector layers, c) temporary raster layers (point b and d)

Peppe



2015-06-23 15:17 GMT+02:00 Giuseppe Aruta <giuseppe.ar...@gmail.com>:

> Hi Michael,
>  I saw you solved this behavior on WorkbenchFrame class.
>
> There 2 problems with Sextante Raster layers, I will try to describe.
>
> Some Sextante Raster plugins save a result file into OS>TEMP folder (The
> plugins: extract selected part of image, warp image to fence, Affine
> transformation (from warping vectors) if the selected layer is a Sextante
> Raster).
>
> 1) OJ doesn't warn that those new raster layers are not saved into the
> project. If a user closes the project without saving it at this stage,
> these layers are lost (*)
>
> 2) Those new files are saved into TEMP folder. So, even if a user reminds
> to save the project (but leaves the files into TEMP folder)  the files are
> deleted, at least for Linux and, I think, for MacOSX, when he shuts down
> the PC.
>
> The old warning message was probably not correct but it was helping to
> remind user the presence of temporary raster layers.
> We need an extra warning message, when OJ is closing down, to warn a user
> a) that there are some "temporary raster files" and to save them to a save
> folder b) to save anyhow the project if he/she wants to save these files as
> project layers.
>
> Best regards
>
> Peppe
>
> (*) note that this happens also for raster files produced using any raster
> tool from Sextante toolbox.
>
> 2015-06-23 13:42 GMT+02:00 <jump-pilot-...@lists.sourceforge.net>:
>
>> Revision: 4505
>>           http://sourceforge.net/p/jump-pilot/code/4505
>> Author:   michaudm
>> Date:     2015-06-23 11:42:46 +0000 (Tue, 23 Jun 2015)
>> Log Message:
>> -----------
>> Fix bug #406 wrong message "layers not saved"
>>
>> Modified Paths:
>> --------------
>>     core/trunk/ChangeLog
>>
>> core/trunk/src/com/vividsolutions/jump/workbench/ui/WorkbenchFrame.java
>>
>> Modified: core/trunk/ChangeLog
>> ===================================================================
>> --- core/trunk/ChangeLog        2015-06-18 15:50:25 UTC (rev 4504)
>> +++ core/trunk/ChangeLog        2015-06-23 11:42:46 UTC (rev 4505)
>> @@ -1,5 +1,8 @@
>>  # for display continuity sake please use 2 spaces instead of tabs
>>
>> +2015-06-23 mmichaud <m.michael.mich...@orange.fr>
>> +  * fix bug #406 wrong message "layers not saved"
>> +
>>  2015-06-17 bertazza
>>    * added Raster Styler to manage symbologies for 1-banded raster
>>    * (available from raster layer contextual menu)
>>
>> Modified:
>> core/trunk/src/com/vividsolutions/jump/workbench/ui/WorkbenchFrame.java
>> ===================================================================
>> ---
>> core/trunk/src/com/vividsolutions/jump/workbench/ui/WorkbenchFrame.java
>>  2015-06-18 15:50:25 UTC (rev 4504)
>> +++
>> core/trunk/src/com/vividsolutions/jump/workbench/ui/WorkbenchFrame.java
>>  2015-06-23 11:42:46 UTC (rev 4505)
>> @@ -1321,7 +1321,7 @@
>>          ArrayList list = new ArrayList();
>>          for (Iterator i = getLayerManagers().iterator(); i.hasNext();) {
>>              LayerManager layerManager = (LayerManager) i.next();
>> -            list.add(layerManager.getLayersWithNullDataSource());
>> +            list.addAll(layerManager.getLayersWithNullDataSource());
>>              //
>> list.add(layerManager.getRasterLayersWithNullDataSource());
>>          }
>>          return list;
>>
>>
>>
>> ------------------------------------------------------------------------------
>> Monitor 25 network devices or servers for free with OpManager!
>> OpManager is web-based network management software that monitors
>> network devices and physical & virtual servers, alerts via email & sms
>> for fault. Monitor 25 devices for free with no restriction. Download now
>> http://ad.doubleclick.net/ddm/clk/292181274;119417398;o
>> _______________________________________________
>> Jump-pilot-devel mailing list
>> Jump-pilot-devel@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
>>
>
>
------------------------------------------------------------------------------
Monitor 25 network devices or servers for free with OpManager!
OpManager is web-based network management software that monitors 
network devices and physical & virtual servers, alerts via email & sms 
for fault. Monitor 25 devices for free with no restriction. Download now
http://ad.doubleclick.net/ddm/clk/292181274;119417398;o
_______________________________________________
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel

Reply via email to