setIsDisabled should be private

Author: simonLeary42

resources/lib/UnityGroup.php

@@ -413,7 +413,7 @@ public function getIsDisabled(): bool
         }
     }
 
-    public function setIsDisabled(bool $new_value): void
+    private function setIsDisabled(bool $new_value): void
     {
         $this->entry->setAttribute("isDisabled", $new_value ? "TRUE" : "FALSE");
     }

test/functional/PIBecomeApproveTest.php

@@ -93,7 +93,7 @@ public function testReenableGroup()
         } finally {
             if ($pi_group->memberUIDExists($approve_uid)) {
                 $pi_group->removeMemberUID($approve_uid);
-                $pi_group->setIsDisabled(true);
+                callPrivateMethod($pi_group, "setIsDisabled", true);
                 assert(!$user->isPI());
             }
         }

test/functional/PIDisableTest.php

@@ -85,9 +85,9 @@ public function testGetDisabledSetDisabled()
         $entry = $LDAP->getPIGroupEntry($pi_group->gid);
         $this->assertEquals([], $entry->getAttribute("isDisabled"));
         try {
-            $pi_group->setIsDisabled(false);
+            callPrivateMethod($pi_group, "setIsDisabled", false);
             $this->assertFalse($pi_group->getIsDisabled());
-            $pi_group->setIsDisabled(true);
+            callPrivateMethod($pi_group, "setIsDisabled", true);
             $this->assertTrue($pi_group->getIsDisabled());
             $entry->removeAttribute("isDisabled");
             $this->assertFalse($pi_group->getIsDisabled());
@@ -111,7 +111,7 @@ public function testPIMgmtShowsBothGroupsWithDisabledAttributeSetFalseAndUnset()
             http_get(__DIR__ . "/../../webroot/admin/pi-mgmt.php"),
         );
         try {
-            $pi_group->setIsDisabled(false);
+            callPrivateMethod($pi_group, "setIsDisabled", false);
             $this->assertEquals(["FALSE"], $entry->getAttribute("isDisabled"));
             $this->assertStringContainsString(
                 $pi_group->gid,