From d26a6f40d07626db1fe6d26aa2adeb53bbd88579 Mon Sep 17 00:00:00 2001 From: Sunil Mohan Adapa Date: Sun, 16 Apr 2023 13:35:42 +0530 Subject: [PATCH] searx: Simplify handling shortcut for front page The older code is likely before the card approach was implemented and cached copy had to updated. It is no longer needed as the shortcuts are not filtered based on login required property in the template code. Tests: - When public visibility is enabled/disabled shortcut is properly shown on the frontpage. - When restarting the service the public visibility properly dictates whether shortcut is show properly. Reviewed-by: James Valleroy --- plinth/modules/searx/__init__.py | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/plinth/modules/searx/__init__.py b/plinth/modules/searx/__init__.py index f367629f8..d0a127b81 100644 --- a/plinth/modules/searx/__init__.py +++ b/plinth/modules/searx/__init__.py @@ -87,9 +87,7 @@ class SearxApp(app_module.App): def set_shortcut_login_required(self, login_required): """Change the login_required property of shortcut.""" - shortcut = self.remove('shortcut-searx') - shortcut.login_required = login_required - self.add(shortcut) + self.get_component('shortcut-searx').login_required = login_required def setup(self, old_version): """Install and configure the app."""