MDL-19430 block_course_overview: post integration clean up

moodle · Jul 23, 2012 · f3c3a84 · f3c3a84
1 parent 4c21d0d
commit f3c3a84
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 12 deletions.
diff --git a/blocks/course_overview/block_course_overview.php b/blocks/course_overview/block_course_overview.php
@@ -22,6 +22,7 @@
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
 require_once($CFG->dirroot.'/blocks/course_overview/locallib.php');
+
 /**
  * Course overview block
  *

diff --git a/blocks/course_overview/lang/en/block_course_overview.php b/blocks/course_overview/lang/en/block_course_overview.php
@@ -34,12 +34,12 @@
 $string['forcedefaultmaxcoursesdesc'] = 'If set then user will not be able to change his/her personal setting';
 $string['hiddencoursecount'] = 'You have {$a} hidden course';
 $string['hiddencoursecountplural'] = 'You have {$a} hidden courses';
+$string['message'] = 'message';
+$string['messages'] = 'messages';
 $string['movecoursehere'] = 'Move course here';
 $string['numtodisplay'] = 'Number of courses to display: ';
 $string['otherexpanded'] = 'Other Courses Expanded';
 $string['preservestates'] = 'Preserve Expanded States';
-$string['message'] = 'message';
-$string['messages'] = 'messages';
 $string['shortnameprefix'] = 'Includes {$a}';
 $string['shortnamesufixsingular'] = ' (and {$a} other)';
 $string['shortnamesufixprural'] = ' (and {$a} others)';

diff --git a/blocks/course_overview/settings.php b/blocks/course_overview/settings.php
@@ -24,12 +24,12 @@
 defined('MOODLE_INTERNAL') || die;
 
 if ($ADMIN->fulltree) {
-    $settings->add(new admin_setting_configtext('block_course_overview/defaultmaxcourses', get_string('defaultmaxcourses', 'block_course_overview'),
-        get_string('defaultmaxcoursesdesc', 'block_course_overview'), 10, PARAM_INT));
-    $settings->add(new admin_setting_configcheckbox('block_course_overview/forcedefaultmaxcourses', get_string('forcedefaultmaxcourses', 'block_course_overview'),
-        get_string('forcedefaultmaxcoursesdesc', 'block_course_overview'), 1, PARAM_INT));
-    $settings->add(new admin_setting_configcheckbox('block_course_overview/showchildren', get_string('showchildren', 'block_course_overview'),
-        get_string('showchildrendesc', 'block_course_overview'), 1, PARAM_INT));
-    $settings->add(new admin_setting_configcheckbox('block_course_overview/showwelcomearea', get_string('showwelcomearea', 'block_course_overview'),
-        get_string('showwelcomeareadesc', 'block_course_overview'), 1, PARAM_INT));
+    $settings->add(new admin_setting_configtext('block_course_overview/defaultmaxcourses', new lang_string('defaultmaxcourses', 'block_course_overview'),
+        new lang_string('defaultmaxcoursesdesc', 'block_course_overview'), 10, PARAM_INT));
+    $settings->add(new admin_setting_configcheckbox('block_course_overview/forcedefaultmaxcourses', new lang_string('forcedefaultmaxcourses', 'block_course_overview'),
+        new lang_string('forcedefaultmaxcoursesdesc', 'block_course_overview'), 1, PARAM_INT));
+    $settings->add(new admin_setting_configcheckbox('block_course_overview/showchildren', new lang_string('showchildren', 'block_course_overview'),
+        new lang_string('showchildrendesc', 'block_course_overview'), 1, PARAM_INT));
+    $settings->add(new admin_setting_configcheckbox('block_course_overview/showwelcomearea', new lang_string('showwelcomearea', 'block_course_overview'),
+        new lang_string('showwelcomeareadesc', 'block_course_overview'), 1, PARAM_INT));
 }
diff --git a/blocks/course_overview/styles.css b/blocks/course_overview/styles.css
@@ -8,7 +8,7 @@
 }
 
 .block_course_overview .coursebox {
-    padding: 15px 0px 10px 10px;
+    padding: 15px 0 10px 10px;
     width: 98%;
 }
 
@@ -47,7 +47,7 @@
 
 .editing .block_course_overview .move {
     float: left;
-    padding: 2px 10px 0px 0px;
+    padding: 2px 10px 0 0;
 }
 
 .editing .block_course_overview .moveicons {