The Geofield Component that comes with the Openlayers Geofield module does not properly handle deleting features.

If you add 2 points, then select one and press "Delete", both points are deleted.

CommentFileSizeAuthor
#1 openlayers_geofield_delete_single_features-2539384-1.patch1.82 KBm.stenta

Comments

m.stenta’s picture

m.stenta
he/him
Primary language English
commented
Status: Active » Needs review
StatusFileSize
new openlayers_geofield_delete_single_features-2539384-1.patch1.82 KB

Here is a patch that fixes this.

I basically simplified the JS code. Previously, it was loading up all the features of the layer and deleting each of them. Now it only deletes the feature that you have selected.

pol’s picture

pol
Location Brussels
commented
Assigned: Unassigned » das-peter

  • das-peter committed 8d49db2 on 7.x-3.x authored by m.stenta 
    Issue #2539384 by m.stenta: Geofield component can't delete features...
das-peter’s picture

das-peter commented
Status: Needs review » Fixed

@m.stenta Looks good :) Thanks for the patch.
While testing the patch I found some other quirks with the geofield handling - fixed those as well.

m.stenta’s picture

m.stenta
he/him
Primary language English
commented

Great! Thanks @das-peter and @Pol!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.