Add a check for upgrader_process_complete
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 50/100
Línea de trabajo
Empieza leyendo las convenciones existentes de sniff y pruebas del repositorio, y después sigue cómo se detectan las llamadas al hook de WordPress upgrader_process_complete. El issue proporciona el hook objetivo y un ejemplo de add_action que infringe la regla; el trabajo está terminado cuando el sniff informa de ese uso como un error sin que se haya identificado ningún caso de falso positivo.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
What problem would the enhancement address for VIP?
Because the contents of /wp-content/plugins and /wp-content/themes are setup as read-only directories, and end-users can't install or update plugins from within the wp-admin, the hook upgrader_process_complete will never run for either plugin install or update on the VIP Go platform.
When a VIP customer installs a plugin (via the GitHub repo) with a function tied to that hook, their development team has to reverse engineer the plugin's code to execute that functionality.
This is not entirely unique to VIP, but only impacts WordPress environments where the /wp-content/plugins and /wp-content/themes are read-only, so it may not be a good fit for the WordPress Coding Standards.
Describe the solution you'd like
A sniff that will flag places where the upgrader_process_complete hook is used. Preferably as an error because it is functionality that will not work.
What code should be reported as a violation?
add_action( 'upgrader_process_complete', 'my_upgrade_function',10, 2);
What code should not be reported as a violation?
I'm unaware of cases where this might be detected as a false positive.
Additional context
None applicable
- Lenguaje dominante
- PHP
- Estrellas
- 261
- Forks
- 44
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
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 Automattic/VIP-Coding-Standards
-
AlwaysReturnInFilter: isInsideIfConditonal() guards the conditions array after reading itPosiblemente ocupada @tomjn la tomó hace 3 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
-
Bug: PreGetPosts warns when the early is_main_query() return is not the first statement in its ifPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 76/100
-
Suppress filters in get_posts false positivePosiblemente ocupada @tomjn la tomó hace 4 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 48/100
-
Breaking Change Type: Maintenance
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
Automattic/VIP-Coding-Standards#849 · 1 comentario ·
Todos los issues de Automattic/VIP-Coding-Standards
Issues similares
-
Talk Review
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
socallinuxexpo/scale-drupal#351 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
code4romania/cpc#47 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
mautic/api-library#351 · 1 comentario ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
nunomaduro/collision#371 ·
-
Lead create/update: a product row without a "product_id" key passes LeadForm validation and fails in the database (500)Posiblemente ocupada @Arslan-TR la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
krayin/laravel-crm#2681 ·
Los mantenedores suelen responder en 2 días