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

actualize_script.php failed since 1.11.0 #1917

Closed
prysme01 opened this issue Jun 5, 2018 · 9 comments
Closed

actualize_script.php failed since 1.11.0 #1917

prysme01 opened this issue Jun 5, 2018 · 9 comments
Assignees
Milestone

Comments

@prysme01
Copy link

prysme01 commented Jun 5, 2018

Since update to 1.11.0 my cron job to actualize feeds is not working anymore
*/30 * * * * www-data php -f /var/www/freshrss/app/actualize_script.php >/tmp/freshrss-cron.log 2>&1

PHP Fatal error: Uncaught Error: Call to a member function attributes() on string in /var/www/freshrss/app/Models/Entry.php:197
Stack trace:
#0 /var/www/freshrss/app/Models/Feed.php(395): FreshRSS_Entry->loadCompleteContent('.entry-content')
#1 /var/www/freshrss/app/Models/Feed.php(324): FreshRSS_Feed->loadEntries(Object(SimplePie))
#2 /var/www/freshrss/app/Controllers/feedController.php(311): FreshRSS_Feed->load(false, false)
#3 /var/www/freshrss/app/Controllers/feedController.php(509): FreshRSS_feed_Controller::actualizeFeed(false, false, '1', NULL, false, false)
#4 /var/www/freshrss/lib/Minz/Dispatcher.php(118): FreshRSS_feed_Controller->actualizeAction()
#5 /var/www/freshrss/lib/Minz/Dispatcher.php(47): Minz_Dispatcher->launchAction('actualizeAction')
#6 /var/www/freshrss/lib/Minz/FrontController.php(84): Minz_Dispatcher->run()
#7 /var/www/freshrss/app/actualize_script.php(61): Minz_FrontController->run()
#8 {main}
thrown in /var/www/freshrss/app/Models/Entry.php on line 197
Results:

@Alkarex Alkarex added this to the 1.12.0 milestone Jun 5, 2018
@Alkarex Alkarex self-assigned this Jun 5, 2018
@Alkarex
Copy link
Member

Alkarex commented Jun 5, 2018

Fix coming

@prysme01
Copy link
Author

prysme01 commented Jun 5, 2018

Great thank you. Manual update is still working, so that's fine :)

Alkarex added a commit to Alkarex/FreshRSS that referenced this issue Jun 5, 2018
@Alkarex
Copy link
Member

Alkarex commented Jun 5, 2018

Could you please test this patch? #1918
If it helps, I can merge it in /dev.

Alkarex added a commit to Alkarex/FreshRSS that referenced this issue Jun 5, 2018
@prysme01
Copy link
Author

prysme01 commented Jun 5, 2018

I'm sorry but I don't know how to apply the patch as I am not using the github version of the code.

Alkarex added a commit that referenced this issue Jun 5, 2018
* Fix type exception in loadCompleteContent

#1917
With some refactoring

* Changelog 1917

#1917
#1918
@Alkarex Alkarex modified the milestones: 1.12.0, 1.11.1 Jun 5, 2018
@prysme01
Copy link
Author

prysme01 commented Jun 5, 2018

ok done, sounds good, no more error.

@Alkarex
Copy link
Member

Alkarex commented Jun 5, 2018

Excellent. Please confirm when you have seen that you properly get new update for both normal feeds and feeds fetching full content.

@Alkarex Alkarex closed this as completed Jun 5, 2018
@prysme01
Copy link
Author

prysme01 commented Jun 5, 2018

I confirm for most of my feed with css selectors. I will have to wait for all of them.
I would be nice to have a test that could validate a css selector when configuring the feed :)

@Alkarex
Copy link
Member

Alkarex commented Jun 5, 2018

Thanks for the feedback 👍

javerous pushed a commit to javerous/FreshRSS that referenced this issue Jan 20, 2020
* Fix type exception in loadCompleteContent

FreshRSS#1917
With some refactoring

* Changelog 1917

FreshRSS#1917
FreshRSS#1918
mdemoss pushed a commit to mdemoss/FreshRSS that referenced this issue Mar 25, 2021
* Fix type exception in loadCompleteContent

FreshRSS#1917
With some refactoring

* Changelog 1917

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

No branches or pull requests

2 participants