Tim,<div><br></div><div>Bugger, I'll check it out. </div><div><br></div><div>What kind of column was it, int, double?</div><div><br></div><div>I had a test with current master and it doesn't seem to have this issue, so I'll test my branch a bit more. I didn't change much in that "logic" part of the calculator mainly just removing old gui references but I'll check it out anyway.</div>
<div><br></div><div>- Nathan </div><div><br><div class="gmail_quote">On Mon, Jan 16, 2012 at 10:09 PM, Tim Sutton <span dir="ltr"><<a href="mailto:lists@linfiniti.com">lists@linfiniti.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi<br>
<div class="im"><br>
<br>
<br>
On Mon, Jan 16, 2012 at 1:01 PM, kimaidou <<a href="mailto:kimaidou@gmail.com">kimaidou@gmail.com</a>> wrote:<br>
> Hi Nathan<br>
><br>
> Great news !<br>
> No objection for me, as it is a great enhancement.<br>
><br>
> Cheers<br>
> Kimaidou<br>
><br>
> 2012/1/15 Nathan Woodrow <<a href="mailto:madmanwoo@gmail.com">madmanwoo@gmail.com</a>><br>
>><br>
>> Hi all,<br>
>><br>
>> I spent some time today adding the new expression builder to the field<br>
>> calculator.<br>
>> The current working version can be found<br>
>> at <a href="https://github.com/NathanW2/Quantum-GIS/tree/field-calc" target="_blank">https://github.com/NathanW2/Quantum-GIS/tree/field-calc</a> with a picture of<br>
>> the current UI attached.<br>
>><br>
>> Could I just get a few people to try it out to test I have everything<br>
>> right, and I'm happy to take any feedback people may have.<br>
>><br>
>> If there are no objections I will merge it into master over the next few<br>
>> days.<br>
<br>
</div>I did a brief test here - using a simple expression:<br>
<br>
"depth" / 3<br>
<br>
applied to the depth field resulted in all nulls. I was expecting it<br>
to replace the original value - though I'm not sure if that is an<br>
underlying issue or something introduced with your changes.<br>
<br>
Regards<br>
<br>
Tim<br>
<div class="im"><br>
<br>
>><br>
>> - Nathan<br>
>><br>
>> _______________________________________________<br>
>> Qgis-developer mailing list<br>
>> <a href="mailto:Qgis-developer@lists.osgeo.org">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>
>><br>
><br>
><br>
> _______________________________________________<br>
> Qgis-developer mailing list<br>
> <a href="mailto:Qgis-developer@lists.osgeo.org">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>
><br>
<br>
<br>
<br>
</div>--<br>
Tim Sutton - QGIS Project Steering Committee Member (Release Manager)<br>
==============================================<br>
Please do not email me off-list with technical<br>
support questions. Using the lists will gain<br>
more exposure for your issues and the knowledge<br>
surrounding your issue will be shared with all.<br>
<br>
Visit <a href="http://linfiniti.com" target="_blank">http://linfiniti.com</a> to find out about:<br>
* QGIS programming and support services<br>
* Mapserver and PostGIS based hosting plans<br>
* FOSS Consulting Services<br>
Skype: timlinux<br>
Irc: timlinux on #qgis at <a href="http://freenode.net" target="_blank">freenode.net</a><br>
==============================================<br>
</blockquote></div><br></div>