Two issues

1) hook_uninstall never deletes them.
2) hook_form_alter was trying to delete them but the form_id had changed from 5 to 6.

Patch attache fixes that, but also moves the variable_del's to a submit function instead of on /viewing/ the form which was a separate but quite related bug.

2) part of the patch was tested, but 1) was assumed to be correct cause, ya know, I /never/ make a mistake. ...

CommentFileSizeAuthor
maxlength_delete_variables.patch1.69 KBhefox
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

cedewey’s picture

cedewey
he/him
Primary language English
Location Denver, CO
CreditAttribution: cedewey at DevCollaborative, LLC commented
Issue summary: View changes
Status: Needs review » Closed (won't fix)

The Drupal 6 version of Maxlength is no longer supported so I'm marking this Closed (won't fix).

I also encourage you, if you haven't already, to upgrade your site to Drupal 8/9. We are actively maintaining that version and you would enjoy all of the other features of the latest version of Drupal.