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.
Problem/Motivation
While HandlerAllTest covers some things, it doesn't covers broken handlers, see #2560897: Drupal\user\Plugin\views\argument\RolesRid is very broken
Proposed resolution
Remaining tasks
User interface changes
API changes
Data model changes
Comment | File | Size | Author |
---|---|---|---|
#21 | 2560949-21.patch | 512 bytes | NivethaSubramaniyan |
#20 | 2560949-20.patch | 537 bytes | NivethaSubramaniyan |
#19 | 2560949-19.patch | 513 bytes | Medha Kumari |
#16 | 2560949-16.patch | 507 bytes | SandeepSingh199 |
#2 | 2560949-2.patch | 4.93 KB | dawehner |
Comments
Comment #2
dawehnerHere is a first part.
Comment #15
quietone CreditAttribution: quietone at PreviousNext commentedAdding a test is a task, changing category.
Comment #16
SandeepSingh199 CreditAttribution: SandeepSingh199 at Srijan | A Material+ Company for Drupal India Association commentedI have checked in 9.4.x , there is no folder exits with name "Handler" or file name HandlerAllTest.php, that’s why we are getting error.
re-rolling patch #2 only with ViewsHandlerManager.php.
Comment #17
rpayanmComment #18
rpayanmComment #19
Medha KumariRerolled the patch #16 in Drupal 10.1.x.
Comment #20
NivethaSubramaniyan CreditAttribution: NivethaSubramaniyan at Srijan | A Material+ Company commentedFixing CCF for the 10.1.x version. Reference - https://www.drupal.org/node/3192283
Comment #21
NivethaSubramaniyan CreditAttribution: NivethaSubramaniyan at Srijan | A Material+ Company commentedFixing CCF and File permission issue
Comment #22
smustgrave CreditAttribution: smustgrave at Mobomo commentedThe file is under the tests/src/functional folder. Since the patches #16-#21 remove functionality removing credit. Still needs a reroll for #2
Comment #23
smustgrave CreditAttribution: smustgrave at Mobomo commentedComment #24
smustgrave CreditAttribution: smustgrave at Mobomo commentedAlso 99% sure debug($item); was not meant to be there.
Comment #25
smustgrave CreditAttribution: smustgrave at Mobomo commentedBefore work is done on this wonder if we can get an issue summary update please.
What is the proposed solution exactly?