Skip to content
Snippets Groups Projects
Commit 97c49565 authored by catch's avatar catch
Browse files

Issue #3416549 by Wim Leers: ConfigExistsConstraintValidator should ignore NULL values

parent 3951fa4d
No related branches found
No related tags found
25 merge requests!8528Issue #3456871 by Tim Bozeman: Support NULL services,!3878Removed unused condition head title for views,!38582585169-10.1.x,!3818Issue #2140179: $entity->original gets stale between updates,!3742Issue #3328429: Create item list field formatter for displaying ordered and unordered lists,!3731Claro: role=button on status report items,!3668Resolve #3347842 "Deprecate the trusted",!3651Issue #3347736: Create new SDC component for Olivero (header-search),!3531Issue #3336994: StringFormatter always displays links to entity even if the user in context does not have access,!3355Issue #3209129: Scrolling problems when adding a block via layout builder,!3226Issue #2987537: Custom menu link entity type should not declare "bundle" entity key,!3154Fixes #2987987 - CSRF token validation broken on routes with optional parameters.,!3133core/modules/system/css/components/hidden.module.css,!2812Issue #3312049: [Followup] Fix Drupal.Commenting.FunctionComment.MissingReturnType returns for NULL,!2378Issue #2875033: Optimize joins and table selection in SQL entity query implementation,!2334Issue #3228209: Add hasRole() method to AccountInterface,!2062Issue #3246454: Add weekly granularity to views date sort,!1105Issue #3025039: New non translatable field on translatable content throws error,!1073issue #3191727: Focus states on mobile second level navigation items fixed,!10223132456: Fix issue where views instances are emptied before an ajax request is complete,!877Issue #2708101: Default value for link text is not saved,!617Issue #3043725: Provide a Entity Handler for user cancelation,!579Issue #2230909: Simple decimals fail to pass validation,!560Move callback classRemove outside of the loop,!555Issue #3202493
Pipeline #91188 canceled
......@@ -43,6 +43,11 @@ public static function create(ContainerInterface $container) {
* {@inheritdoc}
*/
public function validate(mixed $name, Constraint $constraint) {
// This constraint may be used to validate nullable (optional) values.
if ($name === NULL) {
return;
}
if (!in_array($name, $this->configFactory->listAll(), TRUE)) {
$this->context->addViolation($constraint->message, ['@name' => $name]);
}
......
......@@ -40,6 +40,10 @@ public function testValidation(): void {
$this->installConfig('system');
$this->assertCount(0, $data->validate());
// NULL should not trigger a validation error: a value may be nullable.
$data->setValue(NULL);
$this->assertCount(0, $data->validate());
}
}
  • catch @catch

    mentioned in commit a7d396f1

    ·

    mentioned in commit a7d396f1

    Toggle commit list
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment