Hi

I think there is a better way to implement the logic in location_uninstall().

See below link for reference and related discussion.
https://drupal.org/node/1981528

I would like to make patch for this.

CommentFileSizeAuthor
#1 location-rewrite_location_uninstall-2014731-1.patch592 byteszterry95
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

zterry95’s picture

zterry95 CreditAttribution: zterry95 commented
FileSize
location-rewrite_location_uninstall-2014731-1.patch592 bytes

Here is the patch, hope it can help.

zterry95’s picture

zterry95 CreditAttribution: zterry95 commented
Status: Active » Needs review
podarok’s picture

podarok
🇺🇦 he/him/his
Primary language Ukrainian
Location Ukraine
CreditAttribution: podarok commented
Status: Needs review » Fixed

looks good
#1 commited pushed to 7.x-3.x
thanks!!

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

(not verified) CreditAttribution: commented
Issue summary: View changes

Updated issue summary.