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 in Readability.js ? #111

Open
fernand0 opened this issue Sep 29, 2024 · 2 comments
Open

Error in Readability.js ? #111

fernand0 opened this issue Sep 29, 2024 · 2 comments

Comments

@fernand0
Copy link

I'm using readabilitpy 0.2.0 and I've found what I think is an error. In file Readability.js, in line 1227, there is the line:

             values[name] = content.trim();

But content can be empty and it will produce an error.
I've solved it in line 1219, adding a check for content:

       if (elementProperty && content) {

But I'm not sure if it is the correct way to deal with the issue. I can send a PR if this is ok.

Best regards, thank you and keep up the good work!

@jemrobinson
Copy link
Member

Hi @fernand0. This project isn't being actively maintained. If you'd like to submit a PR, I'm happy to review it though.

@fernand0
Copy link
Author

fernand0 commented Oct 7, 2024

Hello,

don't worry, it seems to be a problem of Mozilla readability.js (which your project seems to be installing using npm) and it seems to be solved now.

Thanks!

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

2 participants