ENH Use class name instead of self

silverstripe · Jun 5, 2024 · 0a4626c · 0a4626c
1 parent bad865d
commit 0a4626c
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/src/Forms/GridField/GridFieldAddByDBField.php b/src/Forms/GridField/GridFieldAddByDBField.php
@@ -234,6 +234,6 @@ public function getHTMLFragments($gridField)
         $forTemplate->Fields->push($textField);
         $forTemplate->Fields->push($addAction);
 
-        return [$this->targetFragment => $forTemplate->renderWith(self::class)];
+        return [$this->targetFragment => $forTemplate->renderWith(GridFieldAddByDBField::class)];
     }
 }
diff --git a/src/Model/Blog.php b/src/Model/Blog.php
@@ -475,7 +475,7 @@ public function canEditEditors($member = null)
             return $extended;
         }
 
-        return Permission::checkMember($member, self::MANAGE_USERS);
+        return Permission::checkMember($member, Blog::MANAGE_USERS);
     }
 
     /**
@@ -499,7 +499,7 @@ public function canEditWriters($member = null)
             return true;
         }
 
-        return Permission::checkMember($member, self::MANAGE_USERS);
+        return Permission::checkMember($member, Blog::MANAGE_USERS);
     }
 
     /**
@@ -523,7 +523,7 @@ public function canEditContributors($member = null)
             return true;
         }
 
-        return Permission::checkMember($member, self::MANAGE_USERS);
+        return Permission::checkMember($member, Blog::MANAGE_USERS);
     }
 
     /**

diff --git a/tests/php/Widgets/BlogArchiveWidgetTest.php b/tests/php/Widgets/BlogArchiveWidgetTest.php
@@ -18,7 +18,7 @@ class BlogArchiveWidgetTest extends SapphireTest
     protected function setUp(): void
     {
         if (!class_exists(Widget::class)) {
-            self::$fixture_file = null;
+            BlogArchiveWidgetTest::$fixture_file = null;
             parent::setUp();
             $this->markTestSkipped('Test requires silverstripe/widgets to be installed.');
         }