Merge branch 'MDL-63333_master' of git://github.com/markn86/moodle
authorDavid Monllao <davidm@moodle.com>
Tue, 18 Sep 2018 10:17:46 +0000 (12:17 +0200)
committerDavid Monllao <davidm@moodle.com>
Tue, 18 Sep 2018 10:17:46 +0000 (12:17 +0200)
1  2 
admin/settings/subsystems.php
lang/en/admin.php
lib/db/upgrade.php
version.php

@@@ -15,14 -15,6 +15,12 @@@ if ($hassiteconfig) { // speedup for no
  
      $optionalsubsystems->add(new admin_setting_configcheckbox('messaging', new lang_string('messaging', 'admin'), new lang_string('configmessaging','admin'), 1));
  
-     $optionalsubsystems->add(new admin_setting_configcheckbox('messaginghidereadnotifications', new lang_string('messaginghidereadnotifications', 'admin'), new lang_string('configmessaginghidereadnotifications','admin'), 0));
 +    $optionalsubsystems->add(new admin_setting_configcheckbox('messagingallusers',
 +        new lang_string('messagingallusers', 'admin'),
 +        new lang_string('configmessagingallusers', 'admin'),
 +        0)
 +    );
 +
      $options = array(DAYSECS=>new lang_string('secondstotime86400'), WEEKSECS=>new lang_string('secondstotime604800'), 2620800=>new lang_string('nummonths', 'moodle', 1), 15724800=>new lang_string('nummonths', 'moodle', 6),0=>new lang_string('never'));
      $optionalsubsystems->add(new admin_setting_configselect('messagingdeletereadnotificationsdelay', new lang_string('messagingdeletereadnotificationsdelay', 'admin'), new lang_string('configmessagingdeletereadnotificationsdelay', 'admin'), 604800, $options));
  
@@@ -272,11 -272,9 +272,10 @@@ $string['configmaxbytes'] = 'This speci
  $string['configmaxconsecutiveidentchars'] = 'Passwords must not have more than this number of consecutive identical characters. Use 0 to disable this check.';
  $string['configmaxeditingtime'] = 'This specifies the amount of time people have to re-edit forum postings, glossary comments etc.  Usually 30 minutes is a good value.';
  $string['configmaxevents'] = 'Events to Lookahead';
 -$string['configmessaging'] = 'Should the messaging system between site users be enabled?';
 +$string['configmessaging'] = 'If enabled, users can send messages to other users on the site.';
  $string['configmessagingallowemailoverride'] = 'Allow users to have email message notifications sent to an email address other than the email address in their profile';
- $string['configmessaginghidereadnotifications'] = 'Hide read notifications of events like forum posts when viewing messaging history';
  $string['configmessagingdeletereadnotificationsdelay'] = 'Read notifications can be deleted to save space. How long after a notification is read can it be deleted?';
 +$string['configmessagingallusers'] = 'If enabled, users can choose to allow anyone on the site to send them a message. Otherwise, users can choose to allow only their contacts or others in their courses to send them messages.';
  $string['configminpassworddigits'] = 'Passwords must have at least these many digits.';
  $string['configminpasswordlength'] = 'Passwords must be at least these many characters long.';
  $string['configminpasswordlower'] = 'Passwords must have at least these many lower case letters.';
@@@ -748,8 -746,6 +747,7 @@@ $string['mediapluginwmv'] = 'Enable .wm
  $string['mediapluginyoutube'] = 'Enable YouTube links filter';
  $string['messaging'] = 'Enable messaging system';
  $string['messagingallowemailoverride'] = 'Notification email override';
- $string['messaginghidereadnotifications'] = 'Hide read notifications';
 +$string['messagingallusers'] = 'Allow messages from anyone on the site';
  $string['messagingdeletereadnotificationsdelay'] = 'Delete read notifications';
  $string['minpassworddigits'] = 'Digits';
  $string['minpasswordlength'] = 'Password length';
@@@ -2336,21 -2336,13 +2336,29 @@@ function xmldb_main_upgrade($oldversion
          upgrade_main_savepoint(true, 2018091200.00);
      }
  
 -    if ($oldversion < 2018091700.00) {
 +    if ($oldversion < 2018091400.01) {
 +        if (!isset($CFG->messagingallusers)) {
 +            // For existing instances, $CFG->messagingallusers would be same value $CFG->messaging has.
 +            if (isset($CFG->messaging)) {
 +                set_config('messagingallusers', $CFG->messaging);
 +            } else {
 +                // When $CFG->messaging is not set, default value for $CFG->messaging should be true,
 +                // so $CFG->messagingallusers value should be true as well.
 +                set_config('messagingallusers', 1);
 +            }
 +        }
 +
 +        // Main savepoint reached.
 +        upgrade_main_savepoint(true, 2018091400.01);
 +    }
 +
++    if ($oldversion < 2018091700.01) {
+         // Remove unused setting.
+         unset_config('messaginghidereadnotifications');
+         // Main savepoint reached.
 -        upgrade_main_savepoint(true, 2018091700.00);
++        upgrade_main_savepoint(true, 2018091700.01);
+     }
      return true;
  }
diff --cc version.php
@@@ -29,7 -29,7 +29,7 @@@
  
  defined('MOODLE_INTERNAL') || die();
  
- $version  = 2018091400.01;              // YYYYMMDD      = weekly release date of this DEV branch.
 -$version  = 2018091700.00;              // YYYYMMDD      = weekly release date of this DEV branch.
++$version  = 2018091700.01;              // YYYYMMDD      = weekly release date of this DEV branch.
                                          //         RR    = release increments - 00 in DEV branches.
                                          //           .XX = incremental changes.