Add PHP 8.2 support

The PHP 8.2 failure in the test is a problem with core, I'll look for the relevant issue if not create it.

Hi,
Coming from https://www.drupal.org/project/feeds/issues/3344359 🐛 D10 Deprecated function: Creation of dynamic property Drupal\feeds\Form\MappingForm Closed: duplicate
After your reply there I Installed latest .dev and stil seeing

Deprecated function: Creation of dynamic property Drupal\feeds\Form\MappingForm etc..

I thought I report it then here also.

Thanks for your reply in advance.

Greetings, Martijn

@Summit
You did apply the changes from this issue? The changes here are not merged to the latest dev yet.

Hi, is it my mistake that I do not see a patch to apply? Then sorry, I am not a programmer :)
Otherwise I will look how to apply the changes. Greetings,

See the plain diff link on the issue fork.

I do not have experience with plain diff links, but it works like composer with patch I presume. Just download
https://git.drupalcode.org/project/feeds/-/merge_requests/100.diff and use it as patch? If so I will try this. Greetings,

@Summit
Yes, for more information see https://www.drupal.org/docs/develop/git/using-gitlab-to-contribute-to-dr... →

I've been using this MR for over three months now with PHP 8.2 and I haven't seen any deprecation-related errors in the logs.

Note: true PHP 8.2 support on Drupal 10 is not possible until the following issue is fixed:
https://github.com/laminas/laminas-servicemanager/issues/92

I've merged the code! Thanks for testing @ptmkenny

Marking this issue as postponed until https://github.com/laminas/laminas-servicemanager/issues/92 is fixed. Earlier, I think we cannot do anything to make Feeds installable through Composer without issues on Drupal 10 + PHP 8.2.

Closed 🐛 Problem laminas - installation is not possible Closed: duplicate as a duplicate.

With https://github.com/laminas/laminas-servicemanager/pull/188 merged into the 4.0 branch, can this be picked back up?

We are still out of luck here. While "laminas/laminas-servicemanager" now has a release candidate for its 4.0 branch, "laminas/laminas-feed" hasn't declared compatibility with it yet.

Maybe I should open an issue there.

I've opened an issue on laminas/laminas-feed: https://github.com/laminas/laminas-feed/issues/75.

A quick reply on that indicates that laminas/laminas-servicemanager first needs a full 4.0 release before laminas/laminas-feed can be made compatible with that version.

So I guess that people who would like to use Feeds on Drupal 10 + PHP 8.2 should test laminas/laminas-servicemanager 4.0.0-RC2.

A new release of laminas/laminas-feed was made today where the dependency on laminas/laminas-servicemanager has been removed.

Now drupal/feeds can be installed on PHP 8.2! I just tested this manually, using composer require drupal/feeds on a Drupal 10.1.1 install. Also, now the Feeds tests pass on PHP 8.2 → !

So that makes this issue be fixed.

Automatically closed - issue fixed for 2 weeks with no activity.