#3235 closed defect (bug) (invalid)
admin area update puts a subscription on a revision post (RC7)
Reported by: | wpturk | Owned by: | johnjamesjacoby |
---|---|---|---|
Milestone: | Priority: | high | |
Severity: | normal | Version: | trunk |
Component: | API - Engagements | Keywords: | reporter-feedback |
Cc: |
Description
I am running the latest 2.6 RC7 and discovered problem regarding subscriptions:
This is before updating topic (post_id:54878) in admin area:
mysql> select * from forum_postmeta where meta_key="_bbp_subscription" ; +---------+---------+-------------------+------------+ | meta_id | post_id | meta_key | meta_value | +---------+---------+-------------------+------------+ ..... ..... | 35265 | 54878 | _bbp_subscription | 11743 | +---------+---------+-------------------+------------+
This is after updating the topic (post_id:54878) in admin area:
mysql> select * from forum_postmeta where meta_key="_bbp_subscription" ; +---------+---------+-------------------+------------+ | meta_id | post_id | meta_key | meta_value | +---------+---------+-------------------+------------+ ..... ..... | 35265 | 54878 | _bbp_subscription | 11743 | | 35280 | 54879 | _bbp_subscription | 1 | +---------+---------+-------------------+------------+
admin save puts a subscription on a revision post (54879). I think this should be corrected.
Change History (11)
#1
@
6 years ago
- Component changed from General to API - Engagements
- Keywords needs-patch added
- Milestone changed from Awaiting Review to 2.6
- Owner set to johnjamesjacoby
- Version set to trunk
#4
@
6 years ago
- Resolution fixed deleted
- Status changed from closed to reopened
Hi John,
Unfortunately, this fix (6886) does not work completely. Wordpress Admin save is not putting subscription anymore (thats good) but existing user subscriptions are getting deleted.
Regards.
#5
@
5 years ago
- Keywords reporter-feedback added; needs-patch removed
- Milestone changed from 2.6 to 2.6.1
Hi @wpturk!
I've done a thorough audit of the changes and callstack, and cannot duplicate your experience above.
I do not see where or how all existing user subscriptions could get deleted from editing a topic via WordPress Admin.
There is a function (bbp_remove_topic_from_all_subscriptions()
) that is tasked with performing this action, but only on the spam, trash, and delete actions (not anything related to editing.)
Going to move this into the 2.6.1 milestone pending reporter feedback.
#9
@
5 years ago
- Milestone changed from 2.6.3 to 2.7
@wpturk can you confirm if this is still happening for you?
This bug was introduced in r6811, as part of #3185.
Working on a fix now.