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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

m.stenta’s picture

m.stenta
he/him
Primary language English
CreditAttribution: m.stenta commented
Status: Active » Needs review
FileSize
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
the/that
Location Brussels
CreditAttribution: Pol as a volunteer 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 CreditAttribution: das-peter at Cando 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
CreditAttribution: m.stenta commented

Great! Thanks @das-peter and @Pol!

Status: Fixed » Closed (fixed)

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