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
In the
function hook_validation_constraint_alter(array &$definitions) {
$definitions['Null']['class'] = '\Drupal\mymodule\Validator\Constraints\MyClass';
}
"\Validator\" should be replaced by "\Plugin\Validation\"
Proposed resolution
The function should become
function hook_validation_constraint_alter(array &$definitions) {
$definitions['Null']['class'] = '\Drupal\mymodule\Plugin\Validation\Constraints\MyClass';
}
Remaining tasks
User interface changes
API changes
| Comment | File | Size | Author |
|---|---|---|---|
| #10 | ForHO5d - Imgur.png | 64.62 KB | ilgnerfagundes |
| #7 | 3197135-applied_patch.png | 65.99 KB | Abhijith S |
| #6 | 3197135-6.patch | 527 bytes | Shashwat Purav |
Comments
Comment #2
le72Comment #3
cilefen CreditAttribution: cilefen as a volunteer commentedThere is not a diff or a branch to review, so I am setting this back to "Active".
Comment #4
larowlanComment #5
MeenakshiG CreditAttribution: MeenakshiG at OpenSense Labs for Drupal India Association commentedComment #6
Shashwat Purav CreditAttribution: Shashwat Purav at QED42 for Drupal Association commentedPatch added.
Comment #7
Abhijith S CreditAttribution: Abhijith S as a volunteer and at Zyxware Technologies commentedApplied patch #6 in 9.1.x.The path of validator is changed by this patch.
Comment #8
MeenakshiG CreditAttribution: MeenakshiG at OpenSense Labs for Drupal India Association commentedComment #9
le72Thank you for creating a patch file. Changing status to reviewed.
Comment #10
ilgnerfagundes CreditAttribution: ilgnerfagundes at CI&T commentedThe patch works correctly, RTBC +1
Comment #13
catchCommitted/pushed to 9.2.x and cherry-picked to 9.1.x, thanks!
Comment #14
alexpott@ilgnerfagundes as per #3189587-8: Add documentation for taxonomy terms source plugins and elsewhere, thank you for looking into this issue , but posting screenshots of your codebase or CLI does not advance the issue, since the automated testing infrastructure tells us whether the patch applies correctly.
So, I've removed the issue credit for that screenshot. In the future, you can get credit for issues by reading the issue to understand its purpose, and posting your review or testing of that purpose. Thank you!
Comment #15
ilgnerfagundes CreditAttribution: ilgnerfagundes at CI&T commentedbut alex i just put the screenshot to prove that i downloaded the patch, i ran it and did the tests, people above didn't even put any print and still proved it worked and i didn't?