MDL-69863 tool_brickfield: Change public to private.

abgreeve · May 4, 2021 · 616f20e · 616f20e
1 parent 03ada14
commit 616f20e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 11 deletions.
diff --git a/admin/tool/brickfield/classes/eventobservers.php b/admin/tool/brickfield/classes/eventobservers.php
@@ -36,7 +36,7 @@ class eventobservers {
      * @return bool
      * @throws \dml_exception
      */
-    protected static function course_event_should_be_handled(int $courseid): bool {
+    private static function course_event_should_be_handled(int $courseid): bool {
         return accessibility::is_accessibility_enabled() && analysis::is_enabled() &&
             (empty($courseid) || (isset($courseid) && scheduler::is_course_in_schedule($courseid)));
     }
@@ -47,7 +47,7 @@ protected static function course_event_should_be_handled(int $courseid): bool {
      * @throws \ReflectionException
      * @throws \dml_exception
      */
-    public static function area_altered(base $event) {
+    private static function area_altered(base $event) {
         // Handle if this feature is enabled and this course is in the schedule.
         if (static::course_event_should_be_handled($event->courseid)) {
             manager::find_new_or_updated_areas($event);
@@ -424,7 +424,7 @@ public static function book_chapter_updated(\mod_book\event\chapter_updated $eve
      * @param stdClass $data
      * @throws \dml_exception
      */
-    public static function observer_insert(stdClass $data) {
+    private static function observer_insert(stdClass $data) {
         global $DB;
 
         // Handle if this feature is enabled and this course is in the schedule.

diff --git a/admin/tool/brickfield/settings.php b/admin/tool/brickfield/settings.php
@@ -65,14 +65,6 @@
 
     $settings = new admin_settingpage(manager::PLUGINNAME, get_string('settings', manager::PLUGINNAME));
 
-    $settings->add(new admin_setting_description(
-        manager::PLUGINNAME . '/logo', '',
-        html_writer::img($OUTPUT->image_url('brickfield-logo-small', manager::PLUGINNAME),
-            'logo',
-            ['style' => 'display: block; margin: 0 auto -30px auto; float: right;']
-        )
-    ));
-
     $settings->add(new admin_setting_configcheckbox(
         manager::PLUGINNAME . '/analysistype',
         get_string('analysistype', manager::PLUGINNAME),