fabpot/goutte

[nadar]

it seems that one of your deps is abandoned, maybe consider using the suggested replacement? Or is there a specific reason?

Package fabpot/goutte is abandoned, you should avoid using it. Use symfony/browser-kit instead.

[spekulatius]

Hey @nadar

yeah, Goutte is more or less just a wrapper around other packages at this point. It looks as if using the original dependencies instead is the way to go. I'm pretty busy atm. Do you think you could look into this? Otherwise it will have to wait a bit.

Cheers,
Peter

[monsefsolutions]

First of all, thank you @spekulatius for this great scraper and all the work you put into it. I really appreciate it.

I get a similar error like @nadar mentioned when switching to PHP 8.2.6.

The "Goutte\Client" class is deprecated, use "Symfony\Component\BrowserKit\HttpBrowser" instead

I wish I could help with solving the error, but unfortunately, I'm not that familiar with Goutte and BrowserKit. I will wait patiently for the fix and in the meantime switch back to PHP 7.4.x. Again, thanks for this amazing tool.

[spekulatius]

Hello @nadar and @monsefsolutions,

I've pushed a branch called 181-switch-from-goutte-to-browser-kit to address this issue by switching the base lib. Could you clone the repo into your vendor folder and test this? Thank you!

Cheers,
Peter

[nadar]

In order to test branches, packagist automatically publishes branches. you can access them by using dev-<branch-name>. So in order to test i just did "spekulatius/phpscraper": "dev-181-switch-from-goutte-to-browser-kit as 1.0.2" and composer u.

Looks good so far, will make some tests. thanks

[monsefsolutions]

Hi @spekulatius,

Thank you for your quick response, really appreciate it. I just copied the files from the new branch and I do not get any errors anymore with PHP 8.2.6 active. Thank you!

[spekulatius]

Sounds like we are on a good path. I'll check it a bit more merge it and tag a new major release (as the interface changes)

[spekulatius]

I've merged the branch and will tag a new major release once a number of smaller fixes have been included @nadar & @monsefsolutions.

[nadar]

@spekulatius great, thanks for taking care 👍

[monsefsolutions]

@spekulatius Thank you for the quick responses and solution!

[spekulatius]

Hello @nadar & @monsefsolutions

I've pushed the last changes for v2. Could you give the v2 branch a spin and see if you notice any problems before I merge and tag it? Thank you in advance!

Peter

[monsefsolutions]

Hi Peter, Thank you again for the effort! I will test it at the end of this week and let you know.

[spekulatius]

Okay, let me know if you find any issues @monsefsolutions 👍

[spekulatius]

Hello @monsefsolutions and @nadar,

I've tagged a new major release. Feel free to give it a try and let me know if you find any issues.

Cheers,
Peter

[nadar]

Thanks! I just upgraded and tested to 2.0, works 👍