search

DigitalDJ / tinytinyrss-fever-plugin

Tiny Tiny RSS Fever API Plugin
https://github.com/DigitalDJ/tinytinyrss-fever-plugin
GNU General Public License v3.0
207 stars 28 forks source link

fix to work with newest ttrss #18

Closed froks closed 3 years ago

froks commented 3 years ago

Fixed fever-plugin, so it should work with the newest ttrss changes, I have almost 0 experience with php, so this might not be a "good" fix, or even complete, but it should serve as a good starting point. The _get_enclosures(..) function from Article sounds like it might be private/removed in the future. param_to_bool was just copy&pasted/renamed from ttrss. Paths to db.php and API.php were changed to reflect the new path in ttrss.

eric-pierce commented 3 years ago

Confirming that this pull request does fix the syncing issue - the API still throws several undefined array warnings but those are likely due to the move of TT-RSS to PHP8. Here are some examples of those:

Undefined array key "sid" 1. plugins.local/fever/index.php(56): ttrss_error_handler(2, Undefined array key "sid", plugins.local/fever/index.php, 56)

Undefined array key "order_id" 1. plugins.local/fever/fever_api.php(298): ttrss_error_handler(2, Undefined array key "order_id", plugins.local/fever/fever_api.php, 298) 2. plugins.local/fever/fever_api.php(884): getGroups() 3. plugins.local/fever/index.php(71)

DigitalDJ commented 3 years ago

Thanks. I'm no longer maintaining this project due to Reeder deprecating Fever. Happy to merge pull requests.