MDL-82607 filters: configfilternavigationwithsystemcontext on by default

admin/settings/plugins.php

@@ -287,7 +287,7 @@ if ($hassiteconfig) {
         $items[] = new admin_setting_configcheckbox('filtermatchonepertext', new lang_string('filtermatchonepertext', 'admin'), new lang_string('configfiltermatchonepertext', 'admin'), 0);
         $items[] = new admin_setting_configcheckbox('filternavigationwithsystemcontext',
                 new lang_string('filternavigationwithsystemcontext', 'admin'),
-                new lang_string('configfilternavigationwithsystemcontext', 'admin'), 0);
+                new lang_string('configfilternavigationwithsystemcontext', 'admin'), 1);
         foreach ($items as $item) {
             $item->set_updatedcallback('reset_text_filters_cache');
             $temp->add($item);

course/tests/category_test.php

@@ -1399,6 +1399,9 @@ class category_test extends \advanced_testcase {
         filter_set_global_state('multilang', TEXTFILTER_ON);
         filter_set_applies_to_strings('multilang', true);
 
+        // First test with the performance setting off.
+        set_config('filternavigationwithsystemcontext', 0);
+
         $perf = $filtermanager->get_performance_summary();
         $this->assertEquals(0, $perf[0]['contextswithfilters']);
 

lib/tests/context_helper_test.php

@@ -483,9 +483,6 @@ class context_helper_test extends \advanced_testcase {
         $pagecontext = context\module::instance($page->cmid);
         $systemcontext = context\system::instance();
 
-        // Default is OFF.
-        $this->assertSame('0', $CFG->filternavigationwithsystemcontext);
-
         // First test passed values are returned if disabled.
         set_config('filternavigationwithsystemcontext', '0');