diff options
| author |  Bryan Butler <3256910+BryanButlerGit@users.noreply.github.com>
| 2025-03-07 15:29:27 -0700 |
|---|---|---|
| committer |  GitHub <noreply@github.com>
| 2025-03-07 23:29:27 +0100 |
| commit | 3ec0bdab3840feb7140d70d0eb8d60065337e33a (patch) | |
| tree | 8367f259444f717332cc7ebf4171b3c43af51620 | |
| parent | b9a6c943b242b6ac60eaa85b065858b5f3f9b99b (diff) | |
Fix issue where $keepMax is defaulted to const when non-zero value is compared. (#7416)
Fix issue where $keepMax is defaulted to const when non-zero value is compared.
Fix an issue where $keepMax var is set to ARCHIVING_RETENTION_COUNT_LIMIT
when a user sets "Archiving" -> "Maximum number of articles to keep per
feed" to a non-zero value for a given category.
[EXAMPLE]:
User sets value to 50 (int), This matches the !== comparison and
thus is overridden by ARCHIVING_RETENTION_COUNT_LIMIT.
| -rw-r--r-- | app/Controllers/categoryController.php | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/app/Controllers/categoryController.php b/app/Controllers/categoryController.php index 9fd5fd630..956a462e1 100644 --- a/app/Controllers/categoryController.php +++ b/app/Controllers/categoryController.php @@ -113,7 +113,7 @@ class FreshRSS_category_Controller extends FreshRSS_ActionController { } else { if (!Minz_Request::paramBoolean('enable_keep_max')) { $keepMax = false; - } elseif (($keepMax = Minz_Request::paramInt('keep_max')) !== 0) { + } elseif (($keepMax = Minz_Request::paramInt('keep_max')) === 0) { $keepMax = FreshRSS_Feed::ARCHIVING_RETENTION_COUNT_LIMIT; } if (Minz_Request::paramBoolean('enable_keep_period')) { |