Merge branch 'MDL-25886-user_picture_fields' of git://github.com/mudrd8mz/moodle
authorPetr Skoda <commits@skodak.org>
Mon, 10 Jan 2011 12:43:00 +0000 (13:43 +0100)
committerPetr Skoda <commits@skodak.org>
Mon, 10 Jan 2011 12:43:00 +0000 (13:43 +0100)
12 files changed:
admin/roles/assign.php
blocks/calendar_month/block_calendar_month.php
blocks/community/forms.php
blocks/rss_client/editfeed.php
blog/preferences.php
blog/preferences_form.php
course/category.php
course/modedit.php
lib/accesslib.php
lib/javascript-static.js
mod/data/edit.php
mod/forum/lib.php

index 6922350..65aa221 100755 (executable)
@@ -232,7 +232,8 @@ $assignurl = new moodle_url($PAGE->url, array('roleid'=>$roleid));
     $select = new single_select($PAGE->url, 'roleid', $nameswithcounts, $roleid, null);
     $select->label = get_string('assignanotherrole', 'role');
     echo $OUTPUT->render($select);
-    echo '<p><a href="' . $PAGE->url . '">' . get_string('backtoallroles', 'role') . '</a></p>';
+    $backurl = new moodle_url('/admin/roles/assign.php', array('contextid' => $contextid));
+    echo '<p><a href="' . $backurl->out() . '">' . get_string('backtoallroles', 'role') . '</a></p>';
     echo '</div>';
 
 } else if (empty($assignableroles)) {
index 67488c4..9fa7ecb 100644 (file)
@@ -40,8 +40,6 @@ class block_calendar_month extends block_base {
             calendar_set_referring_course(0);
 
         } else {
-            //MDL-14693: fix calendar on resource page
-            $courseshown =  optional_param( 'id', $this->page->course->id, PARAM_INT );
             // Forcibly filter events to include only those from the particular course we are in.
             $filtercourse    = array($courseshown => $this->page->course);
             $groupeventsfrom = array($courseshown => 1);
index 8d16563..c4abd1e 100644 (file)
@@ -131,7 +131,7 @@ class community_hub_search_form extends moodleform {
             }
         }
         if (!empty($additionalhubs)) {
-            $hubs = $hubs + $additionalhubs;
+            $hubs = array_merge($hubs, $additionalhubs);
         }
 
         if (!empty($hubs)) {
@@ -174,7 +174,7 @@ class community_hub_search_form extends moodleform {
                                     array('class' => $hub['trusted'] ? 'hubtrusted' : 'hubnottrusted'));
                 } else {
                     $hubdescription = html_writer::tag('a', $hub['name'],
-                                    array('class' => 'hublink', 'href' => $hub['url']));
+                                    array('class' => 'hublink hubtrusted', 'href' => $hub['url']));
                 }
 
                 if (empty($firsthub)) {
index b031455..38eeb48 100644 (file)
@@ -44,7 +44,7 @@ class feed_edit_form extends moodleform {
         $mform =& $this->_form;
 
         // Then show the fields about where this block appears.
-        $mform->addElement('header', 'header', get_string('feed', 'block_rss_client'));
+        $mform->addElement('header', 'rsseditfeedheader', get_string('feed', 'block_rss_client'));
 
         $mform->addElement('text', 'url', get_string('feedurl', 'block_rss_client'), array('size' => 60));
         $mform->setType('url', PARAM_URL);
index 366beda..472ee30 100755 (executable)
@@ -73,6 +73,7 @@ require_capability('moodle/blog:view', $context);
 /// If data submitted, then process and store.
 
 $mform = new blog_preferences_form('preferences.php');
+$mform->set_data(array('pagesize' => get_user_preferences('blogpagesize')));
 
 if (!$mform->is_cancelled() && $data = $mform->get_data()) {
     $pagesize = $data->pagesize;
index a1fff64..d5c5005 100644 (file)
@@ -39,7 +39,9 @@ class blog_preferences_form extends moodleform {
         $strpagesize = get_string('pagesize', 'blog');
 
         $mform->addElement('text', 'pagesize', $strpagesize);
-        $mform->setDefault('pagesize', get_user_preferences('pagesize'));
+        $mform->setType('pagesize', PARAM_INT);
+        $mform->addRule('pagesize', null, 'numeric', null, 'client');
+        $mform->setDefault('pagesize', 10);
 
         $this->add_action_buttons();
     }
index 7ca8fc4..12558aa 100644 (file)
         echo $OUTPUT->single_button(new moodle_url('edit.php', $options), get_string('addnewcourse'), 'get');
     }
 
-    if (!empty($CFG->enablecourserequests) && $category->id == $CFG->enablecourserequests) {
+    if (!empty($CFG->enablecourserequests) && $category->id == $CFG->defaultrequestcategory) {
         print_course_request_buttons(get_context_instance(CONTEXT_SYSTEM));
     }
     echo '</div>';
index d6338e6..4b5c615 100644 (file)
@@ -42,12 +42,14 @@ if (!empty($return)) {
 }
 
 if (!empty($add)) {
-    $url->param('add', $add);
-    $PAGE->set_url($url);
-
     $section = required_param('section', PARAM_INT);
     $course  = required_param('course', PARAM_INT);
 
+    $url->param('add', $add);
+    $url->param('section', $section);
+    $url->param('course', $course);
+    $PAGE->set_url($url);
+
     $course = $DB->get_record('course', array('id'=>$course), '*', MUST_EXIST);
     $module = $DB->get_record('modules', array('name'=>$add), '*', MUST_EXIST);
 
index 93db002..87a8287 100755 (executable)
@@ -4459,7 +4459,7 @@ function get_user_roles($context, $userid = 0, $checkparentcontexts = true, $ord
  *
  * @param int $sroleid source roleid
  * @param int $troleid target roleid
- * @return int id or false
+ * @return void
  */
 function allow_override($sroleid, $troleid) {
     global $DB;
@@ -4475,7 +4475,7 @@ function allow_override($sroleid, $troleid) {
  *
  * @param int $sroleid source roleid
  * @param int $troleid target roleid
- * @return int id or false
+ * @return void
  */
 function allow_assign($fromroleid, $targetroleid) {
     global $DB;
@@ -4491,7 +4491,7 @@ function allow_assign($fromroleid, $targetroleid) {
  *
  * @param int $sroleid source roleid
  * @param int $troleid target roleid
- * @return int id or false
+ * @return void
  */
 function allow_switch($fromroleid, $targetroleid) {
     global $DB;
index d79230d..94dd7e0 100644 (file)
@@ -365,7 +365,12 @@ M.util.init_select_autosubmit = function(Y, formid, selectid, nothing) {
                 // event.... usability
                 Y.on('key', processchange, select, 'press:13', form, select.get('selectedIndex'));
                 select.on('blur', processchange, form, select.get('selectedIndex'));
-                select.on('click', processchange, form, select.get('selectedIndex'));
+                //little hack for chrome that need onChange event instead of onClick - see MDL-23224
+                if (Y.UA.webkit) {
+                    select.on('change', processchange, form, select.get('selectedIndex'));
+                } else {
+                    select.on('click', processchange, form, select.get('selectedIndex'));
+                }
             }
         }
     });
index 34aef3f..c1acdb5 100755 (executable)
@@ -198,6 +198,7 @@ if ($datarecord = data_submitted() and confirm_sesskey()) {
     /// Of course, you can't be stopped if you are an editting teacher! =)
 
         if (data_atmaxentries($data) and !has_capability('mod/data:manageentries',$context)){
+            echo $OUTPUT->header();
             echo $OUTPUT->notification(get_string('atmaxentry','data'));
             echo $OUTPUT->footer();
             exit;
index c5b5bcc..8602d85 100644 (file)
@@ -3155,7 +3155,7 @@ function forum_print_post($post, $discussion, $forum, &$cm, $course, $ownpost=fa
             return;
         }
         $output .= html_writer::tag('a', '', array('id'=>'p'.$post->id));
-        $output .= html_writer::start_tag('div', array('class'=>'forumpost clearfix '.$forumpostclass));
+        $output .= html_writer::start_tag('div', array('class'=>'forumpost clearfix'));
         $output .= html_writer::start_tag('div', array('class'=>'row header'));
         $output .= html_writer::tag('div', '', array('class'=>'left picture')); // Picture
         if ($post->parent) {