Skip to content
Snippets Groups Projects
Unverified Commit c92e4d23 authored by Alex Pott's avatar Alex Pott
Browse files

Issue #3480293 by gapple: ConfigTarget::__construct() documentation references...

Issue #3480293 by gapple: ConfigTarget::__construct() documentation references incorrect ToConfig enum name

(cherry picked from commit ba24f059)
parent 7fda601b
No related branches found
No related tags found
2 merge requests!11769Issue #3517987: Add option to contextual filters to encode slashes in query parameter.,!11185Issue #3477324 by andypost, alexpott: Fix usage of str_getcsv() and fgetcsv() for PHP 8.4
Pipeline #339409 passed with warnings
Pipeline: drupal

#339471

    Pipeline: drupal

    #339467

      Pipeline: drupal

      #339459

        +7
        ......@@ -74,8 +74,8 @@ final class ConfigTarget {
        * return an array with the transformed values, also keyed by property path.
        * The callback will receive the form state object as its second argument.
        * The callback may return a special values:
        * - ToConfig::NoMapping, to indicate that the given form value does not
        * need to be mapped onto the Config object
        * - ToConfig::NoOp, to indicate that the given form value does not need to
        * be mapped onto the Config object
        * - ToConfig::DeleteKey to indicate that the targeted property path should
        * be deleted from config.
        * Defaults to NULL.
        ......@@ -237,7 +237,7 @@ public function setValue(Config $config, mixed $value, FormStateInterface $form_
        }
        // Set the returned value, or if a special value (one of the cases in the
        // ConfigTargetValue enum): apply the appropriate action.
        // ToConfig enum): apply the appropriate action.
        array_walk($value, fn (mixed $value, string $property) => match ($value) {
        // No-op.
        ToConfig::NoOp => NULL,
        ......
        ......@@ -286,14 +286,14 @@ public function testSingleTarget(): void {
        $config->set('something', FALSE)->shouldBeCalledTimes(1);
        $config_target->setValue($config->reveal(), 'No', $this->prophesize(FormStateInterface::class)->reveal());
        // Test `ConfigTargetValue::NoMapping`: nothing should happen to the Config.
        // Test `ToConfig::NoOp`: nothing should happen to the Config.
        $config = $this->prophesize(Config::class);
        $config->getName()->willReturn('foo.settings');
        $config->set('something', Argument::any())->shouldBeCalledTimes(0);
        $config->clear('something', Argument::any())->shouldBeCalledTimes(0);
        $config_target->setValue($config->reveal(), '<test:noop>', $this->prophesize(FormStateInterface::class)->reveal());
        // Test `ConfigTargetValue::DeleteKey`: Config::clear() should be called.
        // Test `ToConfig::DeleteKey`: Config::clear() should be called.
        $config = $this->prophesize(Config::class);
        $config->getName()->willReturn('foo.settings');
        $config->clear('something')->shouldBeCalledTimes(1);
        ......
        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