Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

check notification preference according to the type #1424

Closed
wants to merge 1 commit into from

2 participants

@chuck911

obvious bug! when $type==Popup,still check 'Preferences.Email.*'?

@chuck911 chuck911 check preference according to the type
obvious bug! when $type==Popup,still check 'Preferences.Email.*'?
eca7bea
@linc
Collaborator

This all got rewritten. Thanks tho!

@linc linc closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Aug 22, 2012
  1. @chuck911

    check preference according to the type

    chuck911 authored
    obvious bug! when $type==Popup,still check 'Preferences.Email.*'?
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  applications/dashboard/models/class.activitymodel.php
View
2  applications/dashboard/models/class.activitymodel.php
@@ -503,7 +503,7 @@ public static function NotificationPreference($ActivityType, $Preferences, $Type
return $Result;
}
- $ConfigPreference = C('Preferences.Email.'.$ActivityType, '0');
+ $ConfigPreference = C('Preferences.'.$Type.'.'.$ActivityType, '0');
if ($ConfigPreference !== FALSE)
$Preference = ArrayValue($Type.'.'.$ActivityType, $Preferences, $ConfigPreference);
else
Something went wrong with that request. Please try again.