From d8610407663539b1d52d17bc0b8dc88f739db063 Mon Sep 17 00:00:00 2001
From: Kijin Sung <kijin@kijinsung.com>
Date: Wed, 17 Apr 2024 23:28:34 +0900
Subject: [PATCH] Remove unnecessary procAdminRemoveIcons action

---
 modules/admin/admin.admin.controller.php | 19 -------------------
 modules/admin/conf/module.xml            |  1 -
 modules/admin/tpl/js/config.js           | 21 +--------------------
 3 files changed, 1 insertion(+), 40 deletions(-)

diff --git a/modules/admin/admin.admin.controller.php b/modules/admin/admin.admin.controller.php
index 30f5b3978..d210b628f 100644
--- a/modules/admin/admin.admin.controller.php
+++ b/modules/admin/admin.admin.controller.php
@@ -18,25 +18,6 @@ class AdminAdminController extends Admin
 		}
 	}
 
-	/**
-	 * Remove an icon.
-	 *
-	 * @deprecated
-	 */
-	public function procAdminRemoveIcons()
-	{
-		$site_info = Context::get('site_module_info');
-		$icon_name = Rhymix\Framework\Filters\FilenameFilter::clean(Context::get('iconname'));
-		if (Rhymix\Modules\Admin\Models\Icon::deleteIcon($site_info->domain_srl, $icon_name))
-		{
-			$this->setMessage('success_deleted');
-		}
-		else
-		{
-			throw new Exception('fail_to_delete');
-		}
-	}
-
 	/**
 	 * Delete the admin logo.
 	 *
diff --git a/modules/admin/conf/module.xml b/modules/admin/conf/module.xml
index 6d2f68023..6d1481ca8 100644
--- a/modules/admin/conf/module.xml
+++ b/modules/admin/conf/module.xml
@@ -47,7 +47,6 @@
 		<action name="procAdminUpdateFTPInfo" type="controller" />
 		<action name="procAdminRemoveFTPInfo" type="controller" />
 		<action name="procAdminUpdateConfig" type="controller" />
-		<action name="procAdminRemoveIcons" type="controller" />
 		<action name="procAdminDeleteLogo" type="controller" />
 		<action name="getSiteAllList" type="model" permission="root" />
 	</actions>
diff --git a/modules/admin/tpl/js/config.js b/modules/admin/tpl/js/config.js
index f0d72103f..5a044bbfc 100644
--- a/modules/admin/tpl/js/config.js
+++ b/modules/admin/tpl/js/config.js
@@ -30,7 +30,7 @@ jQuery(function($){
 			}
 		}).triggerHandler("change");
 	}
-	
+
 	// Disable rewrite level 2 if test AJAX request fails.
 	if ($('#use_rewrite_2').size() && !$('#use_rewrite_2').is(':checked')) {
 		var testval = 1000 + Math.floor(Math.random() * 9000);
@@ -62,25 +62,6 @@ function viewSiteSearch(){
 	jQuery(".site_keyword_search").css("display","");
 }
 
-var icon = null;
-function deleteIcon(iconname){
-	var params = [];
-	params.iconname = iconname;
-	exec_xml('admin', 'procAdminRemoveIcons', params, iconDeleteMessage, ['error', 'message'], params);
-	icon = iconname;
-}
-function iconDeleteMessage(ret_obj){
-	alert(ret_obj.message);
-
-	if (ret_obj.error == '0')
-	{
-		if (icon == 'favicon.ico'){
-			jQuery('.faviconPreview img').attr('src', 'modules/admin/tpl/img/faviconSample.png');
-		}else if (icon == 'mobicon.png'){
-			jQuery('.mobiconPreview img').attr('src', 'modules/admin/tpl/img/mobiconSample.png');
-		}
-	}
-}
 function doRecompileCacheFile() {
 	if (!confirm(xe.lang.confirm_run)) return;
 	exec_json('admin.procAdminRecompileCacheFile', {}, function(data) {