"Answer to" (_bbp_reply_to) will not be stored

[peterkueger]

This is my first report here. I hope, I give you all wanted infomation...

If I click to "ANSWER" to an existing reply (or topic) the answer will not stored into the database.

If I click to that link, I will be forwarded to:
forums/topic/test-topic/page/2/?bbp_reply_to=2002&_wpnonce=2fa94ab9df#new-post
This is a GET request.

After this, the form field "bbp_reply_to" should be filled in
bbpress/includes/common/template.php on line 1757 with this code:

<input type="hidden" name="bbp_reply_to"    id="bbp_reply_to"    value="<?php bbp_form_reply_to(); ?>" />

The function bbp_form_reply_to calls the function bbp_get_form_reply_to in
bbpress/includes/replies/template.php and the result should be generated with this code in line 2496:

		if ( bbp_is_reply_form_post_request() && isset( $_REQUEST['bbp_reply_to'] ) ) {
			$reply_to = bbp_validate_reply_to( $_REQUEST['bbp_reply_to'] );
		}

But this fails, since bbp_is_reply_form_post_request() results with "false", because we're on a GET request.

So the mentioned form field will not be filled and because of this not inserted into the database.

PS
I'm very impressed about bbPress: It's very stable!!!

[johnjamesjacoby]

Hey @peterkueger, thank you for finding this, and for the kind words about bbPress!

I’ll make sure this gets fixed up in the next minor release. Sorry for the trouble and thank you for the detailed report.

[peterkueger]

Hey @johnjamesjacoby,

you are greasy! I have to spend so much time to find a bug - and you want to fix them immediately? Seems I have to search further more... ;-)

Thanks a lot!!!

PS
Before I installed bbPress I tried to use wpForo. Until now, I found nearly 100 bugs there. And the developer are not interested to understand them.

[peterkueger]

Hey @johnjamesjacoby,

the "ANSWER" link of the topic (first reply in the topic) does not set the bbp_reply_to too. There will not be initiated an extra GET request and because of this the hidden input field will not be set.

Did I find a second bug?

[johnjamesjacoby]

In 7130:

Replies: remove post-form check from bbp_get_form_reply_to().

This commit reinstates the reply-to hierarchy when used inside of a non-POST request, namely via theme-side GET requests where JavaScript may not be relied upon.

In trunk, for 2.7.

See #3385.

[johnjamesjacoby]

In 7131:

Replies: remove post-form check from bbp_get_form_reply_to().

This commit reinstates the reply-to hierarchy when used inside of a non-POST request, namely via theme-side GET requests where JavaScript may not be relied upon.

In branches/2.6, for 2.6.6.

See #3385.

[johnjamesjacoby]

Replying to peterkueger:

the first reply in the topic does not set the bbp_reply_to
Did I find a second bug?

I don't think this is another bug, but it may be related. It appears to work OK in my testing. Going to attach a screenshot shortly to show a reply to the first reply.

[johnjamesjacoby]

Closing as fixed. Please reopen if this ends up not being fixed for you, or not fixed correctly/fully.

Thanks again for helping make bbPress better @peterkueger!

[peterkueger]

Now (2.6-7137) works fine for me. Thank you.

[peterkueger]

Sorry, but the problem still exists.

It's present, since "bbp_thread_replies()" returns 'false' on my end and because of this, the 'js/reply.js' will not be enclosed.

bbpress/src/templates/default/bbpress-function.php, line 183

[johnjamesjacoby]

It's present, since "bbp_thread_replies()" returns 'false' on my end and because of this, the 'js/reply.js' will not be enclosed.

If I understand correctly, this is intentional. If replies are not threaded, there is no parent reply for _bbp_reply_to ID to be saved against, other than the topic ID, which is already represented in other ways.

If what you're asking for, is the "reply" link to continue to store the reply hierarchy even when threaded replies are disabled, I think that might be a different type of feature request. If so, please re-close this ticket and create a new enhancement, if you would be so kind. 🙏

[johnjamesjacoby]

In 7147:

Replies: fix clicking "Reply" link from the lead topic.

This commit adjusts reply.js and bbp_get_topic_reply_link() so clicking "Reply" from the lead topic now behaves similarly to clicking "Reply" from a reply. This includes support for hierarchical replies being on or off.

In branches/2.6, for 2.6.6. See #3385.

[johnjamesjacoby]

In 7148:

Replies: fix clicking "Reply" link from the lead topic.

This commit adjusts reply.js and bbp_get_topic_reply_link() so clicking "Reply" from the lead topic now behaves similarly to clicking "Reply" from a reply. This includes support for hierarchical replies being on or off.

In trunk, for 2.7. Fixes #3385.