- Issue created by @alberto56
### Issue: Fix List Definition Issue
This patch resolves the issue where the list definition was incorrect.### Steps to Reproduce the Issue:
1. Navigate to the affected page in the admin panel.
2. Observe that the incorrect list structure causes an error.### Steps Taken to Fix the Issue:
- Adjusted the array structure to match Drupal's expected format.
- Ensured compatibility with other list-based components.### How to Test the Patch:
1. Apply this patch using `git apply myfix-3436243-1.patch`.
2. Clear the cache using `drush cache:rebuild`.
3. Verify that the issue is resolved by checking the affected page.### Next Steps:
Please review this patch and provide feedback. 🚀- First commit to issue fork.
- Merge request !11416Issue #3511434 by juandhr: Remove reference in DocBlocks to non-existent... → (Open) created by Unnamed author
- 🇨🇴Colombia juandhr Manizales
Hi @alberto56,
I've updated the docblock in Drupal\Core\TypedData\ListInterface and Drupal\Core\TypedData\DataDefinitionInterface interfaces with the existing Drupal\Core\TypedData\ListDataDefinitionInterface interface, please check.
- 🇳🇿New Zealand quietone
Thanks for working on this!
The "proposed resolution" section should be complete and explain the actual change to help the reviewer and committer. Also, this should have some research to support that the change is correct. That is, what is the issue that replaces \Drupal\Core\TypedData\ListDefinitionInterface with something else. Using git blame should make it simple to identify when the changes was made.
Changes are made on on 11.x (our main development branch) first, and are then back ported as needed according to the Core change policies → .
- 🇳🇿New Zealand quietone
This needs confirmation that the class name change is correct, as stated in #10.
What should happen is to find the commit where the old name was changed and read that commit. This command should help find the commit,
git log -S"\Drupal\Core\TypedData\ListDefinitionInterface"
.I have restored the 'remaining tasks' to add this point.
- 🇮🇳India Sivaji_Ganesh_Jojodae Chennai
I don't find reference to original interface name change. However I can see usage change in issue #2002134 → commit id f6161fad