Skip to:
Content

bbPress.org

Opened 15 years ago

Closed 14 years ago

#1462 closed defect (bug) (fixed)

Subscription checkbox while editing post

Reported by: gautamgupta's profile GautamGupta Owned by:
Milestone: 1.1 Priority: high
Severity: major Version: 1.1-alpha
Component: Front-end Keywords: has-patch
Cc: forumi@…

Description

When an admin/mod edits another user's post, then the subscription checkbox should be modified to show the poster's subscription status rather than the current logged in user's (and same while updating).

Attachments (1)

subscription_checkbox_label.patch (945 bytes) - added by dimadin 15 years ago.

Download all attachments as: .zip

Change History (10)

#1 @johnjamesjacoby
15 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [2862]) When editing a topic/post, check for author_id() in bb_is_user_subscribed instead of the current_user. Fixes #1462.

#2 @johnjamesjacoby
15 years ago

The grammar in that commit message was awful, but at least the bug is fixed. :)

#3 @GautamGupta
15 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

The message should be modified too. It should be Notify the author of follow-up posts via email if current user != post author.

ps. We should change followup to follow-up and e-mail to email.

#4 @johnjamesjacoby
15 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

This was fixed in a recent commit.

#5 @GautamGupta
15 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

This message still needs to be fixed:

Should be Notify the author of follow-up posts via email when the post author is not the logged in user and Notify me of follow-up posts via email when the post author is the logged in user.

#6 @johnjamesjacoby
15 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

(In [2911]) Change subscription checkbox text if the post is the current users or someone else's. Fixes #1462.

#7 @dimadin
15 years ago

  • Cc forumi@… added
  • Resolution fixed deleted
  • Status changed from closed to reopened

You have wrong logic in [2911] and I'm surprised you still didn't notice that.

I always get This user should be notified of follow-up posts via email as both administrator and member except when I post a reply.

#8 @dimadin
15 years ago

  • Keywords has-patch added

On my side this patch fixes issue I mentioned in my previous comment, ie. I always get a right text as both administrator and member.

I'm not sure what status of $is_current is now though.

#9 @johnjamesjacoby
14 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

(In [3524]) Fix incorrect text in subscription label when editing a post. Fixes #1462. Props dimadin. (trunk)

Note: See TracTickets for help on using tickets.