<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
Hi Radim,<br>
<br>
<br>
<div class="moz-cite-prefix">On 02/06/2013 03:39 PM, Radim Blazek
wrote:<br>
</div>
<blockquote
cite="mid:CAEMrtYrg_w4mTui2dyD+kAC5fQ_mDsQpNiGRKfob+4rcvHez-w@mail.gmail.com"
type="cite">
<pre wrap="">Hi Denis,
I have commited some changes related to identify tool:
<a class="moz-txt-link-freetext" href="https://github.com/qgis/Quantum-GIS/commit/50d4bb6d4">https://github.com/qgis/Quantum-GIS/commit/50d4bb6d4</a>
I hope it does not break any your original intention.</pre>
</blockquote>
As I see it, it seems to be ok!<br>
<br>
Just pay attention, I pull another request <br>
<meta http-equiv="content-type" content="text/html;
charset=ISO-8859-1">
<a href="https://github.com/qgis/Quantum-GIS/pull/421">https://github.com/qgis/Quantum-GIS/pull/421</a><br>
I changed the behavior of the identify methods: it returns the
results instead of saving them in a private attribute.<br>
<br>
I think I will just need to use the merge tool one more time.....<br>
<br>
Also, your changes to the raster result structure made me think that
we could use a single struct for both vector and raster result.<br>
It might be easier to look into the results.<br>
Any opinion on this? Nathan?<br>
<br>
<blockquote
cite="mid:CAEMrtYrg_w4mTui2dyD+kAC5fQ_mDsQpNiGRKfob+4rcvHez-w@mail.gmail.com"
type="cite">
<pre wrap="">One problem I found, if you look at this
<a class="moz-txt-link-freetext" href="https://github.com/qgis/Quantum-GIS/commit/f47d8204#L2L98">https://github.com/qgis/Quantum-GIS/commit/f47d8204#L2L98</a>
Jef changed that from ~layerList.isEmpty() to !layerList.isEmpty() but
that was not working (current layer was not used). I have changed it
to "if ( mode == ActiveLayer && layerList.isEmpty() )". I dont know
what was exactly your idea with "~", could you verify/clarify?</pre>
</blockquote>
Well, the idea is that the first mode of identification (i.e. active
layer) should not be used if a list of layers is given as argument.
Indeed, if a list is given, the method will look in all the layers.<br>
<br>
<br>
</body>
</html>