Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Based on #1741094: [meta] Move as much metadata out of annotations as possible we should should convert this three flags, which are connected to each other in one part.
Comment | File | Size | Author |
---|---|---|---|
#9 | 1744334-uses_fields_row_class_plugin-9.patch | 21.8 KB | dawehner |
#9 | interdiff.txt | 16.72 KB | dawehner |
#7 | interdiff.txt | 931 bytes | dawehner |
#6 | 1744334-uses_fields_row_class_plugin-6.patch | 10.92 KB | dawehner |
#4 | 1744334-uses_fields_row_class_plugin-4.patch | 9.39 KB | dawehner |
Comments
Comment #1
dawehnerHere is a first version for that.
Comment #2
dawehner.
Comment #4
dawehnerTim found the problem with calling !empty()
Here is a new version of the patch
Comment #6
dawehnerI had missing one of them, but hey that's what we have tests for.
Comment #7
dawehner.
Comment #9
dawehnerUpdated to use camelCase.
Comment #10
tim.plunkettThis looks good to me. At first I was confused by the 'uses_fields' in lib/Drupal/views/Plugin/views/style/StylePluginBase.php, but those are for a form field!
Comment #11
tim.plunkettCommitted.
Comment #13
xjmComment #14
xjmComment #15
xjmComment #16
Chris Matthews CreditAttribution: Chris Matthews as a volunteer and at City of Oaks Design commentedFor more information as to why this issue was moved to the Drupal core project, please see issue #3030347: Plan to clean process issue queue
Comment #17
Chris Matthews CreditAttribution: Chris Matthews as a volunteer and at City of Oaks Design commentedMoving back to the contributed Views issue queue and closing as outdated per https://www.drupal.org/project/views/issues/3030347#comment-13023447