Merge branch 'MDL-46346_blog' of https://github.com/andyjdavis/moodle
authorSam Hemelryk <sam@moodle.com>
Mon, 14 Jul 2014 22:32:28 +0000 (10:32 +1200)
committerSam Hemelryk <sam@moodle.com>
Mon, 14 Jul 2014 22:33:13 +0000 (10:33 +1200)
1  2 
blog/edit_form.php
blog/external_blog_edit.php
blog/external_blog_edit_form.php

@@@ -130,7 -142,12 +142,13 @@@ class blog_edit_form extends moodlefor
          $mform->setDefault('courseid', $courseid);
      }
  
-     function validation($data, $files) {
+     /**
+      * Validate the blog form data.
 -     * @param $data array Data to be validated
 -     * @param $files void unused
++     * @param array $data Data to be validated
++     * @param array $files unused
++     * @return array|bool
+      */
+     public function validation($data, $files) {
          global $CFG, $DB, $USER;
  
          $errors = array();
Simple merge
@@@ -118,11 -117,9 +117,11 @@@ class blog_edit_external_form extends m
          if ($id = $mform->getElementValue('id')) {
              $mform->setDefault('autotags', implode(',', tag_get_tags_array('blog_external', $id)));
              $mform->freeze('url');
 -            $mform->freeze('filtertags');
 +            if ($mform->elementExists('filtertags')) {
 +                $mform->freeze('filtertags');
 +            }
              // TODO change the filtertags element to a multiple select, using the tags of the external blog
-             // Use $rss->get_channel_tags()
+             // Use $rss->get_channel_tags().
          }
      }
  }