From 198cc5a09d468ac62babb9455424a256244636b4 Mon Sep 17 00:00:00 2001
From: slawkens <slawkens@gmail.com>
Date: Sat, 16 Sep 2023 11:30:35 +0200
Subject: [PATCH] Cosmetics

---
 admin/pages/changelog.php | 14 ++++++++------
 admin/pages/news.php      |  5 +++--
 admin/pages/pages.php     |  9 ++++++---
 system/libs/changelog.php |  1 +
 system/pages/faq.php      |  4 +++-
 5 files changed, 21 insertions(+), 12 deletions(-)

diff --git a/admin/pages/changelog.php b/admin/pages/changelog.php
index 34e7c01a..31318eb5 100644
--- a/admin/pages/changelog.php
+++ b/admin/pages/changelog.php
@@ -49,12 +49,13 @@ if(!empty($action))
 			$body = '';
 			$type = $where = $player_id = $create_date = 0;
 
-			success("Added successful.");
+			success('Added successful.');
 		}
 	}
 	else if($action == 'delete') {
-		Changelog::delete($id, $errors);
-		success("Deleted successful.");
+		if (Changelog::delete($id, $errors)) {
+			success('Deleted successful.');
+		}
 	}
 	else if($action == 'edit')
 	{
@@ -71,13 +72,14 @@ if(!empty($action))
 				$action = $body = '';
 				$type = $where = $player_id = $create_date = 0;
 
-				success("Updated successful.");
+				success('Updated successful.');
 			}
 		}
 	}
 	else if($action == 'hide') {
-		Changelog::toggleHidden($id, $errors, $status);
-		success(($status == 1 ? 'Show' : 'Hide') . " successful.");
+		if (Changelog::toggleHidden($id, $errors, $status)) {
+			success(($status == 1 ? 'Show' : 'Hide') . ' successful.');
+		}
 	}
 
 	if(!empty($errors))
diff --git a/admin/pages/news.php b/admin/pages/news.php
index b268966d..36f3fc52 100644
--- a/admin/pages/news.php
+++ b/admin/pages/news.php
@@ -91,8 +91,9 @@ if(!empty($action))
 		}
 	}
 	else if($action == 'hide') {
-		News::toggleHidden($id, $errors, $status);
-		success(($status == 1 ? 'Show' : 'Hide') . " successful.");
+		if (News::toggleHidden($id, $errors, $status)) {
+			success(($status == 1 ? 'Show' : 'Hide') . ' successful.');
+		}
 	}
 
 	if(!empty($errors))
diff --git a/admin/pages/pages.php b/admin/pages/pages.php
index 6be569f3..40a640af 100644
--- a/admin/pages/pages.php
+++ b/admin/pages/pages.php
@@ -89,8 +89,9 @@ if (!empty($action)) {
 			}
 		}
 	} else if ($action == 'hide') {
-		Pages::toggleHidden($id, $errors, $status);
-		success(($status == 1 ? 'Show' : 'Hide') . ' successful.');
+		if (Pages::toggleHidden($id, $errors, $status)) {
+			success(($status == 1 ? 'Show' : 'Hide') . ' successful.');
+		}
 	}
 
 	if (!empty($errors))
@@ -238,7 +239,9 @@ class Pages
 			$row = ModelsPages::find($id);
 			if ($row) {
 				$row->hidden = $row->hidden == 1 ? 0 : 1;
-				$row->save();
+				if (!$row->save()) {
+					$errors[] = 'Fail during toggle hidden Page.';
+				}
 				$status = $row->hidden;
 			}
 			else {
diff --git a/system/libs/changelog.php b/system/libs/changelog.php
index e612aa5b..f6d2011b 100644
--- a/system/libs/changelog.php
+++ b/system/libs/changelog.php
@@ -95,6 +95,7 @@ class Changelog
 				if (!$row->save()) {
 					$errors[] = 'Fail during toggle hidden Changelog.';
 				}
+				$status = $row->hidden;
 			} else {
 				$errors[] = 'Changelog with id ' . $id . ' does not exists.';
 			}
diff --git a/system/pages/faq.php b/system/pages/faq.php
index f99a9fa8..1c7f5a29 100644
--- a/system/pages/faq.php
+++ b/system/pages/faq.php
@@ -153,7 +153,9 @@ class FAQ
 			$row = ModelsFAQ::find($id);
 			if ($row) {
 				$row->hidden = ($row->hidden == 1 ? 0 : 1);
-				$row->save();
+				if (!$row->save()) {
+					$errors[] = 'Fail during toggle hidden FAQ.';
+				}
 			} else {
 				$errors[] = 'FAQ with id ' . $id . ' does not exists.';
 			}