MDL-41562 file resource: fix page heading levels.
authorRossiani Wijaya <rwijaya@moodle.com>
Thu, 5 Sep 2013 04:56:56 +0000 (12:56 +0800)
committerDan Poltawski <dan@moodle.com>
Tue, 10 Sep 2013 09:22:08 +0000 (17:22 +0800)
Also removing printheading options setting.

mod/resource/backup/moodle1/lib.php
mod/resource/index.php
mod/resource/lang/en/resource.php
mod/resource/lib.php
mod/resource/locallib.php
mod/resource/mod_form.php
mod/resource/settings.php
mod/resource/tests/generator/lib.php

index 48585ff..8fb5af3 100644 (file)
@@ -145,7 +145,7 @@ class moodle1_mod_resource_handler extends moodle1_mod_handler {
         $resource['timemodified']    = $data['timemodified'];
 
         // populate display and displayoptions fields
-        $options = array('printheading' => 0, 'printintro' => 1);
+        $options = array('printintro' => 1);
         if ($data['options'] == 'frame') {
             $resource['display'] = RESOURCELIB_DISPLAY_FRAME;
 
index 3e9baf3..145af41 100644 (file)
@@ -47,6 +47,7 @@ $PAGE->set_title($course->shortname.': '.$strresources);
 $PAGE->set_heading($course->fullname);
 $PAGE->navbar->add($strresources);
 echo $OUTPUT->header();
+echo $OUTPUT->heading($strresources);
 
 if (!$resources = get_all_instances_in_course('resource', $course)) {
     notice(get_string('thereareno', 'moodle', $strresources), "$CFG->wwwroot/course/view.php?id=$course->id");
index b2f3250..a6850f5 100644 (file)
@@ -88,8 +88,6 @@ $string['popupresource'] = 'This resource should appear in a popup window.';
 $string['popupresourcelink'] = 'If it didn\'t, click here: {$a}';
 $string['popupwidth'] = 'Pop-up width (in pixels)';
 $string['popupwidthexplain'] = 'Specifies default width of popup windows.';
-$string['printheading'] = 'Display resource name';
-$string['printheadingexplain'] = 'Display resource name above content? Some display types may not display resource name even if enabled.';
 $string['printintro'] = 'Display resource description';
 $string['printintroexplain'] = 'Display resource description below content? Some display types may not display description even if enabled.';
 $string['resource:addinstance'] = 'Add a new resource';
index f76a1f9..5c72bb7 100644 (file)
@@ -136,7 +136,6 @@ function resource_set_display_options($data) {
         $displayoptions['popupheight'] = $data->popupheight;
     }
     if (in_array($data->display, array(RESOURCELIB_DISPLAY_AUTO, RESOURCELIB_DISPLAY_EMBED, RESOURCELIB_DISPLAY_FRAME))) {
-        $displayoptions['printheading'] = (int)!empty($data->printheading);
         $displayoptions['printintro']   = (int)!empty($data->printintro);
     }
     if (!empty($data->showsize)) {
@@ -510,7 +509,6 @@ function resource_dndupload_handle($uploadinfo) {
     $data->display = $config->display;
     $data->popupheight = $config->popupheight;
     $data->popupwidth = $config->popupwidth;
-    $data->printheading = $config->printheading;
     $data->printintro = $config->printintro;
     $data->showsize = (isset($config->showsize)) ? $config->showsize : 0;
     $data->showtype = (isset($config->showtype)) ? $config->showtype : 0;
index efef555..9dcef63 100644 (file)
@@ -263,16 +263,12 @@ function resource_print_header($resource, $cm, $course) {
  * @param object $resource
  * @param object $cm
  * @param object $course
- * @param bool $ignoresettings print even if not specified in modedit
+ * @param bool $notused This variable is no longer used
  * @return void
  */
-function resource_print_heading($resource, $cm, $course, $ignoresettings=false) {
+function resource_print_heading($resource, $cm, $course, $notused = false) {
     global $OUTPUT;
-
-    $options = empty($resource->displayoptions) ? array() : unserialize($resource->displayoptions);
-    if ($ignoresettings or !empty($options['printheading'])) {
-        echo $OUTPUT->heading(format_string($resource->name), 2, 'main', 'resourceheading');
-    }
+    echo $OUTPUT->heading(format_string($resource->name), 2);
 }
 
 /**
index a771cc0..1758380 100644 (file)
@@ -125,13 +125,6 @@ class mod_resource_mod_form extends moodleform_mod {
         if (array_key_exists(RESOURCELIB_DISPLAY_AUTO, $options) or
           array_key_exists(RESOURCELIB_DISPLAY_EMBED, $options) or
           array_key_exists(RESOURCELIB_DISPLAY_FRAME, $options)) {
-            $mform->addElement('checkbox', 'printheading', get_string('printheading', 'resource'));
-            $mform->disabledIf('printheading', 'display', 'eq', RESOURCELIB_DISPLAY_POPUP);
-            $mform->disabledIf('printheading', 'display', 'eq', RESOURCELIB_DISPLAY_DOWNLOAD);
-            $mform->disabledIf('printheading', 'display', 'eq', RESOURCELIB_DISPLAY_OPEN);
-            $mform->disabledIf('printheading', 'display', 'eq', RESOURCELIB_DISPLAY_NEW);
-            $mform->setDefault('printheading', $config->printheading);
-
             $mform->addElement('checkbox', 'printintro', get_string('printintro', 'resource'));
             $mform->disabledIf('printintro', 'display', 'eq', RESOURCELIB_DISPLAY_POPUP);
             $mform->disabledIf('printintro', 'display', 'eq', RESOURCELIB_DISPLAY_DOWNLOAD);
@@ -168,9 +161,6 @@ class mod_resource_mod_form extends moodleform_mod {
             if (isset($displayoptions['printintro'])) {
                 $default_values['printintro'] = $displayoptions['printintro'];
             }
-            if (isset($displayoptions['printheading'])) {
-                $default_values['printheading'] = $displayoptions['printheading'];
-            }
             if (!empty($displayoptions['popupwidth'])) {
                 $default_values['popupwidth'] = $displayoptions['popupwidth'];
             }
index 762d706..0fa8b6b 100644 (file)
@@ -56,8 +56,6 @@ if ($ADMIN->fulltree) {
     //--- modedit defaults -----------------------------------------------------------------------------------
     $settings->add(new admin_setting_heading('resourcemodeditdefaults', get_string('modeditdefaults', 'admin'), get_string('condifmodeditdefaults', 'admin')));
 
-    $settings->add(new admin_setting_configcheckbox('resource/printheading',
-        get_string('printheading', 'resource'), get_string('printheadingexplain', 'resource'), 0));
     $settings->add(new admin_setting_configcheckbox('resource/printintro',
         get_string('printintro', 'resource'), get_string('printintroexplain', 'resource'), 1));
     $settings->add(new admin_setting_configselect('resource/display',
index 4be97bd..dd8aacc 100644 (file)
@@ -77,9 +77,6 @@ class mod_resource_generator extends testing_module_generator {
         } else {
             $record->cmidnumber = '';
         }
-        if (!isset($record->printheading)) {
-            $record->printheading = 1;
-        }
         if (!isset($record->printintro)) {
             $record->printintro = 0;
         }