Convert ViewsRelationship plugin discovery to attributes

Open on Drupal.org →

Created on 11 March 2024, about 1 year ago

Updated 30 March 2024, about 1 year ago

Problem/Motivation

In 📌 Use PHP attributes instead of doctrine annotations Fixed we added support for attribute based plugin discovery.
As part of that issue we converted block and action plugins.

This issue is to convert \Drupal\views\Annotation\ViewsRelationshipplugins to use Attributes.

Proposed resolution

Add a class to represent the new Attribute - Example
Update the plugin manager constructor to include both the attribute and annotation class names - example
Convert all plugins that use the annotation to use the new attribute - example

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

📌 Task

Status

Fixed

Version

10.3 ✨

Component

Last updated about 8 hours ago

Maintained by
🇩🇪Germany @dawehner
🇺🇸United States @tim.plunkett
🇬🇧United Kingdom @damiankloip
🇺🇸United States @xjm
🇳🇱Netherlands @Lendude

Created by

🇦🇺Australia mstrelan

Live updates comments and jobs are added and updated live.

Needs Review Queue Initiative
Used to track the progress of issues reviewed by the Drupal Needs Review Queue Initiative.

Sign in to follow issues

Merge Requests

!7034Convert ViewsRelationship plugin discovery to attributes
Open
🇦🇺Australia mstrelan
updated about 1 year ago

Comments & Activities

Issue created by @mstrelan
Merge request !7034Issue #3426956: Convert ViewsRelationship plugin discovery to attributes → (Open) created by mstrelan
Status changed to Postponed about 1 year ago5:34am 14 March 2024
Comment about 1 year ago →
🇦🇺Australia mstrelan
Postponed on 📌 Convert ViewsField plugin discovery to attributes Needs review which has the required changes to ViewsHandlerManager.
Pipeline finished with Failed
about 1 year ago
Total: 527s
#118801
First commit to issue fork.
Pipeline finished with Canceled
about 1 year ago
Total: 490s
#119859
Status changed to Needs review about 1 year ago10:05am 15 March 2024
Comment about 1 year ago →
🇷🇺Russia sorlov
Pipeline finished with Failed
about 1 year ago
Total: 506s
#119871
Status changed to Needs work about 1 year ago2:21pm 15 March 2024
Comment about 1 year ago →
🇺🇸United States smustgrave
Appears to have a test failure, may need a rebase
Pipeline finished with Success
about 1 year ago
Total: 484s
#120212
Status changed to RTBC about 1 year ago3:20pm 15 March 2024
Comment about 1 year ago →
🇺🇸United States smustgrave
Had some bandwidth and rebased which resolved the test failure and MR wasn't in draft so went ahead and reviewed.

Searching core all 5 instances of @ViewsRelationship have been replaced
Comment about 1 year ago →
🇬🇧United Kingdom alexpott 🇪🇺🌍
Committed and pushed 27b07de8bb to 11.x and b6ef30f27b to 10.3.x. Thanks!
Comment about 1 year ago →
System Message

alexpott → committed b6ef30f2 on 10.3.x
Issue #3426956 by sorlov, mstrelan, smustgrave: Convert...
Status changed to Fixed about 1 year ago5:27pm 16 March 2024
Comment about 1 year ago →
System Message

alexpott → committed 27b07de8 on 11.x
Issue #3426956 by sorlov, mstrelan, smustgrave: Convert...
Comment about 1 year ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024