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

Comments

hongpong’s picture

hongpong 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 commented
Component: Miscellaneous » Code
Status: Active » Needs review
StatusFileSize
new 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 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
commented

The patch #2 works for me.

renatog’s picture

renatog 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 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.