Skip to:
Content

BuddyPress.org

Opened 12 years ago

Closed 12 years ago

#4923 closed defect (bug) (fixed)

Emails need to check for settings component

Reported by: r-a-y's profile r-a-y Owned by:
Milestone: 1.7.1 Priority: normal
Severity: normal Version: 1.7
Component: Settings Keywords: has-patch early commit
Cc:

Description

Since the settings component is not one of the autoloaded components in BP 1.7, we need to remove the "disable notifications" line from all BP emails when the settings component is diabled.

Otherwise, when a user clicks on the /settings/notifications/ link, the user will get a 404.

Attachments (1)

4923.01.patch​ (7.1 KB) - added by r-a-y 12 years ago.

Download all attachments as: .zip

Change History (5)

@r-a-y
12 years ago

#1 @boonebgorges
12 years ago

  • Component changed from Core to Settings
  • Milestone changed from 1.7 to 1.7.1

Good catch. Not really a regression, but definitely more visible now that Settings isn't active by default.

#2 @johnjamesjacoby
12 years ago

  • Keywords early added

#3 @DJPaul
12 years ago

  • Keywords commit added

Looks good

#4 @boonebgorges
12 years ago

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

r6916 r6917

Note: See TracTickets for help on using tickets.