- 🇺🇸United States smustgrave
This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request → as a guide.
Please include an interdiff with your patch
We can't just add this here, we have to provide backward compatibility and a deprecation in case someone extended this class. We can't add new deprecations in 9.5.x or 10.0.x so this has to go to 10.1.x now.
has not been addressed
Tagging for change record also for the new paramter
- 🇬🇧United Kingdom longwave UK
Retitling the issue given that Seven has been removed, but Claro has the same code.
- Assigned to spokje
- @spokje opened merge request.
- Issue was unassigned.
- Status changed to Needs review
almost 2 years ago 1:47pm 28 January 2023 - Status changed to RTBC
almost 2 years ago 6:27pm 28 January 2023 - 🇺🇸United States smustgrave
Thank you @Spokje for adding the trigger_error.
Reviewing the MR code everything looks clean
Deprecation has been added
Deprecation is being tested
New functions have correct typehintsRan the test locally without the fix and got
Failed asserting that string matches format description. --- Expected +++ Actual @@ @@ @expectedDeprecation: -%A Calling Drupal\media_library\Plugin\Field\FieldWidget\MediaLibraryWidget::__construct without
Verified that the claro media library widget works fine.
Great job!
- Status changed to Needs work
almost 2 years ago 7:37pm 14 February 2023 - 🇺🇸United States bnjmnm Ann Arbor, MI
Tested manually in Claro and spotted some regressions
- "Show media item weights" button is not styled and is mising the following classes
.action-link action-link--extrasmall .action-link--icon-show .media-library-widget__toggle-weight
- "Add Media" button is missing the
.media-library-open-button
class. This one I'm unsure if the class is used anywhere, but even if Claro is internal this should probably be preserved unless there's a compelling reason to get rid of it.
- "Show media item weights" button is not styled and is mising the following classes
- last update
over 1 year ago 29,456 pass - last update
over 1 year ago 29,456 pass - 🇳🇱Netherlands spokje
This is the plain diff from the 10.1.x: https://git.drupalcode.org/project/drupal/-/merge_requests/3323.diff
Using that on top of a new MR, based on 11.x.
- last update
over 1 year ago 29,909 pass - @spokje opened merge request.
- 🇺🇸United States alphex Atlanta, GA USA
This needs to be re-rolled for Drupal 10.3.6