On 27.07.2016 23:53, Michaël Michaud wrote:
> Hi Ede,
> 
> I'm tying to add database writer to the new SaveWizardPlugIn.
> 
> I added a SaveToDataStoreWizard in the initialize method of SaveWizardPlugIn,
> So far so good (it does not do anything yet ;-)
> 
> In SaveWizardPlugIn execute method, I see that inputData is referenced under 
> the key
> SaveFileWizard.DATAKEY_LAYERNAME
> 
> Should I change it to
> 
> SaveWizardPlugin.DATAKEY_LAYERNAME as SaveWizardPlugin should not depends
> on a particular wizard ?

sounds reasonable to me :)
 
> Also I think that the normalization of layerName should be handle by each 
> wizard
> as the rule may be different for a filename and for a tablename. 

it's intended as an optional input for savewizards to use or not. i like to 
have a central place that generalizes the layername in a way that most savers 
could use.

>And maybe
> multiple layer should be accepted at this stage, some wizards may be able to

yeah, no.. let's keep it simple for now and focus on saving one layer as ...

> handle several layer...

maybe at a later stage, but essentially i think that this needs another plugin 
alltogether, maybe using components of this one.
more details coming in the answer to your other email.

..ede

------------------------------------------------------------------------------
_______________________________________________
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel

Reply via email to