imdbphp icon indicating copy to clipboard operation
imdbphp copied to clipboard

Added new method main awards

Open duck7000 opened this issue 3 years ago • 8 comments

This method get only the awards info at the title page inside the yellow block. I made this because this is a simple version for if you only want this info instead of all awards (witch can be a lot of information)

It may be useful to others as well.

duck7000 avatar Feb 19 '22 19:02 duck7000

I'm aware that this needs a test, there never was one so i will, in time, to figure something out

duck7000 avatar Feb 25 '22 11:02 duck7000

I will make a test but after this is reviewed

duck7000 avatar Mar 14 '22 18:03 duck7000

I assume that there is no interest in this new method. If someone do report to this PR

If no response i will close this as it has no point if no one is interested. no harm feelings here

duck7000 avatar Apr 08 '22 13:04 duck7000

it is interesting to me. I can test it. But give me the files than i will do it.

Thomasdouscha avatar Apr 08 '22 20:04 Thomasdouscha

1.test tt0068646 {"awards":"Won 3 Oscars","wins":"31","nominations":"30"} 2.test tt0120338 {"awards":"Won 11 Oscars","wins":"125","nominations":"83"} 3.test tt5034838 {"awards":"","wins":"2","nominations":"14"} 4.test tt6156584 {"awards":"","wins":"4","nominations":"22"} 5.test tt2560140 empty

It works quite well.

Thomasdouscha avatar Apr 08 '22 21:04 Thomasdouscha

Okay that is nice to here you have interest and that it works.

I'm not sure @tboothman will merge this though, it means another extra method to maintain but if necessary i will do the maintenance

duck7000 avatar Apr 09 '22 10:04 duck7000

I think he will merge it and it returns summarize info which exactly we needed. Thanks a lot @duck7000 ! Good job!

Thomasdouscha avatar Apr 09 '22 13:04 Thomasdouscha

@tboothman please let you merge it. Of course it is your call!

Thomasdouscha avatar Apr 09 '22 13:04 Thomasdouscha

@tboothman , @jreklund Last try, can/will you merge this or not? Or at least give your opinion about this?

duck7000 avatar Oct 10 '22 10:10 duck7000

It was working in my tests.

Thomasdouscha avatar Oct 10 '22 11:10 Thomasdouscha