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

Fixed method parsearticles #274

Closed
wants to merge 2 commits into from
Closed

Fixed method parsearticles #274

wants to merge 2 commits into from

Conversation

duck7000
Copy link
Contributor

This method was not working.
#259 reveals that the problem is in this method
I modified it al lot to get rid of at least one regular expression and easier to read

it also fixes these methods as well as they use this parsearticles() :
interviews(), articles(), pictorials(), magcovers()

I don't just understand why auturl and inturl are there?
I suspect for compatibility reasons but it doesn't make any sense to me

This method was not working.
#259 reveals that the problem is in this method
@duck7000
Copy link
Contributor Author

@ALL can someone test this method so that we can assure it now works properly?

@Thomasdouscha
Copy link

Actually i dont use these methods. But i am gonna try and test in weekend.

@duck7000
Copy link
Contributor Author

Is this something that is still used by someone?
I fixed this but there seems no interest in it at all

@tboothman
Copy link
Owner

Could you add a test please for whatever it is that this is fixing. As far as I can see the only test that covers this method is test_magcovers. It passed before and it passes now which is great but then what are all the changes for? Some of it seems to be dealing with dates expressed as day month year, month year and year but I have no idea what the last half is doing.

I'd be fine with removing inturl and auturl as they've been empty strings since 2011. It looks like the publicity listings used to link somewhere.

@duck7000
Copy link
Contributor Author

Wow i'm getting sick and tired of lax attitude here!
This pr is open since feb this year (that is almost 8 months later!) and this is the first time you even responded, and your response complains about what this is fixing?
Well i suggest you start reading the issue again!
The whole idea about my modifications is to deal with different formatting of dates, years etc and with the details part (the part you do not seems to understand)
And the original poster of that issue tested my fixes and confirms it works for his problem..

Even if i did made a test it would still not even been reviewed, now would it?

I have the feeling that the attitude of maintainers here is to sit and wait if someone complains about something isn't working, or if they self have a problem with imdbphp. At least @jreklund admits that he only commits to this repo if he has a problem in his own project.

This is de second time i explode like this and that will be the last one, i quit doing this any longer.

@duck7000 duck7000 closed this Oct 22, 2022
@duck7000 duck7000 deleted the parseArticles branch October 22, 2022 17:15
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

Successfully merging this pull request may close these issues.

3 participants