Im not sure where to open this issue with the Client Side Validation module or here.

Console error.

       $.each(Drupal.settings.clientsideValidation.forms, function (f) {
          if ($(context).find('#' + f).length || $(context).is('#' + f)) {
            update = true;
          }
        });

When authcache runs authcacheP13nReplaceWith the above Client Side Validation JS throws the following error.

TypeError: t.getAttribute is not a function. (In 't.getAttribute("id")', 't.getAttribute' is undefined)

CommentFileSizeAuthor
#5 Screen Shot 2016-09-29 at 2.13.30 PM.png451.5 KBroam2345
#5 Screen Shot 2016-09-29 at 2.13.07 PM.png148.6 KBroam2345
#3 2807483-attach-elements-only.diff904 bytesznerol
Screen Shot 2016-09-27 at 5.40.29 PM.png105.14 KBroam2345

Comments

lathan created an issue. See original summary.

roam2345’s picture

roam2345 commented

Maybe just note this some where... I made a choice between the two and was thankfully easy enough to remove clientside validation from my stack.

znerol’s picture

znerol commented
Status: Active » Needs review
StatusFileSize
new 2807483-attach-elements-only.diff904 bytes

Nice catch. I think that might be a combination of bugs in Authcache and Client Side Validation.

  1. Authcache probably should only call Drupal.attachBehaviors for HTML elements. It looks like it also tries to attach text nodes.
  2. Client Side Validation shouldn't use $(context).find(selector) but rather $(selector, context).

Patch attached for the first problem. Do you mind applying the patch and then report back whether it resolves the incompatibility?

roam2345’s picture

roam2345 commented

@znerol I owe you a beer, I will get back to you at the soonest.

roam2345’s picture

roam2345 commented
StatusFileSize
new Screen Shot 2016-09-29 at 2.13.07 PM.png148.6 KB
new Screen Shot 2016-09-29 at 2.13.30 PM.png451.5 KB

@znerol Still falling over for me.

znerol’s picture

znerol commented

Interesting. Do you mind checking which data is causing that trouble?

Maybe add a console.log statement in authcache_p13n.js:

$elem.each(function() {
  console.log(this) // <- add this line to debug or set a breakpoint and examine the contents of the this-variable.
  Drupal.attachBehaviors(this);
});
znerol’s picture

znerol commented
Status: Needs review » Postponed (maintainer needs more info)

I tried to reproduce this problem with Chrome and Firefox to no avail. Which browser/OS do you use? Is there any standard Drupal form where the problem occurs (e.g., the contact form or the comment form)?

roam2345’s picture

roam2345 commented

@znerol sorry crazy week have not had a gap to look into this, hope to over next day or two.

znerol’s picture

znerol commented

Any updates on this?