DocBlocks for filters inside a conditional expression are not imported
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- php
- Área
- documentation
Línea de trabajo
Empieza reproduciendo los dos ejemplos de PHP y siguiendo cómo se asocian las llamadas a apply_filters dentro de expresiones condicionales con los DocBlocks. Compara el resultado del parser para ambos ejemplos con la documentación esperada de los filtros, incluido wp_save_post_revision_check_for_changes. Se considera terminado cuando DocBlock 2 está asignado a in_conditional_expression y el DocBlock correcto está asignado a in_conditional.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
DocBlocks outside a conditional are not imported if there are other filters (with DocBlocks) within the conditional.
Example 1
In this example DocBlock 2 is not imported for the filter in_conditional_expression
/**
* DocBlock 1
*/
function test_docblocks() {
/**
* DocBlock 2
*/
if ( apply_filters( 'in_conditional_expression', true ) ) {
/**
* DocBlock 3
*/
$value = apply_filters( 'in_conditional', true );
}
}
Example 2
In this example the (wrong) DocBlock 2 is imported for the filter 'in_conditional`.
/**
* DocBlock 1
*/
function test_docblocks() {
/**
* DocBlock 2
*/
if ( apply_filters( 'in_conditional_expression', true ) ) {
$value = apply_filters( 'in_conditional', true );
}
}
The wp_save_post_revision_check_for_changes filter doesn't get its DocBlock imported because of this bug.
See this filter in trac
- Lenguaje dominante
- PHP
- Estrellas
- 239
- Forks
- 81
- Merge medio
- 16 h 29 min
- PR fusionados (30 d)
- 8
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
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 WordPress/phpdoc-parser
-
Include constants in exports Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
WordPress/phpdoc-parser#269 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 64/100
WordPress/phpdoc-parser#268 · 1 comentario ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
WordPress/phpdoc-parser#254 ·
-
Compatibility updates required Abierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
WordPress/phpdoc-parser#246 · 9 comentarios ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
WordPress/phpdoc-parser#235 · 1 comentario ·
Todos los issues de WordPress/phpdoc-parser
Issues similares
-
tooling
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
UX
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
ProfessionalWiki/NeoWiki#1525 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
OpenConext/OpenConext-engineblock#2122 ·
-
Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
Automattic/safe-publish#594 ·