remove cluster notice feature

Author: simonLeary42

README.md

@@ -22,9 +22,7 @@ Typical Usage:
 Admin Features:
 
 - Automatic updating of LDAP to reflect current state of users, groups, organizations, PI groups
-- Cluster notices
-  - Added to front page, mailed, and exposed via REST API
-- WYSIWYG HTML editor for webpage contents, cluster notices
+- WYSIWYG HTML editor for webpage contents
 - Branding customization for multiple domains simultaneously
 - Custom UIDNumber / GIDNumber mappings for specific users
 - Login as another user

defaults/config.ini.default

@@ -59,7 +59,7 @@ light_foreground_smallheader = "#666666"  ; Disabled text when in light mode
 light_borders = "#dddddd"  ; Color used for borders and dividers in light mode
 light_footer_background = "#fafafa"  ; Background color for footer in light mode
 light_footer_foreground = "#bbbbbb"  ; Text color for footer in light mode
-light_panel_background = "#e6e6e6"  ; Background color for panels on the page (ie. cluster notices)
+light_panel_background = "#e6e6e6"  ; Background color for panels on the page
 accent = "#881c1c"  ; Primary accent color
 accent_1 = "#941e1e"  ; a little brighter
 accent_2 = "#9c2020"  ; On hover color for element using accent color

resources/lib/UnitySQL.php

@@ -7,7 +7,6 @@
 class UnitySQL
 {
     private const string TABLE_REQS = "requests";
-    private const string TABLE_NOTICES = "notices";
     private const string TABLE_PAGES = "pages";
     private const string TABLE_AUDIT_LOG = "audit_log";
     private const string TABLE_ACCOUNT_DELETION_REQUESTS = "account_deletion_requests";
@@ -139,67 +138,6 @@ public function deleteRequestsByUser(string $user): void
         $stmt->execute();
     }
 
-    public function addNotice(
-        string $title,
-        string $date,
-        string $content,
-        UnityUser $operator,
-    ): void {
-        $table = self::TABLE_NOTICES;
-        $stmt = $this->conn->prepare(
-            "INSERT INTO $table (date, title, message) VALUES (:date, :title, :message)",
-        );
-        $stmt->bindParam(":date", $date);
-        $stmt->bindParam(":title", $title);
-        $stmt->bindParam(":message", $content);
-
-        $stmt->execute();
-
-        $this->addLog($operator->uid, $_SERVER["REMOTE_ADDR"], "added_cluster_notice", $operator);
-    }
-
-    public function editNotice(string $id, string $title, string $date, string $content): void
-    {
-        $table = self::TABLE_NOTICES;
-        $stmt = $this->conn->prepare(
-            "UPDATE $table SET date=:date, title=:title, message=:message WHERE id=:id",
-        );
-        $stmt->bindParam(":date", $date);
-        $stmt->bindParam(":title", $title);
-        $stmt->bindParam(":message", $content);
-        $stmt->bindParam(":id", $id);
-
-        $stmt->execute();
-    }
-
-    public function deleteNotice(string $id): void
-    {
-        $stmt = $this->conn->prepare("DELETE FROM " . self::TABLE_NOTICES . " WHERE id=:id");
-        $stmt->bindParam(":id", $id);
-
-        $stmt->execute();
-    }
-
-    public function getNotice(string $id): array
-    {
-        $stmt = $this->conn->prepare("SELECT * FROM " . self::TABLE_NOTICES . " WHERE id=:id");
-        $stmt->bindParam(":id", $id);
-
-        $stmt->execute();
-
-        return $stmt->fetchAll()[0];
-    }
-
-    public function getNotices(): array
-    {
-        $stmt = $this->conn->prepare(
-            "SELECT * FROM " . self::TABLE_NOTICES . " ORDER BY date DESC",
-        );
-        $stmt->execute();
-
-        return $stmt->fetchAll();
-    }
-
     public function getPages(): array
     {
         $stmt = $this->conn->prepare("SELECT * FROM " . self::TABLE_PAGES);

resources/templates/header.php

@@ -114,7 +114,6 @@
             // Admin only pages
             echo getHyperlink("User Management", "admin/user-mgmt.php");
             echo getHyperlink("PI Management", "admin/pi-mgmt.php");
-            echo getHyperlink("Cluster Notices", "admin/notices.php");
             echo getHyperlink("Content Management", "admin/content.php");
         }
     } else {

test/functional/PageLoadTest.php

@@ -14,7 +14,6 @@ public static function provider()
             [$admin, __DIR__ . "/../../webroot/admin/pi-mgmt.php"],
             [$admin, __DIR__ . "/../../webroot/admin/user-mgmt.php"],
             [$admin, __DIR__ . "/../../webroot/admin/content.php"],
-            [$admin, __DIR__ . "/../../webroot/admin/notices.php"],
             [$nonexistent_user, __DIR__ . "/../../webroot/panel/new_account.php"],
             [$normal_user, __DIR__ . "/../../webroot/panel/account.php"],
             [$normal_user, __DIR__ . "/../../webroot/panel/groups.php"],

tools/docker-dev/sql/bootstrap.sql

@@ -75,27 +75,6 @@ DELIMITER ;
 
 -- --------------------------------------------------------
 
---
--- Table structure for table `notices`
---
-
-CREATE TABLE `notices` (
-  `id` int(11) NOT NULL,
-  `date` timestamp NOT NULL DEFAULT current_timestamp(),
-  `title` varchar(300) NOT NULL,
-  `message` longtext NOT NULL
-) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4 COLLATE=utf8mb4_general_ci;
-
---
--- Dumping data for table `notices`
---
-
-INSERT INTO `notices` (`id`, `date`, `title`, `message`) VALUES
-(9, '2022-09-19 15:49:10', 'Example Notice 1', '<p>Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.</p>'),
-(10, '2022-09-14 11:48:39', 'Example Notice 2', '<p>Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.</p>');
-
--- --------------------------------------------------------
-
 --
 -- Table structure for table `pages`
 --
@@ -144,12 +123,6 @@ ALTER TABLE `account_deletion_requests`
 ALTER TABLE `audit_log`
   ADD PRIMARY KEY (`id`);
 
---
--- Indexes for table `notices`
---
-ALTER TABLE `notices`
-  ADD PRIMARY KEY (`id`);
-
 --
 -- Indexes for table `pages`
 --
@@ -178,12 +151,6 @@ ALTER TABLE `account_deletion_requests`
 ALTER TABLE `audit_log`
   MODIFY `id` int(11) NOT NULL AUTO_INCREMENT;
 
---
--- AUTO_INCREMENT for table `notices`
---
-ALTER TABLE `notices`
-  MODIFY `id` int(11) NOT NULL AUTO_INCREMENT, AUTO_INCREMENT=12;
-
 --
 -- AUTO_INCREMENT for table `pages`
 --

webroot/admin/notices.php

@@ -266,44 +266,6 @@ footer #footerLogos img {
   }
 }
 
-/* notices */
-div.notice {
-  background: var(--light_panel_background);
-  border-radius: 10px;
-  margin: 20px 0 20px 0;
-  padding-bottom: 5px;
-}
-
-div.notice > span.noticeTitle {
-  display: block;
-  background: var(--accent);
-  color: var(--accent_foreground);
-  border-radius: 8px 8px 0 0;
-  padding: 10px 10px 0 10px;
-  font-size: 14pt;
-  font-weight: bold;
-}
-
-div.notice > span.noticeDate {
-  display: block;
-  background: var(--accent);
-  color: var(--accent_foreground);
-  padding: 0 10px 10px 10px;
-}
-
-div.notice > p {
-  margin: 10px 15px 10px 15px;
-  padding: 0;
-}
-
-div.notice > div.noticeText {
-  padding: 0 10px 0 10px;
-}
-
-div.notice > button {
-  margin: 0 0 5px 10px;
-}
-
 #viewAsBar {
   background: orange;
   padding: 10px;