Implement DERItem::getReferenceableBundles

The blocker for this has had some movement, would be great to get some reviews on the new test coverage :)

🐛 ResourceTypeRepository wrongly assumes that all entity reference fields have the setting "target_type" Fixed has been committed.

🥳

#5 is still pending.

Re-roll #4.

RE #5: the core issue also didn't not add tests for ::getReferenceableBundles() in 🐛 ResourceTypeRepository wrongly assumes that all entity reference fields have the setting "target_type" Fixed . All the test added in that issue is for deprecation. Also unit test is going to only check that for a given mock object we have some array structure that is already being tested in DER Kernel tests so what exactly trying to test then? I'd say we should add functional tests that can help close ✨ Dynamic Entity Reference normalization support Active .

HEAD is failing on 11.x atm after field UI changes. I'm working on fixing it. So ignore the test fails for now but it needs functional test and mentioned above.

This looks good to go to me

Upgraded to 10.2.0 today and got bit by this. Can we land this and tag a new release?

There are some other fails as well. Please see https://github.com/jibran/dynamic_entity_reference/pull/9. Feel free to make it green. I'm trying to find some time but no luck.

I think we can fix this and create a follow up the field settings of DER.

I had some time to work on this today. I didn't solve it, but I did figure out where the fault lies. If you manually install DER module on 10.2 and install the inline_form_errors module, you'll see that the issue is with the new field wizard process added in 10.2.

DynamicEntityReferenceItem::storageSettingsFormValidate is triggered and we can't save the field form.

Let's see how much this fixes. I hope I got all the fixes pulled in from the referenced PR.

Hmm, can we enable gitlab integration? It looks like we are being gently nudged to move that direction as I can't run the test bot w/ 10.2 any more.

@heddn it's enabled by default now... You may need to include a .gitlab-ci.yml file, but there are instructions on doing so.

It does not however work with patches so you will need to re-roll this into an MR.

There is already a PR for it. https://git.drupalcode.org/project/dynamic_entity_reference/-/merge_requ.... It is not merged as it is not passing yet.

@jibran that MR has a lot of changes that are out of scope for getting tests passing, and seem to overlap with 🐛 Declaration of DynamicEntityReferenceItemNormalizer::normalize() must be compatible Needs review and/or 🐛 DynamicEntityReferenceItemNormalizer broken on D10 Needs review as well.

It would be great to know the list of priorities to getting DER 10.1 compatible and perhaps constrain the gitlab CI MR to just adding the pipeline and getting tests green?

NW because #20 isn't applying

I'm happy to commit #14 as is and fix the remaining fails in follow-up issue(s).

Here is the roadmap to fix 10.2 integration.

• 🐛 Implement EntityReferenceItemInterface Needs work
• 📌 For 10.2 and above the logic to prevent field hooks relating to selection needs to target a different hook Active
• 📌 PHPCS fixes form Drupal 10.2 Active

Committed and pushed #14. Thanks!

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