Ajax crashes on new reply to comment

[SGr33n]

Hi,

It's since a few svn versions that if you reply to a comment the ajax sender crashes. When you click on the send button, the class loading is added to the submit button, but the comment is never added, and the loading class remains there until you reload the page.

[r-a-y]

Do you have WP_DEBUG set to true? If so, what error messages are in wp-content/debug.log?

If you have WP_DEBUG set to true, set it to false and see if AJAX works again.

[SGr33n]

Hi r-a-y!

I disabled WP_DEBUG but still don't works. I don't have that file, and the firefox debug console doesn't show me any error. The strange thing is that the error occurs just replying to a comment... commenting an activity works pretty good.

[r-a-y]

Can you outline the steps that you're doing to get the AJAX crash? Explain every step, what page you're on, what textbox you're typing in, etc. so I can try to duplicate this.

[SGr33n]

Sure!

I click on reply button:

<a id="acomment-reply-148227-from-148260" class="acomment-reply bp-primary-action" href="#acomment-148260">Reply</a>

Then buddypress.js adds the form:

<form id="ac-form-148227" class="ac-form" method="post" action="http://ww4.ircaserta.com/attivita/reply/" style="display: block;">
<div class="ac-reply-avatar">
<img class="avatar user-1-avatar avatar-80 photo" width="80" height="80" alt="Foto del profilo di SGr33n" src="http://ww4.ircaserta.com/wp-content/uploads/avatars/1/3047da0262dfccd6795722f43a636a6e-bpthumb.jpg">
</div>
<div class="ac-reply-content">
<div class="ac-textarea">
<textarea id="ac-input-148227" class="ac-input bp-suggestions" name="ac_input_148227"></textarea>
</div>
<input type="submit" value="Send" name="ac_form_submit">
<a class="ac-reply-cancel" href="#">Cancella</a>
<input type="hidden" value="148227" name="comment_form_id">
</div>
<input id="_wpnonce_new_activity_comment" type="hidden" value="23ace042f5" name="_wpnonce_new_activity_comment">
<input type="hidden" value="/" name="_wp_http_referer">
</form>

I write the comment then I push on the submit button and then it stops working.

[r-a-y]

Confirmed.

For anyone else reading, this occurs when you attempt to reply to an activity comment in the sitewide activity stream.

It appears to be hanging here with a Maximum Execution Time error:
​https://buddypress.trac.wordpress.org/browser/tags/2.0.2/bp-templates/bp-legacy/buddypress-functions.php#L783

Will take a closer look.

[boonebgorges]

r-a-y - Check out ​https://buddypress.trac.wordpress.org/changeset/8226 - it could be that this needs to be ported into this AJAX callback as well.

[r-a-y]

The explicit check ​here was causing the problem.

01.patch casts the $parent_id as an integer, which fixes the problem.

This problem did not occur in BP 2.0.2 though. Will try to use git bisect to see where this bug started happening.

[r-a-y]

git bisect to the rescue! r8708 broke this while attempting to fix unit tests caused by r8697.

What is the best way forward?

[boonebgorges]

Thanks for the research, r-a-y. I'd go with your focused patch in 5811.01.patch. The changes in r8708 are an improvement that we do not want to roll back if we can help it. If they've had any other ramifications, I'm sure we'll find out about them eventually.

[r-a-y]

In 8828:

bp-legacy: Fix 'Maximum Execution Time' fatal error when attempting to reply to an activity item.

Changes in r8708 caused an infinite loop to occur while manually
determining the activity comment's depth. This is due to the type casting
in r8708.

To fix this in bp-legacy, we also type cast the various item IDs to prevent
the infinite loop from occurring.

Fixes #5811.