I get the above error when

1. Add "Add ajax comment" to a view
2. click on the add comment link in a view or if I click on edit, reply, delete

CommentFileSizeAuthor
#9 ajax-error-2299709-9.patch681 byteshitfactory

Comments

socialnicheguru’s picture

socialnicheguru commented

line 101 is a typo:

options.data['nodejs_client_socket_id'] = Drupal.Nodejs.socket.socket.sessionid;

should be the following I think:

options.data['nodejs_client_socket_id'] = Drupal.Nodejs.socket.sessionid;
braindrift’s picture

braindrift commented

@SocialNickeGuru
does options.data['nodejs_client_socket_id'] = Drupal.Nodejs.socket.sessionid; work for you?

socialnicheguru’s picture

socialnicheguru commented

its in nodejs/nodejs.js line 101
Yes it seems to have worked for me.

socialnicheguru’s picture

socialnicheguru commented

this is still an issue.

neograph734’s picture

neograph734
he/him
Location Netherlands
commented

Are you sure? Just checked line 101 of a fresh download (7.x-1.9) and it reads options.data['nodejs_client_socket_id'] = Drupal.Nodejs.socket.sessionid;

neograph734’s picture

neograph734
he/him
Location Netherlands
commented
Related issues: +#2376715: All PHP messages are always allowed and all JS messages are always denied.

It seems however that this related to an issue I've recently created, where Drupal.Nodejs.socket.sessionid is not defined.

socialnicheguru’s picture

socialnicheguru commented

I think the error is in the dev version

neograph734’s picture

neograph734
he/him
Location Netherlands
commented

I should learn to read better... Can you confirm Drupal.Nodejs.socket.sessionid; is set in dev? As in 1.9 it is undefined. Perhaps I should switch to dev as well.

hitfactory’s picture

hitfactory commented
Status: Active » Needs review
StatusFileSize
new ajax-error-2299709-9.patch681 bytes

Confirming the fix suggested by SocialNicheGuru in #1 works with latest dev so adding patch for review.

geresy’s picture

geresy commented

works great, thanks for this! #1 saved the day

socialnicheguru’s picture

socialnicheguru commented

this has been working for me for quite sometime.

socialnicheguru’s picture

socialnicheguru commented
Status: Needs review » Reviewed & tested by the community
formatC'vt’s picture

formatC'vt commented

confirm, patch fix this bug

neograph734’s picture

neograph734
he/him
Location Netherlands
commented

The recently committed patch here #2288625: socket.sessionid has gone away? might need to be changed here as well as posted by rhclayto.

neograph734’s picture

neograph734
he/him
Location Netherlands
commented
Related issues: -#2376715: All PHP messages are always allowed and all JS messages are always denied. +#2288625: socket.sessionid has gone away?
socialnicheguru’s picture

socialnicheguru commented
Status: Reviewed & tested by the community » Needs work

So according to the following does "sessionid" have to be changed to "socket.id":
https://www.drupal.org/node/2288625#comment-9842725