From e5c5a1bf424754a417abdb25656c99d345dac2f9 Mon Sep 17 00:00:00 2001 From: wn_ Date: Fri, 29 Aug 2025 12:19:04 +0000 Subject: Make 'owner_uid' required for 'Feeds::_get_title()' and 'Feeds::_get_cat_title()'. --- classes/API.php | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'classes/API.php') diff --git a/classes/API.php b/classes/API.php index 03f5a2502..6255bf81f 100644 --- a/classes/API.php +++ b/classes/API.php @@ -175,7 +175,7 @@ class API extends Handler { if ($unread || !$unread_only) { array_push($cats, [ 'id' => $cat_id, - 'title' => Feeds::_get_cat_title($cat_id), + 'title' => Feeds::_get_cat_title($cat_id, $_SESSION['uid']), 'unread' => (int) $unread, ]); } @@ -574,7 +574,7 @@ class API extends Handler { $unread = Feeds::_get_counters($i, false, true); if ($unread || !$unread_only) { - $title = Feeds::_get_title($i); + $title = Feeds::_get_title($i, false, $_SESSION['uid']); $row = [ 'id' => $i, -- cgit v1.2.3-54-g00ecf From ecef0ae9512a3227221e107fe47dab91da6e5f4c Mon Sep 17 00:00:00 2001 From: wn_ Date: Fri, 29 Aug 2025 12:27:44 +0000 Subject: Tweak the 'Feeds::_get_title()' param order to make PHP happy. Required params need to go before optional. --- classes/API.php | 2 +- classes/Feeds.php | 6 +++--- classes/OPML.php | 6 +++--- classes/Pref_Feeds.php | 4 ++-- classes/Pref_Filters.php | 6 +++--- plugins/af_psql_trgm/init.php | 2 +- 6 files changed, 13 insertions(+), 13 deletions(-) (limited to 'classes/API.php') diff --git a/classes/API.php b/classes/API.php index 6255bf81f..4b49e4b37 100644 --- a/classes/API.php +++ b/classes/API.php @@ -574,7 +574,7 @@ class API extends Handler { $unread = Feeds::_get_counters($i, false, true); if ($unread || !$unread_only) { - $title = Feeds::_get_title($i, false, $_SESSION['uid']); + $title = Feeds::_get_title($i, $_SESSION['uid']); $row = [ 'id' => $i, diff --git a/classes/Feeds.php b/classes/Feeds.php index b32e0bb2a..4f124f42c 100644 --- a/classes/Feeds.php +++ b/classes/Feeds.php @@ -677,7 +677,7 @@ class Feeds extends Handler_Protected {
-

Feed Debugger: _get_title($feed_id, false, $_SESSION['uid']) ?>

+

Feed Debugger: _get_title($feed_id, $_SESSION['uid']) ?>

@@ -1178,7 +1178,7 @@ class Feeds extends Handler_Protected { } } - static function _get_title(int|string $id, bool $cat = false, int $owner_uid): string { + static function _get_title(int|string $id, int $owner_uid, bool $cat = false): string { if ($cat) { return self::_get_cat_title($id, $owner_uid); } else if ($id == Feeds::FEED_STARRED) { @@ -1638,7 +1638,7 @@ class Feeds extends Handler_Protected { $last_error = $row["last_error"]; $last_updated = $row["last_updated"]; } else { - $feed_title = self::_get_title($feed, false, $owner_uid); + $feed_title = self::_get_title($feed, $owner_uid); } } } diff --git a/classes/OPML.php b/classes/OPML.php index 42f31749b..af17a7124 100644 --- a/classes/OPML.php +++ b/classes/OPML.php @@ -182,8 +182,8 @@ class OPML extends Handler_Protected { if ($cat_filter && $tmp_line["cat_id"] || $tmp_line["feed_id"]) { $tmp_line["feed"] = Feeds::_get_title( $cat_filter ? $tmp_line["cat_id"] : $tmp_line["feed_id"], - $cat_filter, - $_SESSION['uid']); + $_SESSION['uid'], + $cat_filter); } else { $tmp_line["feed"] = ""; } @@ -200,7 +200,7 @@ class OPML extends Handler_Protected { } } else { if ($feed_id) { - array_push($match, [Feeds::_get_title((int)$feed_id, false, $owner_uid), false, false]); + array_push($match, [Feeds::_get_title((int)$feed_id, $owner_uid), false, false]); } else { array_push($match, [0, false, true]); } diff --git a/classes/Pref_Feeds.php b/classes/Pref_Feeds.php index 2e3821c45..e27a9ccbd 100644 --- a/classes/Pref_Feeds.php +++ b/classes/Pref_Feeds.php @@ -1115,7 +1115,7 @@ class Pref_Feeds extends Handler_Protected { */ private function feedlist_init_feed(int $feed_id, ?string $title = null, bool $unread = false, string $error = '', string $updated = ''): array { if (!$title) - $title = Feeds::_get_title($feed_id, false, $_SESSION['uid']); + $title = Feeds::_get_title($feed_id, $_SESSION['uid']); if ($unread === false) $unread = Feeds::_get_counters($feed_id, false, true); @@ -1284,7 +1284,7 @@ class Pref_Feeds extends Handler_Protected { ]); print json_encode([ - "title" => Feeds::_get_title($feed_id, $is_cat, $_SESSION['uid']), + "title" => Feeds::_get_title($feed_id, $_SESSION['uid'], $is_cat), "link" => $link ]); } diff --git a/classes/Pref_Filters.php b/classes/Pref_Filters.php index f2ae50c7b..645027f5f 100644 --- a/classes/Pref_Filters.php +++ b/classes/Pref_Filters.php @@ -264,7 +264,7 @@ class Pref_Filters extends Handler_Protected { array_push($feeds_fmt, Feeds::_get_cat_title($feed_id, $_SESSION['uid'])); } else { if ($feed_id) - array_push($feeds_fmt, Feeds::_get_title((int)$feed_id, false, $_SESSION['uid'])); + array_push($feeds_fmt, Feeds::_get_title((int)$feed_id, $_SESSION['uid'])); else array_push($feeds_fmt, __("All feeds")); } @@ -276,7 +276,7 @@ class Pref_Filters extends Handler_Protected { $where = $rule->cat_filter ? Feeds::_get_cat_title($rule->cat_id ?? 0, $_SESSION['uid']) : ($rule->feed_id ? - Feeds::_get_title($rule->feed_id, false, $_SESSION['uid']) : __("All feeds")); + Feeds::_get_title($rule->feed_id, $_SESSION['uid']) : __("All feeds")); } $inverse_class = $rule->inverse ? "inverse" : ""; @@ -463,7 +463,7 @@ class Pref_Filters extends Handler_Protected { array_push($feeds_fmt, Feeds::_get_cat_title($feed_id, $_SESSION['uid'])); } else { if ($feed_id) - array_push($feeds_fmt, Feeds::_get_title((int)$feed_id, false, $_SESSION['uid'])); + array_push($feeds_fmt, Feeds::_get_title((int)$feed_id, $_SESSION['uid'])); else array_push($feeds_fmt, __("All feeds")); } diff --git a/plugins/af_psql_trgm/init.php b/plugins/af_psql_trgm/init.php index 87c4d16e8..591cd7ea1 100644 --- a/plugins/af_psql_trgm/init.php +++ b/plugins/af_psql_trgm/init.php @@ -210,7 +210,7 @@ class Af_Psql_Trgm extends Plugin {
  • rss_feed - +
    • -- cgit v1.2.3-54-g00ecf