From fe742de211a35cd61f76dda477209b02fb8c25e3 Mon Sep 17 00:00:00 2001
From: Roman Kelesidis <roman25052006.kelesh@gmail.com>
Date: Tue, 22 Aug 2023 00:32:06 +0700
Subject: [PATCH] Show set/unset downloaded actions in log actions (#858)

---
 admin/admin_log.php              |  2 ++
 library/language/source/main.php |  2 ++
 modcp.php                        |  9 +++++++++
 src/Legacy/LogAction.php         | 16 +++++++++-------
 4 files changed, 22 insertions(+), 7 deletions(-)

diff --git a/admin/admin_log.php b/admin/admin_log.php
index a23ce806f..8b631c509 100644
--- a/admin/admin_log.php
+++ b/admin/admin_log.php
@@ -223,6 +223,8 @@ if ($log_rowset) {
             case $log_type['mod_topic_move']:
             case $log_type['mod_topic_lock']:
             case $log_type['mod_topic_unlock']:
+            case $log_type['mod_topic_set_downloaded']:
+            case $log_type['mod_topic_unset_downloaded']:
             case $log_type['mod_post_delete']:
             case $log_type['mod_post_pin']:
             case $log_type['mod_post_unpin']:
diff --git a/library/language/source/main.php b/library/language/source/main.php
index 9ae85d699..500a20eed 100644
--- a/library/language/source/main.php
+++ b/library/language/source/main.php
@@ -2550,6 +2550,8 @@ $lang['LOG_ACTION']['LOG_TYPE'] = array(
     'mod_topic_lock' => 'Topic:<br /> <b>closed</b>',
     'mod_topic_unlock' => 'Topic:<br /> <b>opened</b>',
     'mod_topic_split' => 'Topic:<br /> <b>split</b>',
+    'mod_topic_set_downloaded' => 'Topic:<br /> <b>set downloaded</b>',
+    'mod_topic_unset_downloaded' => 'Topic:<br /> <b>unset downloaded</b>',
     'mod_post_delete' => 'Post:<br /> <b>deleted</b>',
     'mod_post_pin' => 'Post:<br /> <b>pinned</b>',
     'mod_post_unpin' => 'Post:<br /> <b>unpinned</b>',
diff --git a/modcp.php b/modcp.php
index 0ad6f6167..754c0dafd 100644
--- a/modcp.php
+++ b/modcp.php
@@ -361,6 +361,15 @@ switch ($mode) {
             clear_dl_list($topic_csv);
         }
 
+        // Log action
+        $type = ($set_download) ? 'mod_topic_set_downloaded' : 'mod_topic_unset_downloaded';
+
+        $log_action->mod($type, array(
+            'forum_id' => $forum_id,
+            'topic_id' => $topic_id,
+            'topic_title' => get_topic_title($topic_id),
+        ));
+
         $msg = ($set_download) ? $lang['TOPICS_DOWN_SETS'] : $lang['TOPICS_DOWN_UNSETS'];
         bb_die(return_msg_mcp($msg));
 
diff --git a/src/Legacy/LogAction.php b/src/Legacy/LogAction.php
index 732c942a5..a9b1df22e 100644
--- a/src/Legacy/LogAction.php
+++ b/src/Legacy/LogAction.php
@@ -20,13 +20,15 @@ class LogAction
         'mod_topic_move' => 2,
         'mod_topic_lock' => 3,
         'mod_topic_unlock' => 4,
-        'mod_post_delete' => 5,
-        'mod_post_pin' => 6,
-        'mod_post_unpin' => 7,
-        'mod_topic_split' => 8,
-        'adm_user_delete' => 9,
-        'adm_user_ban' => 10,
-        'adm_user_unban' => 11,
+        'mod_topic_set_downloaded' => 5,
+        'mod_topic_unset_downloaded' => 6,
+        'mod_post_delete' => 7,
+        'mod_post_pin' => 8,
+        'mod_post_unpin' => 9,
+        'mod_topic_split' => 10,
+        'adm_user_delete' => 11,
+        'adm_user_ban' => 12,
+        'adm_user_unban' => 13,
     ];
     public $log_type_select = [];
     public $log_disabled = false;