Advanced Forum alters the CSS ID for the node form, and the node form tag is not found, causing a JS error "Cannot call method 'setAttribute' of undefined".

Rather than add specific support for Advanced Forum it seemed like the best approach might be to simply use jQuery to just both #node_form and the more specific form ID in it's selector - this operates as a fallback for Advanced Forum and perhaps other modules.

CommentFileSizeAuthor
#4 interdiff.txt578 bytesDeciphered
#4 obscureurl-2097973-4.patch1.13 KBDeciphered
#1 2097973-obscureurl-advanced-forum.patch1.15 KBOwen Barton
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Owen Barton’s picture

Owen Barton CreditAttribution: Owen Barton commented
FileSize
2097973-obscureurl-advanced-forum.patch1.15 KB

Status: Needs review » Needs work

The last submitted patch, 2097973-obscureurl-advanced-forum.patch, failed testing.

Sswater Shi’s picture

Sswater Shi CreditAttribution: Sswater Shi commented

I meet the same problem and find the same reason.

My solution for reference:

Over "7.x-3.3", file "botcha.recipe.controller.inc", Line 897:

          $("#{$js_form_id},.{$js_form_id}").get(0).setAttribute('action', '{$submit}');
Deciphered’s picture

Deciphered CreditAttribution: Deciphered commented
Issue summary: View changes
Status: Needs work » Needs review
FileSize
obscureurl-2097973-4.patch1.13 KB
interdiff.txt578 bytes

Updated patch and added a tweak to the Comments rule so that it works if AJAX has changed the ID (which it does).