Revert "MDL-61876 admin: set default forceclean=1"
authorAndrew Nicols <andrew@nicols.co.uk>
Tue, 8 May 2018 05:28:12 +0000 (13:28 +0800)
committerAndrew Nicols <andrew@nicols.co.uk>
Tue, 8 May 2018 05:28:12 +0000 (13:28 +0800)
admin/settings/security.php
course/tests/externallib_test.php
lib/tests/weblib_format_text_test.php
media/player/swf/tests/player_test.php

index 3580934..b6a9f89 100644 (file)
@@ -44,7 +44,7 @@ if ($hassiteconfig) { // speedup for non-admins, add all caps used on this page
                 new lang_string('configuserquota', 'admin', $params), $defaultuserquota, PARAM_INT, 30));
 
     $temp->add(new admin_setting_configcheckbox('forceclean', new lang_string('forceclean', 'core_admin'),
-        new lang_string('forceclean_desc', 'core_admin'), 1));
+        new lang_string('forceclean_desc', 'core_admin'), 0));
 
     $temp->add(new admin_setting_configcheckbox('allowobjectembed', new lang_string('allowobjectembed', 'admin'), new lang_string('configallowobjectembed', 'admin'), 0));
     $temp->add(new admin_setting_configcheckbox('enabletrusttext', new lang_string('enabletrusttext', 'admin'), new lang_string('configenabletrusttext', 'admin'), 0));
index be0a707..fdd0380 100644 (file)
@@ -858,9 +858,7 @@ class core_course_externallib_testcase extends externallib_advanced_testcase {
      * Test get_course_contents
      */
     public function test_get_course_contents() {
-        global $CFG;
         $this->resetAfterTest(true);
-        $CFG->forceclean = 0;
 
         list($course, $forumcm, $datacm, $pagecm, $labelcm, $urlcm) = $this->prepare_get_course_contents_test();
 
index 5a8410f..9d6d3b1 100644 (file)
@@ -109,9 +109,6 @@ class core_weblib_format_text_testcase extends advanced_testcase {
      * @param string $expected The expected filter value
      */
     public function test_format_text_blanktarget($link, $expected) {
-        global $CFG;
-        $this->resetAfterTest();
-        $CFG->forceclean = 0;
         $actual = format_text($link, FORMAT_MOODLE, array('blanktarget' => true, 'filter' => false, 'noclean' => true));
         $this->assertEquals($expected, $actual);
     }
index 49c6761..f7b053f 100644 (file)
@@ -104,7 +104,6 @@ class media_swf_testcase extends advanced_testcase {
      */
     public function test_embed_link() {
         global $CFG;
-        $CFG->forceclean = 0;
         $url = new moodle_url('http://example.org/some_filename.swf');
         $text = html_writer::link($url, 'Watch this one');
         $content = format_text($text, FORMAT_HTML, ['trusted' => true]);
@@ -114,13 +113,9 @@ class media_swf_testcase extends advanced_testcase {
         $this->assertRegExp('~width="' . $CFG->media_default_width . '" height="' .
             $CFG->media_default_height . '"~', $content);
 
-        // Not working without trust or with $CFG->forceclean!
+        // Not working without trust!
         $content = format_text($text, FORMAT_HTML);
         $this->assertNotRegExp('~mediaplugin_swf~', $content);
-
-        $CFG->forceclean = 1;
-        $content = format_text($text, FORMAT_HTML, ['trusted' => true]);
-        $this->assertNotRegExp('~mediaplugin_swf~', $content);
     }
 
     /**
@@ -130,8 +125,6 @@ class media_swf_testcase extends advanced_testcase {
      */
     public function test_embed_media() {
         global $CFG;
-        $CFG->forceclean = 0;
-
         $url = new moodle_url('http://example.org/some_filename.swf');
         $trackurl = new moodle_url('http://example.org/some_filename.vtt');
         $text = '<video controls="true"><source src="'.$url.'"/>' .