MDL-43481 lib: minor cleanups to commands
authorDan Poltawski <dan@moodle.com>
Tue, 7 Jan 2014 07:48:06 +0000 (15:48 +0800)
committerDan Poltawski <dan@moodle.com>
Tue, 7 Jan 2014 07:48:06 +0000 (15:48 +0800)
And remove unused strings - thanks to Simon Coggins!

lang/en/admin.php
lib/tests/moodlelib_test.php

index 5bff7f6..7e79c1d 100644 (file)
@@ -1101,7 +1101,6 @@ $string['upgradetimedout'] = 'Upgrade timed out, please restart the upgrade.';
 $string['upgrade197notice'] = '<p>Moodle 1.9.7 contains a number of security fixes to user passwords and backups to protect the user data on your site. As a result some of your settings and permissions relating to backups may have changed.<br />
 See the <a href="http://docs.moodle.org/dev/Moodle_1.9.7_release_notes" target="_blank">Moodle 1.9.7 release notes</a> for full details.</p>';
 $string['upgrade197noticesubject'] = 'Moodle 1.9.7 upgrade security notices';
-$string['upgrade197salt'] = 'To reduce the risk of password theft, you are strongly recommended to set a password salt.<br />See the <a href="{$a}" target="_blank">password salting documentation</a> for details.';
 $string['upgradingdata'] = 'Upgrading data';
 $string['upgradinglogs'] = 'Upgrading logs';
 $string['upgradingversion'] = 'Upgrading to new version';
index c6afa09..5971a4a 100644 (file)
@@ -2181,7 +2181,7 @@ class core_moodlelib_testcase extends advanced_testcase {
      * Test function validate_internal_user_password().
      */
     public function test_validate_internal_user_password() {
-        // Otherwise test bcrypt hashes.
+        // Test bcrypt hashes.
         $validhashes = array(
             'pw' => '$2y$10$LOSDi5eaQJhutSRun.OVJ.ZSxQZabCMay7TO1KmzMkDMPvU40zGXK',
             'abc' => '$2y$10$VWTOhVdsBbWwtdWNDRHSpewjd3aXBQlBQf5rBY/hVhw8hciarFhXa',
@@ -2216,7 +2216,7 @@ class core_moodlelib_testcase extends advanced_testcase {
             $user->password = $hash;
             $this->assertTrue(validate_internal_user_password($user, $password));
 
-            // Otherwise they should not be in md5 format.
+            // They should not be in md5 format.
             $this->assertFalse(password_is_legacy_hash($hash));
 
             // Check that cost factor in hash is correctly set.
@@ -2249,7 +2249,7 @@ class core_moodlelib_testcase extends advanced_testcase {
         // Update the password.
         update_internal_user_password($user, 'password');
 
-        // Otherwise password should have been updated to a bcrypt hash.
+        // Password should have been updated to a bcrypt hash.
         $this->assertFalse(password_is_legacy_hash($user->password));
     }