[Qgis-developer] crash on deleting a column (virtual field)

[Salvatore Larosa]

Hi Giuseppe,

On Sat, Sep 26, 2015 at 8:16 PM, Giuseppe Sucameli
<brush.tyler at gmail.com> wrote:
> Hi Salvatore,
> I'm unable to find the ticket no., I would like to know if the virtual field
> issue is fixed
> in current master

not fixed yet and I didn't open the ticket, but I have a fix for it. [0]
Attributes map must be updated just after the signal has been emitted
otherwise the attribute index changes before to remove the index
throws the exception on
the attribute form code. It happens only when the attribute table is open.
You fixed a very similar issue recently (#13366)

> (today I'm unable to build the master on Ubuntu 14.04 due
> to an
> error linking geos library, related to the following changes [1]).

known issue, I had to edit my CMakeCache.txt
like Matthias has pointed out on this mailing list [2].

> Did you have time to have a look at it?
> Otherwise, if you need help in crushing this bug, please just ask.

sure, thanks for helping

Ciao,
-SL

[0] - https://gist.github.com/slarosa/9843a5ee38161d4e2d92
[2] - http://lists.osgeo.org/pipermail/qgis-developer/2015-September/039327.html

>
> Cheers!
> Giuseppe
>
> [1]
> https://github.com/qgis/QGIS/commit/e9e638f6f40baf22f4d0667a98fb8e034fefd78c
>
>
> On Wed, Sep 23, 2015 at 12:12 AM Salvatore Larosa <lrssvtml at gmail.com>
> wrote:
>>
>> Hi Giuseppe,
>>
>> On Tue, Sep 22, 2015 at 10:55 PM, Giuseppe Sucameli
>> <brush.tyler at gmail.com> wrote:
>> > Hi Salvatore,
>> > I can reproduce the issue you reported, I get segfault following those
>> > steps.
>>
>> thank you Giuseppe for having tried it, maybe I am managed to find a fix
>> for it.
>>
>> > Please, file a new ticket for it, AFAICS it's not related to the other
>> > issue.
>>
>> I will do.
>>
>> Saluti,
>> -SL
>>
>> > Ciao.
>> > Giuseppe
>> >
>> >
>> > On Tue, Sep 22, 2015 at 3:06 PM Salvatore Larosa <lrssvtml at gmail.com>
>> > wrote:
>> >>
>> >> Hi Tom,
>> >>
>> >> On Tue, Sep 22, 2015 at 10:02 AM, Tom Chadwin <tom.chadwin at nnpa.org.uk>
>> >> wrote:
>> >> > Hi Salvatore
>> >> >
>> >> > Following the steps in Pedro's ticket does *not* crash for me.
>> >> > However,
>> >> > I'll
>> >> > ask some questions to clarify:
>> >> >
>> >> > 1. I did not select "Virtual field", as that was not mentioned in the
>> >> > ticket
>> >>
>> >> I added a comment to the issue with the steps already mentioned in this
>> >> trhead.
>> >>
>> >> > 2. I only gave a static value for the contents o the field, rather
>> >> > than
>> >> > an
>> >> > expression - does this only happen with more complex expressions?
>> >>
>> >> yes it occurs with any value, not needed a complex expression.
>> >>
>> >> > 3. I did not close the attribute table between creating the new field
>> >> > and
>> >> > deleting it
>> >>
>> >> correct, you have to keep attribute dialog open
>> >>
>> >> > Does anyone else *not* get a crash following the steps in
>> >> > https://hub.qgis.org/issues/13366?
>> >>
>> >> Actually there are two different issue apparently, the first one
>> >> (Pedro's issue) happens with native field, but it was already fixed
>> >> [0],
>> >> the second one (mine) happens when creating a virtual field by field
>> >> calculator and then deleting it.
>> >>
>> >> Maybe should I file a new ticket for it?
>> >>
>> >> Thanks for your feedback.
>> >>
>> >> --
>> >> Salvatore Larosa
>> >> linkedIn: http://linkedin.com/in/larosasalvatore
>> >> twitter: @lrssvt
>> >> skype: s.larosa
>> >> IRC: lrssvt on freenode
>> >> _______________________________________________
>> >> Qgis-developer mailing list
>> >> Qgis-developer at lists.osgeo.org
>> >> http://lists.osgeo.org/mailman/listinfo/qgis-developer
>> >
>> > --
>> > Giuseppe Sucameli
>>
>> --
>> Salvatore Larosa
>> linkedIn: http://linkedin.com/in/larosasalvatore
>> twitter: @lrssvt
>> skype: s.larosa
>> IRC: lrssvt on freenode
>
> --
> Giuseppe Sucameli



-- 
Salvatore Larosa
linkedIn: http://linkedin.com/in/larosasalvatore
twitter: @lrssvt
skype: s.larosa
IRC: lrssvt on freenode