From 23ba48c71f0d41bbe012d668349f6516dad527b4 Mon Sep 17 00:00:00 2001 From: Alexis Degrugillier Date: Fri, 5 Sep 2025 09:56:46 -0400 Subject: Change how files are included (#7916) 1. `include`, `include_once`, `require` and `require_once` are expressions not functions, parentheses are not necessary. 2. to move up the directory tree, it's better to use the `dirname` function instead of relying on `/..`. --- app/Controllers/updateController.php | 6 +++--- app/Controllers/userController.php | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) (limited to 'app/Controllers') diff --git a/app/Controllers/updateController.php b/app/Controllers/updateController.php index d97b18c70..ba343c81e 100644 --- a/app/Controllers/updateController.php +++ b/app/Controllers/updateController.php @@ -128,7 +128,7 @@ class FreshRSS_update_Controller extends FreshRSS_ActionController { Minz_Error::error(403); } - include_once(LIB_PATH . '/lib_install.php'); + include_once LIB_PATH . '/lib_install.php'; invalidateHttpCache(); @@ -277,7 +277,7 @@ class FreshRSS_update_Controller extends FreshRSS_ActionController { if (self::isGit()) { $res = !self::hasGitUpdate(); } else { - require(UPDATE_FILENAME); + require UPDATE_FILENAME; // @phpstan-ignore function.notFound $res = do_post_update(); } @@ -299,7 +299,7 @@ class FreshRSS_update_Controller extends FreshRSS_ActionController { if (self::isGit()) { $res = self::gitPull(); } else { - require(UPDATE_FILENAME); + require UPDATE_FILENAME; if (Minz_Request::isPost()) { // @phpstan-ignore function.notFound save_info_update(); diff --git a/app/Controllers/userController.php b/app/Controllers/userController.php index 67a97c45a..17879f3d0 100644 --- a/app/Controllers/userController.php +++ b/app/Controllers/userController.php @@ -288,7 +288,7 @@ class FreshRSS_user_Controller extends FreshRSS_ActionController { $customUserConfigPath = join_path(DATA_PATH, 'config-user.custom.php'); if (file_exists($customUserConfigPath)) { - $customUserConfig = include($customUserConfigPath); + $customUserConfig = include $customUserConfigPath; if (is_array($customUserConfig)) { $userConfig = $customUserConfig; } -- cgit v1.2.3