- 🇮🇳India sanket.addweb
Thanks @Cayenne, I have tested and created a patch for the revision differences in the address field.
- Status changed to Needs review
over 1 year ago 10:31am 25 April 2024 - 🇧🇪Belgium ikeigenwijs
.patch applies , but appears to be not exactly the same issue.
apache log after .patch
PHP Fatal error: Declaration of Drupal\\diff\\Plugin\\diff\\Field\\AddressFieldBuilder::build(Drupal\\Core\\Field\\FieldItemListInterface $field_items) must be compatible with Drupal\\diff\\FieldDiffBuilderInterface::build(Drupal\\Core\\Field\\FieldItemListInterface $field_items): mixed in web/modules/contrib/diff/src/Plugin/diff/Field/AddressFieldBuilder.php on line 72, referer: /node/158564/revisions
- 🇦🇺Australia acbramley
The address module implements this itself https://git.drupalcode.org/project/address/-/blob/2.1.x/src/Plugin/diff/...
Note that there's currently a bug with it for Diff 2.x and the latest stable of address (2.0.4) https://www.drupal.org/project/address/issues/3460590#comment-16132112 🐛 AddressFieldBuilder::build no longer compatible with diff's CoreFieldBuilder::build Active