aboutsummaryrefslogtreecommitdiff
path: root/app/Controllers/feedController.php
diff options
context:
space:
mode:
authorGravatar Alexandre Alapetite <alexandre@alapetite.fr> 2021-12-02 23:25:07 +0100
committerGravatar GitHub <noreply@github.com> 2021-12-02 23:25:07 +0100
commita2ab9cf83aaead96497a70a1430ce0424c0d8316 (patch)
tree015985aac14f6bd6800ec8cbeb5166e5f9479ea5 /app/Controllers/feedController.php
parent4e00c2d3373af223380b9f8f168007bb8e42b6b9 (diff)
Minz request avoid custom methods (#4020)
Take advantage of PHP7+ null-coalescing operator `??` to make code more standard, shorter, and faster instead of custom function with no extra functionality. Allows code to be better tested and fix two PHPstan errors: ``` ------ ----------------------------------------- Line app/Controllers/configureController.php ------ ----------------------------------------- 410 Cannot unset offset 'rid' on string. ------ ----------------------------------------- ------ ------------------------------------ Line lib/Minz/FrontController.php ------ ------------------------------------ 70 Cannot unset offset 'c' on string. 71 Cannot unset offset 'a' on string. ------ ------------------------------------ ``` https://github.com/FreshRSS/FreshRSS/issues/4016
Diffstat (limited to 'app/Controllers/feedController.php')
-rwxr-xr-xapp/Controllers/feedController.php2
1 files changed, 1 insertions, 1 deletions
diff --git a/app/Controllers/feedController.php b/app/Controllers/feedController.php
index afb05f17e..c94b3216a 100755
--- a/app/Controllers/feedController.php
+++ b/app/Controllers/feedController.php
@@ -566,7 +566,7 @@ class FreshRSS_feed_Controller extends Minz_ActionController {
$url = Minz_Request::param('url');
$force = Minz_Request::param('force');
$maxFeeds = (int)Minz_Request::param('maxFeeds');
- $noCommit = Minz_Request::fetchPOST('noCommit', 0) == 1;
+ $noCommit = ($_POST['noCommit'] ?? 0) == 1;
if ($id == -1 && !$noCommit) { //Special request only to commit & refresh DB cache
$updated_feeds = 0;