Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
in form.inc, _form_get_error was replaced with form_get_error.
_form_get_error should have been moved to legacy.inc, where it is still called.
Comment | File | Size | Author |
---|---|---|---|
#1 | legacy-form-error.patch | 606 bytes | webchick |
Comments
Comment #1
webchickIndeed. This seems to cause problems with any module using the old form_* functions (which is almost all non-core modules atm), so pretty important this gets fixed!
Here is a simple patch that just adds in the copy from common.inc before Oct. 7. That seemed to solve the immediate problems I was getting, though there also seem to be other functions missing which may have other effects elsewhere.
Comment #2
webchickChanging status.
Comment #3
chx CreditAttribution: chx commentedRemove legacy.inc http://drupal.org/node/33852 is the fix. Read that issue.
Comment #4
Dave Cohen CreditAttribution: Dave Cohen commentedThat's not the fix for me. I'm using legacy.inc!
I've got code from my 4.6.x modules that uses the old forms API. While yes, it would be nice to upgrade it all to the new API, that's not going to happen right away. So I need legacy.inc.
Comment #5
Dave Cohen CreditAttribution: Dave Cohen commentedChanged status. Not a duplicate.
Comment #6
jvandyk CreditAttribution: jvandyk commentedThis patch is needed to make modules that have not had their forms rewritten work.
Comment #7
hunmonk CreditAttribution: hunmonk commentedlegacy.inc is broken in more ways than just this. we're working on the fix, with the intention of having the old forms mostly supported in 4.7. by 'mostly supported', i mean that there will still be some changes devs will need to make to continue to take advantage of legacy support, but they will be minor compared to a full forms upgrade.
please be aware that this is being done ONLY to give devs a proper amount of time to get their mods converted. the new api fixes security issues, so please convert! legacy support will be removed from HEAD some time after 4.7 is branched--which means no legacy support for 4.8...
Comment #8
chx CreditAttribution: chx commentedComment #9
bdragon CreditAttribution: bdragon commentedLegacy is dead.