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.
No brainer patch: term->vid is not always set, eg on the taxonomy term edit form.
There will probably be a follow up shortly. I'm getting this notice 6 times on a single node edit form with 1 vocabulary:
Notice: Undefined index: #post in form_builder() (line 897 of /var/www/glowbox/glowbox/drupal/includes/form.inc).
I've traced it back to hs_taxonomy_form() , still investigating, but if you happen to have a clue let me know. Hope I find it today because one of our clients is giving in a lot of nodes and the error log is getting messed up :)
Comment | File | Size | Author |
---|---|---|---|
#2 | hs.notices.patch | 2.61 KB | swentel |
hs_taxonomy.notices.patch | 659 bytes | swentel | |
Comments
Comment #1
swentel CreditAttribution: swentel commentedOk, that #post notice only happens on a pressflow installation, looking at diff in includes directory, fun! (not)
Comment #2
swentel CreditAttribution: swentel commentedOk, updated patch, found some more notices, still hunting that #post notice - might be a Drupal Core bug exposed by pressflow which has better error handling ...
Comment #3
swentel CreditAttribution: swentel commentedComment #4
Wim LeersImmediately committed your work so far :)
http://drupal.org/cvs?commit=417110
Waiting for your #post notice patch now :)
Comment #5
swentel CreditAttribution: swentel commentedI can't find it, it seems more like a really nice hidden Drupal Core bug, exposed by the latest Pressflow, because it pops up in a few other modules. I've patched my installation self right now because I don't have time anymore. If you have a spare 5 minutes, take the latest pressflow and a single HS vocubulary on an edit form and see the notices (normally 6) popping up. I'm fairly sure it's not your fault, so setting fixed for now!
Edit
In case you want to test. Again, 90% sure this is Drupal Core
Comment #6
Wim LeersOh yes, I've seen that one too. I hacked my test site's form.inc to fix it. You should report that in the Drupal core issue queue.
Comment #7
swentel CreditAttribution: swentel commentedAlright, filed against core, now lets wait a few months :)
Comment #8
Wim LeersHehe :P
Comment #9
KarenS CreditAttribution: KarenS commentedCan anyone provide a link to the core issue for #form error in #5? I'm getting that too.
Comment #10
KarenS CreditAttribution: KarenS commentedThe core issue is #905156: PHP notice in form_builder function, for anyone else looking.