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

Issue #3488179 follow-up by alexpott: RecipeConfigurator::getIncludedRecipe()...

Issue #3488179 follow-up by alexpott: RecipeConfigurator::getIncludedRecipe() should statically cache recipe objects to avoid performance problems

(cherry picked from commit e2a92ae2)
parent 66f97b08
No related branches found
No related tags found
1 merge request!11185Issue #3477324 by andypost, alexpott: Fix usage of str_getcsv() and fgetcsv() for PHP 8.4
Pipeline #344270 passed with warnings
Pipeline: drupal

#344305

    Pipeline: drupal

    #344301

      Pipeline: drupal

      #344296

        +7
        ...@@ -133,15 +133,15 @@ public function describeAll(): array { ...@@ -133,15 +133,15 @@ public function describeAll(): array {
        * constraints. * constraints.
        */ */
        public function collectAll(InputCollectorInterface $collector, array &$processed = []): void { public function collectAll(InputCollectorInterface $collector, array &$processed = []): void {
        if (is_array($this->values)) {
        throw new \LogicException('Input values cannot be changed once they have been set.');
        }
        // Don't bother collecting values for a recipe we've already seen. // Don't bother collecting values for a recipe we've already seen.
        if (in_array($this->prefix, $processed, TRUE)) { if (in_array($this->prefix, $processed, TRUE)) {
        return; return;
        } }
        if (is_array($this->values)) {
        throw new \LogicException('Input values cannot be changed once they have been set.');
        }
        // First, collect values for the recipe's dependencies. // First, collect values for the recipe's dependencies.
        /** @var \Drupal\Core\Recipe\Recipe $dependency */ /** @var \Drupal\Core\Recipe\Recipe $dependency */
        foreach ($this->dependencies->recipes as $dependency) { foreach ($this->dependencies->recipes as $dependency) {
        ......
        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