Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse files

MDL-32137 Fix message preference settings.

When a messaging output is set to forced, users should not be able to change
that. Regression intoroduced in bb3546f made possible to change checkboxes
(but not avoid forced settings). This fix replace checkboxes for 'forced'
settings with the corresponding message, similar to how it worked for
disallowed before.
  • Loading branch information...
commit de132c20f8af96b4da486ef47b98c962453ca1f3 1 parent c856a1f
Ruslan Kabalin authored
Showing with 16 additions and 24 deletions.
  1. +0 −1  lang/en/message.php
  2. +16 −23 message/renderer.php
View
1  lang/en/message.php
@@ -91,7 +91,6 @@
$string['nomessagesfound'] = 'No messages were found';
$string['noreply'] = 'Do not reply to this message';
$string['nosearchresults'] = 'There were no results from your search';
-$string['notpermitted'] = 'Not permitted';
$string['offline'] = 'Offline';
$string['offlinecontacts'] = 'Offline contacts ({$a})';
$string['online'] = 'Online';
View
39 message/renderer.php
@@ -262,40 +262,33 @@ public function manage_messagingoptions($processors, $providers, $preferences, $
if (isset($defaultpreferences->{$defaultpreference})) {
$permitted = $defaultpreferences->{$defaultpreference};
}
- // If settings are disallowed, just display the message that
- // the setting is not permitted, if not use user settings or
- // force them.
- if ($permitted == 'disallowed') {
+ // If settings are disallowed or forced, just display the
+ // corresponding message, if not use user settings.
+ if (in_array($permitted, array('disallowed', 'forced'))) {
if ($state == 'loggedoff') {
// skip if we are rendering the second line
continue;
}
- $cellcontent = html_writer::nonempty_tag('div', get_string('notpermitted', 'message'), array('class' => 'dimmed_text'));
+ $cellcontent = html_writer::nonempty_tag('div', get_string($permitted, 'message'), array('class' => 'dimmed_text'));
$optioncell = new html_table_cell($cellcontent);
$optioncell->rowspan = 2;
$optioncell->attributes['class'] = 'disallowed';
} else {
- // determine user preferences and use then unless we force
- // the preferences.
+ // determine user preferences and use them.
$disabled = array();
- if ($permitted == 'forced') {
- $checked = true;
+ $checked = false;
+ if ($notificationsdisabled) {
$disabled['disabled'] = 1;
+ }
+ // See if user has touched this preference
+ if (isset($preferences->{$preferencebase.'_'.$state})) {
+ // User have some preferneces for this state in the database, use them
+ $checked = isset($preferences->{$preferencebase.'_'.$state}[$processor->name]);
} else {
- $checked = false;
- if ($notificationsdisabled) {
- $disabled['disabled'] = 1;
- }
- // See if user has touched this preference
- if (isset($preferences->{$preferencebase.'_'.$state})) {
- // User have some preferneces for this state in the database, use them
- $checked = isset($preferences->{$preferencebase.'_'.$state}[$processor->name]);
- } else {
- // User has not set this preference yet, using site default preferences set by admin
- $defaultpreference = 'message_provider_'.$preferencebase.'_'.$state;
- if (isset($defaultpreferences->{$defaultpreference})) {
- $checked = (int)in_array($processor->name, explode(',', $defaultpreferences->{$defaultpreference}));
- }
+ // User has not set this preference yet, using site default preferences set by admin
+ $defaultpreference = 'message_provider_'.$preferencebase.'_'.$state;
+ if (isset($defaultpreferences->{$defaultpreference})) {
+ $checked = (int)in_array($processor->name, explode(',', $defaultpreferences->{$defaultpreference}));
}
}
$elementname = $preferencebase.'_'.$state.'['.$processor->name.']';
Please sign in to comment.
Something went wrong with that request. Please try again.