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

Failure in makefulltextfeed.php #529

Closed
ghost opened this issue Mar 6, 2014 · 4 comments
Closed

Failure in makefulltextfeed.php #529

ghost opened this issue Mar 6, 2014 · 4 comments
Labels
Milestone

Comments

@ghost
Copy link

ghost commented Mar 6, 2014

Hello,
I am using wallabag 1.5.2.

I want to save the following link: http://www.heise.de/newsticker/meldung/Arbeitgeber-darf-Urlaub-in-der-Elternzeit-kuerzen-2120838.html

And if I do that with the bookmarklet or the FF extension, wallabag throws the following error:

Warning: Creating default object from empty value in /var/www/virtual/username/wallabag.domain.com/inc/3rdparty/makefulltextfeed.php on line 1108

Fatal error: Call to undefined method stdClass::get() in /var/www/virtual/username/wallabag.domain.com/inc/3rdparty/makefulltextfeed.php on line 1109

After the error I am logged out from wallabag. It does work with other links, but this is a example, where it does not work.

@mariroz
Copy link
Contributor

mariroz commented Mar 6, 2014

@jlnostr, thank you for reporting, this error is related to content with multiple pages (wallabag will save all pages of your article).
It is already fixed in dev branch (you can try it) and will be fixed in next wallabag release.

@nicosomb nicosomb added the Bug label Mar 7, 2014
@nicosomb nicosomb added this to the 1.5.3 milestone Mar 7, 2014
@xldrkp
Copy link

xldrkp commented Mar 11, 2014

Thanks, @jlnostr, have the same problem. Looking forward to the fix. Thanks for the software!
XL

@nicosomb nicosomb modified the milestones: 1.6.0, 1.5.3 Mar 12, 2014
@nicosomb
Copy link
Member

@jlnostr @xldrkp can you try dev branch please? I want to know if I can close this issue.

@ghost
Copy link
Author

ghost commented Mar 14, 2014

Yes, you can close it. In the development tree it worked perfectly :-)

@ghost ghost closed this as completed Mar 14, 2014
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants