Posted by damiankloip on October 29, 2012 at 11:35am
9 followers
| Project: | Drupal core |
| Version: | 8.x-dev |
| Component: | views.module |
| Category: | task |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | needs review |
| Issue tags: | DX (Developer Experience), VDC |
Issue Summary
Similar to #1810480: Provide the plugin_id to support views metadata integration, we will benefit from also having the module that implemented or owns the plugin.
This is in preparation for what we will do with broken/missing handlers. Which is also related to #1823608: Admin views in core.
Comments
#1
#2
This should be very quick to implement, we just add this to addItem.
Now we just need to add this to all the current YAML files.... *Looks at dawehner*
#3
Updated all the yml files with the script in the attachment.
Just linking an issue with does something related but at the exact space in code. #1810480: Provide the plugin_id to support views metadata integration
#4
The last submitted patch, drupal-1825896-3.patch, failed testing.
#5
Let's postpone this on #1810480: Provide the plugin_id to support views metadata integration as it requires the same kind of assertions.
#6
.
#7
As per #5
#8
See also: #1965164: Add a @PluginID annotation for plugins that only need an ID, like Views handlers
#9
Here is an initial patch, which also fixes some of the yml files, so my script could work.
#10
The last submitted patch, drupal-1825896-9.patch, failed testing.
#11
Just a rerole. (frontpage got changed)
#12
The last submitted patch, drupal-1825896-11.patch, failed testing.
#13
#11: drupal-1825896-11.patch queued for re-testing.
#14
The last submitted patch, drupal-1825896-11.patch, failed testing.
#15
Yet another one.
#16
The last submitted patch, drupal-1825896-15.patch, failed testing.
#17
It's great to see that our test coverage works properly.
#18
The last submitted patch, drupal-1825896-17.patch, failed testing.
#19
#17: drupal-1825896-17.patch queued for re-testing.