Admin area updates delete subscriptions on posts.

[wpturk]

I am running the latest 2.6.4 and discovered problem regarding subscriptions:

This is before updating topic (post_id:60483) in admin area:

mysql> select * from forum_postmeta where meta_key="_bbp_subscription" and post_id="60483" ;
+---------+---------+-------------------+------------+
| meta_id | post_id | meta_key          | meta_value |
+---------+---------+-------------------+------------+
|   61987 |   60483 | _bbp_subscription | 4811       |
+---------+---------+-------------------+------------+
1 row in set (0.00 sec)

This is after updating the topic (post_id:60483) in admin area:

mysql> select * from forum_postmeta where meta_key="_bbp_subscription" and post_id="60483" ;
Empty set (0.00 sec)

The subscription is DELETED after admin changes something and updates a post.

[wpturk]

Ok it took me a few hours but I found what is deleting the subscription:

Line 811 in bbpress/includes/topics/functions.php (if I comment this line it works fine)

                // Check if subscribed
                $subscribed = bbp_is_user_subscribed( $author_id, $topic_id );

                // Check for action
                $subscheck  = ( ! empty( $_POST['bbp_topic_subscription'] ) && ( 'bbp_subscribe' === $_POST['bbp_topic_subscription'] ) )
                        ? true
                        : false;

                // Subscribed and unsubscribing
                if ( ( true === $subscribed ) && ( false === $subscheck ) ) {
                        bbp_remove_user_subscription( $author_id, $topic_id );

During backend update there will be no checkbox selected and $subscheck is false and bbp_remove_user_subscription is executed. This explains also why only author subscription is deleted not other subscribers.

[wpturk]

the same issue for replies: bbpress/includes/replies/functions.php

[johnjamesjacoby]

Hey @wpturk, sorry for the delay in looking into this.

Unfortunately, I'm unable to replicate this under normal conditions, and I could use a bit more information from you. Is it possible to detail exactly what steps need to be taken to force this to happen?

I've swapped between two Keymaster users, subscribed to two separate topics, and editing either topic from within WordPress Admin does not remove subscriptions from either, regardless of the user and their subscription status.

I was able to force it by de-registering the bbp_topic_subscriptions_metabox meta-box and then editing a topic I was subscribed to. That forces it because the "bbp_topic_subscription" input ends up not being output, failing the subcheck but passing as subscribed. This behavior is unintended, but definitely an edge-case. It's worth fixing, but likely not in a minor release.

Any ideas or help you can offer is welcome and greatly appreciated. 🙏

[wpturk]

Hi John,

Thanks looking on this.

I swapped to the default wp theme and deactivated plugins, the same problem occurs there too. The topic author subscription gets deleted on post saving in backend.

When I print all $POST variables on save_post hook, bbp_topic_subscription returns empty.

<?php
...
    [_thumbnail_id] => -1
    [page] => 0
    [paged] => 1
    [bbp_topic_subscription] => 
    [post_name] => paypal-devri-turkiyede-kapaniyor-mu
    [icl_make_translatable] => topic,0
    [_icl_nonce_imi] => 3658332707
    [post_mime_type] => 
    [ID] => 283
...

Sorry, I have no idea why bbp_topic_subscriptions_metabox is not delivering the subscriptions.