From f7cbe5170d4ce7c059c81e97e69a90901f59beee Mon Sep 17 00:00:00 2001 From: slawkens Date: Mon, 11 Sep 2023 16:16:38 +0200 Subject: [PATCH 01/14] set display quest default to false --- system/settings.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/system/settings.php b/system/settings.php index 4f0d7597..29149e1f 100644 --- a/system/settings.php +++ b/system/settings.php @@ -1127,7 +1127,7 @@ Sent by MyAAC,
'name' => 'Display Quests', 'type' => 'boolean', 'desc' => 'Show characters quests. Can be configured below', - 'default' => true, + 'default' => false, ], 'quests' => [ 'name' => 'Quests List', From 673e40350ac2479f41adbf43debf0f443955d98d Mon Sep 17 00:00:00 2001 From: slawkens Date: Tue, 12 Sep 2023 11:42:03 +0200 Subject: [PATCH 02/14] Small adjustment to menus install --- system/migrations/17.php | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/system/migrations/17.php b/system/migrations/17.php index 6ff83b84..8c7cf8a0 100644 --- a/system/migrations/17.php +++ b/system/migrations/17.php @@ -14,8 +14,9 @@ CREATE TABLE `myaac_menu` PRIMARY KEY (`id`) ) ENGINE=InnoDB DEFAULT CHARACTER SET=utf8; "); - - require_once LIBS . 'plugins.php'; - Plugins::installMenus('kathrine', require TEMPLATES . 'kathrine/menus.php'); - Plugins::installMenus('tibiacom', require TEMPLATES . 'tibiacom/menus.php'); } + +require_once LIBS . 'plugins.php'; +Plugins::installMenus('kathrine', require TEMPLATES . 'kathrine/menus.php'); +Plugins::installMenus('tibiacom', require TEMPLATES . 'tibiacom/menus.php'); + From 95b1460b1302e44d3cb5a4db10cc4082e5d83011 Mon Sep 17 00:00:00 2001 From: slawkens Date: Tue, 12 Sep 2023 12:08:09 +0200 Subject: [PATCH 03/14] Fix news delete message --- system/libs/news.php | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/system/libs/news.php b/system/libs/news.php index 352b2a1e..fb4afc5d 100644 --- a/system/libs/news.php +++ b/system/libs/news.php @@ -78,12 +78,14 @@ class News if(isset($id)) { $row = ModelsNews::find($id); - if($row) + if($row) { if (!$row->delete()) { $errors[] = 'Fail during delete News.'; } - else + } + else { $errors[] = 'News with id ' . $id . ' does not exists.'; + } } else $errors[] = 'News id not set.'; From 2d02d8d8b3f7bb3a02adb9884eaa3b6985dbd0fa Mon Sep 17 00:00:00 2001 From: slawkens Date: Tue, 12 Sep 2023 12:09:39 +0200 Subject: [PATCH 04/14] Fix news delete message part 2 --- admin/pages/news.php | 5 +++-- system/libs/news.php | 6 +++--- 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/admin/pages/news.php b/admin/pages/news.php index 66398b09..2c5102e6 100644 --- a/admin/pages/news.php +++ b/admin/pages/news.php @@ -56,8 +56,9 @@ if(!empty($action)) } } else if($action == 'delete') { - News::delete($id, $errors); - success("Deleted successful."); + if (News::delete($id, $errors)) { + success("Deleted successful."); + } } else if($action == 'edit') { diff --git a/system/libs/news.php b/system/libs/news.php index fb4afc5d..b42fefc6 100644 --- a/system/libs/news.php +++ b/system/libs/news.php @@ -75,8 +75,7 @@ class News static public function delete($id, &$errors) { - if(isset($id)) - { + if(isset($id)) { $row = ModelsNews::find($id); if($row) { if (!$row->delete()) { @@ -87,8 +86,9 @@ class News $errors[] = 'News with id ' . $id . ' does not exists.'; } } - else + else { $errors[] = 'News id not set.'; + } if(count($errors)) { return false; From 4c4089a155e3a9c054131afd3da8bfc33a7d0ac7 Mon Sep 17 00:00:00 2001 From: slawkens Date: Tue, 12 Sep 2023 12:11:49 +0200 Subject: [PATCH 05/14] Quotes & const --- admin/pages/news.php | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/admin/pages/news.php b/admin/pages/news.php index 2c5102e6..81153e4f 100644 --- a/admin/pages/news.php +++ b/admin/pages/news.php @@ -23,10 +23,10 @@ if (!hasFlag(FLAG_CONTENT_PAGES) && !superAdmin()) { header('X-XSS-Protection:0'); // some constants, used mainly by database (cannot by modified without schema changes) -define('NEWS_TITLE_LIMIT', 100); -define('NEWS_BODY_LIMIT', 65535); // maximum news body length -define('ARTICLE_TEXT_LIMIT', 300); -define('ARTICLE_IMAGE_LIMIT', 100); +const NEWS_TITLE_LIMIT = 100; +const NEWS_BODY_LIMIT = 65535; // maximum news body length +const ARTICLE_TEXT_LIMIT = 300; +const ARTICLE_IMAGE_LIMIT = 100; $name = $p_title = ''; if(!empty($action)) @@ -52,12 +52,12 @@ if(!empty($action)) $p_title = $body = $comments = $article_text = $article_image = ''; $type = $category = $player_id = 0; - success("Added successful."); + success('Added successful.'); } } else if($action == 'delete') { if (News::delete($id, $errors)) { - success("Deleted successful."); + success('Deleted successful.'); } } else if($action == 'edit') @@ -83,7 +83,7 @@ if(!empty($action)) $action = $p_title = $body = $comments = $article_text = $article_image = ''; $type = $category = $player_id = 0; - success("Updated successful."); + success('Updated successful.'); } } } From f76615e59bceaad13672e3dcf4dd08e682359d14 Mon Sep 17 00:00:00 2001 From: slawkens Date: Sat, 16 Sep 2023 05:54:41 +0200 Subject: [PATCH 06/14] Fix getGuildLogoById --- system/functions.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/system/functions.php b/system/functions.php index 5ea941cf..bb8b018d 100644 --- a/system/functions.php +++ b/system/functions.php @@ -1606,7 +1606,7 @@ function getGuildLogoById($id) $guild = Guild::where('id', intval($id))->select('logo_name')->first(); if ($guild) { - $guildLogo = $query->logo_name; + $guildLogo = $guild->logo_name; if (!empty($guildLogo) && file_exists(GUILD_IMAGES_DIR . $guildLogo)) { $logo = $guildLogo; From 89c2e84bff7c7ffe83446e4f4ed9bf61be4826ca Mon Sep 17 00:00:00 2001 From: slawkens Date: Sat, 16 Sep 2023 07:12:04 +0200 Subject: [PATCH 07/14] Fix alert class name --- system/functions.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/system/functions.php b/system/functions.php index bb8b018d..37f241bc 100644 --- a/system/functions.php +++ b/system/functions.php @@ -1451,7 +1451,7 @@ function echo_success($message) function echo_error($message) { global $error; - echo '
' . $message . '
'; + echo '
' . $message . '
'; $error = true; } From 08339fe8b67e9ac89f26b74df1015eb85133a1ce Mon Sep 17 00:00:00 2001 From: slawkens Date: Sat, 16 Sep 2023 10:19:22 +0200 Subject: [PATCH 08/14] Fix XSS in tibiacom template - subtopic --- templates/tibiacom/index.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/templates/tibiacom/index.php b/templates/tibiacom/index.php index 95c73c33..06c58e50 100644 --- a/templates/tibiacom/index.php +++ b/templates/tibiacom/index.php @@ -28,7 +28,7 @@ if(isset($config['boxes'])) Date: Sat, 16 Sep 2023 10:19:54 +0200 Subject: [PATCH 09/14] Fix XSS in accounts editor --- admin/pages/accounts.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/admin/pages/accounts.php b/admin/pages/accounts.php index f3022e9c..95490a1b 100644 --- a/admin/pages/accounts.php +++ b/admin/pages/accounts.php @@ -583,7 +583,7 @@ else if (isset($_REQUEST['search'])) {
- +
From e9f155fb49aa99838a710a0568923e87bc0718f3 Mon Sep 17 00:00:00 2001 From: slawkens Date: Sat, 16 Sep 2023 10:20:34 +0200 Subject: [PATCH 10/14] Fix XSS in players editor --- admin/pages/players.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/admin/pages/players.php b/admin/pages/players.php index b409c15e..153601b7 100644 --- a/admin/pages/players.php +++ b/admin/pages/players.php @@ -872,7 +872,7 @@ else if (isset($_REQUEST['search'])) {
- +
From 0ac0f4e7a8211937a231fe8f09b3fe07e95ece39 Mon Sep 17 00:00:00 2001 From: slawkens Date: Sat, 16 Sep 2023 10:22:10 +0200 Subject: [PATCH 11/14] Fixes --- admin/pages/accounts.php | 22 +++++++++--------- admin/pages/players.php | 48 ++++++++++++++++++++-------------------- 2 files changed, 35 insertions(+), 35 deletions(-) diff --git a/admin/pages/accounts.php b/admin/pages/accounts.php index 95490a1b..214b82f0 100644 --- a/admin/pages/accounts.php +++ b/admin/pages/accounts.php @@ -327,8 +327,8 @@ else if (isset($_REQUEST['search'])) {
@@ -338,8 +338,8 @@ else if (isset($_REQUEST['search'])) {
@@ -347,8 +347,8 @@ else if (isset($_REQUEST['search'])) {
@@ -403,8 +403,8 @@ else if (isset($_REQUEST['search'])) {
@@ -581,7 +581,7 @@ else if (isset($_REQUEST['search'])) {
- +
@@ -590,9 +590,9 @@ else if (isset($_REQUEST['search'])) {
- +
- +
diff --git a/admin/pages/players.php b/admin/pages/players.php index 153601b7..eaa06494 100644 --- a/admin/pages/players.php +++ b/admin/pages/players.php @@ -390,8 +390,8 @@ else if (isset($_REQUEST['search'])) {
@@ -399,8 +399,8 @@ else if (isset($_REQUEST['search'])) { @@ -410,8 +410,8 @@ else if (isset($_REQUEST['search'])) {
@@ -424,8 +424,8 @@ else if (isset($_REQUEST['search'])) { $configTowns[$player->getTownId()] = 'Unknown Town'; } - foreach ($configTowns as $id => $town): ?> - + foreach ($configTowns as $_id => $town): ?> +
@@ -436,8 +436,8 @@ else if (isset($_REQUEST['search'])) { @@ -554,16 +554,16 @@ else if (isset($_REQUEST['search'])) {
$info) { + foreach ($skills as $_id => $info) { ?>
- ' . $info[0] . ' - '; ?> + ' . $info[0] . ' + '; ?>
- ' . $info[0] . ' tries - '; ?> + ' . $info[0] . ' tries + '; ?>
@@ -622,7 +622,7 @@ else if (isset($_REQUEST['search'])) { if ($outfitlist) { ?> $s_name) { - echo ''; + foreach ($addon_type as $_id => $s_name) { + echo ''; } ?> @@ -704,7 +704,7 @@ else if (isset($_REQUEST['search'])) {
- + [max. length: 2000 chars, 50 lines (ENTERs)]
@@ -841,7 +841,7 @@ else if (isset($_REQUEST['search'])) { const $addonvalue = $('#look_addons'); - $('#look_addons').on('change', () => { + $addonvalue.on('change', () => { updateOutfit(); }); @@ -870,7 +870,7 @@ else if (isset($_REQUEST['search'])) {
- +
@@ -879,9 +879,9 @@ else if (isset($_REQUEST['search'])) {
- +
- +
@@ -892,7 +892,7 @@ else if (isset($_REQUEST['search'])) {