Uninstalling admin_menu via drush pmu gives Error: Call to undefined function variable_del()

CommentFileSizeAuthor
#2 admin_menu-error-in-uninstall-2526328-2-8.x.patch1.81 KBjami
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

HongPong’s picture

HongPong CreditAttribution: HongPong at kor group commented

See https://www.drupal.org/node/1667896 for the necessary changes for D8 and #2401457: replace system variables with configuration API for the bug thread on this.

jami’s picture

jami CreditAttribution: jami at ThinkShout commented
Component: Miscellaneous » Code
Status: Active » Needs review
FileSize
admin_menu-error-in-uninstall-2526328-2-8.x.patch1.81 KB

I see some overlap with the linked issue, but this patch solves this error specifically (and adds new variable delete syntax), as well as fixing another uninstall error related to how services are set up. Of course feel free to roll these changes in with a larger patch, or apply this smaller patch on its own.

Status: Needs review » Needs work

The last submitted patch, 2: admin_menu-error-in-uninstall-2526328-2-8.x.patch, failed testing.

mimran’s picture

mimran CreditAttribution: mimran commented

we need to port all the variables to configs since drupal 8 not supporting variable_set(),variable_get(),variable_del() anymore

thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
CreditAttribution: thalles at CI&T commented

The patch #2 works for me.

renatog’s picture

renatog
Location Campinas
CreditAttribution: renatog at CI&T commented
Status: Needs work » Reviewed & tested by the community

#2 Really works good

Thanks a lot

  • RenatoG committed 27683c9 on 8.x-3.x authored by jami 
    Issue #2526328 by jami, RenatoG, HongPong, wla_g, mimran, thalles: Error...
renatog’s picture

renatog
Location Campinas
CreditAttribution: renatog at CI&T commented
Status: Reviewed & tested by the community » Fixed

Committed to the dev branch.

Thank you, guys.

Good Work!

Best,

Status: Fixed » Closed (fixed)

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