Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feeds no longer sync to Reeder for iOS after updating TT-RSS and Fever #31

Open
eli-oat opened this issue Jun 4, 2017 · 7 comments
Open

Comments

@eli-oat
Copy link

eli-oat commented Jun 4, 2017

I recently updated to the most recent v17.4 of TT-RSS, and the most recent version of tinytinyrss-fever-plugin. Before updating everything worked 💯, and I was able to sync to Reeder 3 on iOS. Since updating, however, I'm no longer able to do so.

Everything is configured the same as before, and I've tried updating my fever password, but no dice.

  • API access is enabled
  • Fever password is enabled
  • When I visit .../tt-rss/plugins/fever/ I see {"api_version":3,"auth":0}

I'm not seeing any errors in Reeder, either, it just said "nothing synced yet," or something along those lines.

@marcelinollano
Copy link

Same here, not working anymore with v17.4

@cdoenges
Copy link

cdoenges commented Jun 5, 2017

The Apache error.log shows:

PHP Fatal error: Call to undefined function get_article_enclosures() in /var/www/html/tt-rss/plugins/fever/fever_api.php on line 561

Looks like get_article_enclosures() was renamed or removed in TT-RSS v17.4

@cdoenges
Copy link

cdoenges commented Jun 5, 2017

Change line 561 of fever_api.php from

enclosures = get_article_enclosures($line["id"]);

to

$enclosures = Article::get_article_enclosures($line["id"]);

to fix that issue.

It appears there are some more ...

PHP Fatal error:  Call to undefined function ccache_update() in /var/www/html/tt-rss/plugins/fever/fever_api.php on line 716

cdoenges pushed a commit to cdoenges/tinytinyrss-fever-plugin that referenced this issue Jun 5, 2017
…S v17.4.

Rename get_article_enclosures() to Article::get_article_enclosures()
and ccache_update() to CCache::update().
@cdoenges
Copy link

cdoenges commented Jun 5, 2017

I have created a pull request for the fix.

It appears that there are already two pending pull requests for the same issue. See ticket #25 and #30.

@eli-oat
Copy link
Author

eli-oat commented Jun 6, 2017

I applied @cdoenges's code locally and it works perfectly. 👍

@MrMuetze
Copy link

worked well for me too, thanks for the solution!

@oelna
Copy link

oelna commented Dec 17, 2017

I combined this and the changes from #33 and it works.
I also used Shaun Inman's Fever HTML widget to check for an auth value of 1 in the API calls. Maybe this is of help to somebody. I had never set up tt-rss or the Fever plugin before, and the Reeder app on iOS wouldn't sign in, so this was somewhat helpful as a tool for debugging and figuring out how to get going.

Now if only somebody made a Fever theme …

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants