[7.x] Reintroduce parser error for mixed pipe/shorthand modifier syntax
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 66/100
- Tipo de issue
- Documentación
- Claridad
- Bien especificado
- Estado de actividad
- Tranquilo
- Stack tecnológico
- php
- Área
- documentation
Línea de trabajo
Dos de las tres tareas ya están completadas: el PR #15075 restauró la lógica de errores del analizador de #14892 y la prueba de regresión se eliminó. El único trabajo abierto es la tercera tarea: documentar el cambio de comportamiento en la guía de actualización. Comience leyendo el PR #15075 y el PR revertido #15073 para capturar el cambio exacto visible para el usuario: mezclar modificadores de tubería (pipe) y key=value (p. ej. {{ last_modified | format="U" }}) ahora lanza un error del analizador. Se considera terminado cuando la guía de actualización contiene una entrada que describe que esta sintaxis que antes funcionaba ahora produce un error. Tenga en cuenta que el issue no tiene comentarios después de 72 días, así que confirme que la guía de actualización reside en este repositorio antes de comenzar.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
#14892 threw a parser error when a tag mixed pipe (|) and shorthand key=value modifier syntax, fixing #13080. Reverted in #15073 — it broke previously working syntax like {{ last_modified | format="U" }}, a breaking change for a minor release.
Mixing both styles is always invalid and should always throw, e.g.:
{{ active_platforms | sort="value" }}
...
{{ /active_platforms }}
{{ last_modified | format="U" }}
Tasks:
- Bring back the logic from #14892 -> #15075
- Drop the regression test
- Mention the changes in the upgrade guide
Related: #13080, #14892, #15073
- Lenguaje dominante
- PHP
- Estrellas
- 4.9k
- Forks
- 647
- Merge medio
- 1 d 7 h
- PR fusionados (30 d)
- 103
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de statamic/cms
-
preferences
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
There's a default crop when cropping imagesPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abiertoassets
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
statamic/cms#15567 · 1 comentario · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
Make native filesystem directory permissions configurable to support shared POSIX ACL accessAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
Los mantenedores suelen responder en 1 día
-
accessibility
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
Los mantenedores suelen responder en 1 día
-
safari ui
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
Los mantenedores suelen responder en 1 día
Todos los issues de statamic/cms
Issues similares
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
-
cloning objets from planningAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
glpi-project/glpi#25922 ·
Los mantenedores suelen responder en 1 día
-
Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
Automattic/safe-publish#726 ·
Los mantenedores suelen responder en 1 día
-
Translation TwigBridge
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
symfony/symfony-docs#23308 ·
Los mantenedores suelen responder en 3 días
-
sync-en
Dificultad 1/5 Menos de una hora Aptitud para principiantes 84/100
Los mantenedores suelen responder en 1 día