<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Sep 5, 2017 at 9:14 AM, Richard Duivenvoorde <span dir="ltr"><<a href="mailto:rdmailings@duif.net" target="_blank">rdmailings@duif.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
While it's nice to impress new users to use this 'drag a file into a<br>
database' feature :-) .... I think it's also a 'shoot and run' solution:<br>
either OK or nothing...<br>
<br>
Would it be an idea to (sorry dialog haters) have a summary/confirmation<br>
dialog in between this action? Telling the user that 'this file/table<br>
with such columns be copied/moved to that file/table with these columns'?<br>
<br>
In this case, it would also have made it possible to stop this overwrite<br>
action?<br></blockquote><div><br></div><div>I don't think it's a good design: if the result of the action is reversible a confirm dialog is unnecessary, and we are trying to elminate all unnecessary modals.<br></div><div>I'd stick to the confirm dialog for all potentially destructive actions.<br></div><div><br></div><div>I recommend to follow the new geopackage implementation behavior:</div><div><div>- if the drop is on self: abort<br></div>- if the drop overrides an existing table: confirm</div>- if the drop does not override and hence creates a new table: notify on success or error<div><br></div><div>If you've done a wrong drop, you can easily delete the imported layer (at least now it's implemented for geopackage and all ogr/gdal, provided it's supported by the gdal/ogr provider)</div><div><br></div><div>Regards.<br></div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Regards,<br>
<br>
Richard Duivenvoorde<br>
<span class="gmail-"><br>
On <a href="tel:05-09-17%2009" value="+3905091709">05-09-17 09</a>:04, Alessandro Pasotti wrote:<br>
> It looks like a bug.<br>
><br>
> Did you file a ticket?<br>
><br>
> The expected behavior is that the layer you are dragging is imported<br>
> (without overwriting!) into the destination DB (the drop target).<br>
><br>
> In case of overwriting, a confirm dialog should appear.<br>
><br>
> I've recently worked on that for geopackage and I've also added some<br>
> logic to prevent dropping onto itself (that could lead to data loss).<br>
><br>
> Cheers<br>
><br>
><br>
><br>
> On Tue, Sep 5, 2017 at 8:39 AM, Martin Bain<br>
</span>> <<a href="mailto:Martin.Bain@lismore.nsw.gov.au">Martin.Bain@lismore.nsw.gov.<wbr>au</a> <mailto:<a href="mailto:Martin.Bain@lismore.nsw.gov.au">Martin.Bain@lismore.<wbr>nsw.gov.au</a>>><br>
> wrote:<br>
><br>
> Hi List,____<br>
><br>
> I just discovered a brutal feature of the Browser Panel.____<br>
><br>
> __ __<br>
<span class="gmail-">><br>
> When browsing SQL Server if you happen to click and drag on a table<br>
> then when you release the mouse button a dialog box will pop up<br>
</span>> saying “Import Competed”.____<br>
><br>
> __ __<br>
<span class="gmail-">><br>
> The table under the cursor when mouse button is released gets<br>
</span>> overwritten with an empty table, without any sort of prompt!____<br>
><br>
> __ __<br>
<span class="gmail-">><br>
> Within SQL Server I have now revoked DDL permissions from the user<br>
</span>> that did this (and myself).____<br>
><br>
> __ __<br>
<span class="gmail-">><br>
> But was wondering, Is there a way to prevent or modify this<br>
</span>> behaviour in QGIS?____<br>
><br>
> __ __<br>
><br>
> Thanks,____<br>
><br>
> Martin.____<br>
<span class="gmail-">><br>
><br>
> This email is intended for the named recipient only. The information<br>
> it contains may be confidential. If you are not the intended<br>
> recipient you must not reproduce or distribute any part of this<br>
> email, disclose its contents to any other party or take any action<br>
> in reliance on it. If you have received this email in error, please<br>
> contact the sender immediately and delete the message.<br>
><br>
> ______________________________<wbr>_________________<br>
> Qgis-user mailing list<br>
</span>> <a href="mailto:Qgis-user@lists.osgeo.org">Qgis-user@lists.osgeo.org</a> <mailto:<a href="mailto:Qgis-user@lists.osgeo.org">Qgis-user@lists.osgeo.<wbr>org</a>><br>
<span class="gmail-">> List info: <a href="https://lists.osgeo.org/mailman/listinfo/qgis-user" rel="noreferrer" target="_blank">https://lists.osgeo.org/<wbr>mailman/listinfo/qgis-user</a><br>
> <<a href="https://lists.osgeo.org/mailman/listinfo/qgis-user" rel="noreferrer" target="_blank">https://lists.osgeo.org/<wbr>mailman/listinfo/qgis-user</a>><br>
> Unsubscribe: <a href="https://lists.osgeo.org/mailman/listinfo/qgis-user" rel="noreferrer" target="_blank">https://lists.osgeo.org/<wbr>mailman/listinfo/qgis-user</a><br>
> <<a href="https://lists.osgeo.org/mailman/listinfo/qgis-user" rel="noreferrer" target="_blank">https://lists.osgeo.org/<wbr>mailman/listinfo/qgis-user</a>><br>
><br>
><br>
><br>
><br>
> --<br>
> Alessandro Pasotti<br>
</span>> w3: <a href="http://www.itopen.it" rel="noreferrer" target="_blank">www.itopen.it</a> <<a href="http://www.itopen.it" rel="noreferrer" target="_blank">http://www.itopen.it</a>><br>
<div class="gmail-HOEnZb"><div class="gmail-h5">><br>
><br>
> ______________________________<wbr>_________________<br>
> Qgis-user mailing list<br>
> <a href="mailto:Qgis-user@lists.osgeo.org">Qgis-user@lists.osgeo.org</a><br>
> List info: <a href="https://lists.osgeo.org/mailman/listinfo/qgis-user" rel="noreferrer" target="_blank">https://lists.osgeo.org/<wbr>mailman/listinfo/qgis-user</a><br>
> Unsubscribe: <a href="https://lists.osgeo.org/mailman/listinfo/qgis-user" rel="noreferrer" target="_blank">https://lists.osgeo.org/<wbr>mailman/listinfo/qgis-user</a><br>
><br>
<br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature">Alessandro Pasotti<br>w3: <a href="http://www.itopen.it" target="_blank">www.itopen.it</a></div>
</div></div>