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

Error parsing file imported from Pocket #718

Closed
tcitworld opened this issue Jun 4, 2014 · 8 comments
Closed

Error parsing file imported from Pocket #718

tcitworld opened this issue Jun 4, 2014 · 8 comments
Labels
Milestone

Comments

@tcitworld
Copy link
Member

I have Version 1.7.0.
File exported from Pocket, nothing big : https://gist.github.com/tcitworld/e277c35311be80cf0ba9

Error : Fatal error: Call to a member function find() on a non-object in C:\wamp\www\wallabag-1.7.0\inc\3rdparty\simple_html_dom.php on line 1114
Coming from : Poche.class.php:923
For some reason, $this->root is not defined in simple_html_dom.php. Any idea, @mariroz ?

@tcitworld tcitworld added the Bug label Jun 4, 2014
@mariroz
Copy link
Contributor

mariroz commented Jun 5, 2014

@tcitworld , will investigate

@tcitworld
Copy link
Member Author

There is also another file at #723.

@rubykon
Copy link

rubykon commented Jun 18, 2014

Having the same problem, independent from the php version I use. :) 👍 and independent from the branch I use.

@ebrodeur
Copy link

I attempted to import my Pocket file and received the same error:

Fatal error: Call to a member function find() on a non-object in /home/adm2398/public_html/wallabag/inc/3rdparty/simple_html_dom.php on line 1114

Fresh install of v1.70.

@mariroz
Copy link
Contributor

mariroz commented Jun 23, 2014

have an idea, tomorrow will have some time, hope - will fix this issue.

@tcitworld
Copy link
Member Author

Worked perfectly for me. Thanks a lot !

@rubykon, @ebrodeur, I merge in dev branch, can you give it a try too ? :)

tcitworld added a commit that referenced this issue Jun 25, 2014
fix of issue #718: Error parsing file imported from Pocket #718
@ebrodeur
Copy link

Import worked. It's updating the articles. Thanks!

@tcitworld
Copy link
Member Author

I close here, since it appears to be fixed.

@nicosomb nicosomb modified the milestones: 1.7.1, 1.8.0 Jul 7, 2014
@nicosomb nicosomb mentioned this issue Jul 15, 2014
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

5 participants