<div dir="ltr"><div>Ugh, there's something other than "merge master into branch" in that commit...<br></div><div>Ok, I'll going to merge it as well.<br></div><div>Regards.<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, May 27, 2015 at 9:39 AM, Etienne Trimaille <span dir="ltr"><<a href="mailto:etienne.trimaille@gmail.com" target="_blank">etienne.trimaille@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div>Thanks Giuseppe for the merge.<br></div>In the third commit, I changed the new feature 'createView' from Hugo to use the <span>'getSqlQuery'. But this commit hasn't been pushed. It would be nice to use always this function, if someone wants to create a view with the selected query.<br><br></span></div><span>Regards,<br></span></div><span>Etienne<br></span> </div><div class="gmail_extra"><br><div class="gmail_quote">2015-05-27 1:07 GMT+02:00 Giuseppe Sucameli <span dir="ltr"><<a href="mailto:brush.tyler@gmail.com" target="_blank">brush.tyler@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div>Merged and closed both PR.<br></div>Best regards.<br></div>Giuseppe<br></div><div class="gmail_extra"><div><div><br><div class="gmail_quote">On Tue, May 26, 2015 at 1:25 PM, Etienne Trimaille <span dir="ltr"><<a href="mailto:etienne.trimaille@gmail.com" target="_blank">etienne.trimaille@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Sorry for the double merge. It comes after the work from Hugo.<br><br></div><div>This feature was on the changelog for 2.8, but the selected query couldn't be loaded on the map canvas. We could only execute the selected part to see the table in the sql window. That's why there is a backport.<br></div><div><br></div><div><br></div></div><div><div><div class="gmail_extra"><br><div class="gmail_quote">2015-05-26 10:54 GMT+02:00 Giuseppe Sucameli <span dir="ltr"><<a href="mailto:brush.tyler@gmail.com" target="_blank">brush.tyler@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div>Hi,<br></div>I agree the changes are good, <br><br></div><div>I don't know if the double merge on PR 1999 could break something, but we can just pick the right commit.<br></div><div><br></div><div><br></div></div><div class="gmail_extra"><div><div><br><div class="gmail_quote">On Tue, May 26, 2015 at 9:46 AM, René-Luc Dhont <span dir="ltr"><<a href="mailto:rldhont@gmail.com" target="_blank">rldhont@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I looked at the code and it's pretty simple. If some code are selected, the selection has run otherwise all the code has run.<br>
<br>
Some one to agree this PR ?<span><br>
<br>
Le 25/05/2015 19:08, Paolo Cavallini a écrit :<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Am 25.05.2015 um 18:40 schrieb Etienne Trimaille:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I opened two PR one month ago about to run only a selected query in<br>
the db manager.<br>
<br>
<a href="https://github.com/qgis/QGIS/pull/1999" target="_blank">https://github.com/qgis/QGIS/pull/1999</a><br>
<a href="https://github.com/qgis/QGIS/pull/2000" target="_blank">https://github.com/qgis/QGIS/pull/2000</a><br>
<br>
Could you please review them and merge ?<br>
</blockquote></blockquote>
Agreed, it would be good to have them.<br>
Anyone can review them?<br>
Thanks.<br>
</blockquote>
<br></span><div><div>
_______________________________________________<br>
Qgis-developer mailing list<br>
<a href="mailto:Qgis-developer@lists.osgeo.org" target="_blank">Qgis-developer@lists.osgeo.org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/qgis-developer" target="_blank">http://lists.osgeo.org/mailman/listinfo/qgis-developer</a><br>
</div></div></blockquote></div><br><br clear="all"><span class="HOEnZb"><font color="#888888"><br></font></span></div></div><span class="HOEnZb"><font color="#888888"><span><font color="#888888">-- <br><div>Giuseppe Sucameli</div>
</font></span></font></span></div><span class="HOEnZb"><font color="#888888">
<br>_______________________________________________<br>
Qgis-developer mailing list<br>
<a href="mailto:Qgis-developer@lists.osgeo.org" target="_blank">Qgis-developer@lists.osgeo.org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/qgis-developer" target="_blank">http://lists.osgeo.org/mailman/listinfo/qgis-developer</a><br></font></span></blockquote></div><span class="HOEnZb"><font color="#888888"><br></font></span></div><span class="HOEnZb"><font color="#888888">
</font></span></div></div></blockquote></div><span class="HOEnZb"><font color="#888888"><br><br clear="all"><br></font></span></div></div><span class="HOEnZb"><font color="#888888"><span><font color="#888888">-- <br><div>Giuseppe Sucameli</div>
</font></span></font></span></div>
</blockquote></div><br></div>
</blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature">Giuseppe Sucameli</div>
</div>