Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

Not restoring parent Each's key on some conditions

Abierto Apto para principiantes
  • ¿Algo no está bien?
#823 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
75/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
php
Área
backend-api-design

Línea de trabajo

El problema está en la clase EachHandler, específicamente en las dos declaraciones return que no restauran la clave del Each padre. Comience por localizar el archivo EachHandler en la base de código. Examine el método validate para comprender el flujo. La solución implica agregar $context->setParameter(Each::PARAMETER_EACH_KEY, $originalEachKey); antes de cada return problemático. Ejecute las pruebas existentes relacionadas con la validación de Each para asegurarse de que la solución no rompa nada.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

Description

In EachHandler there are 2 returns without restoring in context parent Each's key.

Code in question:

        $originalEachKey = $context->getParameter(Each::PARAMETER_EACH_KEY); // <== storing parent key

        foreach ($value as $index => $item) {
            if (!is_int($index) && !is_string($index)) {
                return (new Result())->addError($rule->getIncorrectInputKeyMessage(), [ // <== bug, return without restoring parent key
                    'property' => $context->getTranslatedProperty(),
                    'Property' => $context->getCapitalizedTranslatedProperty(),
                    'type' => get_debug_type($index),
                ]);
            }

            $context->setParameter(Each::PARAMETER_EACH_KEY, $index);

            $itemResult = $context->validate($item, $rules);
            if ($itemResult->isValid()) {
                continue;
            }

            foreach ($itemResult->getErrors() as $error) {
                $result->addErrorWithoutPostProcessing(
                    $error->getMessage(),
                    $error->getParameters(),
                    $error->getValuePath() === [] ? [$index] : [$index, ...$error->getValuePath()],
                );
            }

            if ($rule->stopOnError) {
                return $result; // <== bug, return without restoring parent key
            }
        }

        $context->setParameter(Each::PARAMETER_EACH_KEY, $originalEachKey); // <== restoring parent key

        return $result;
Package version

No response

PHP version

No response

Lenguaje dominante
PHP
Estrellas
167
Forks
47
Merge medio
1 d 16 h
PR fusionados (30 d)
4

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de yiisoft/validator

Todos los issues de yiisoft/validator

Issues similares

Más issues de PHP

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.