PHP 8.5: "Using null as an array offset" deprecation in ACF_Data::is() when rendering a block with empty data (local-meta.php is_request)
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 68/100
Línea de trabajo
Empieza por includes/local-meta.php y ACF_Local_Meta::is_request() y sigue después la ruta de llamada notificada hasta includes/class-acf-data.php:159 y las funciones relacionadas con campos locales. Reproduce el problema con PHP 8.5 y un bloque ACF con datos vacíos; verifica que la deprecación ya no se produce al renderizar en el front end ni en el editor.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Describe the bug
On PHP 8.5, rendering an ACF block that has no saved field data (e.g. a block with no fields of its own that only wraps InnerBlocks) triggers:
Deprecated: Using null as an array offset is deprecated, use an empty string instead in wp-content/plugins/advanced-custom-fields-pro/includes/class-acf-data.php on line 159
It fires once per such block, on the front end and in the block editor.
Cause
ACF_Local_Meta::is_request() calls acf_is_field_key( key( $meta ) ). When the block's data is empty, key( [] ) returns null, which goes through the acf/is_field_key filter → _acf_apply_is_local_field_key() → acf_is_local_field_key( null ) → ACF_Data::is( null ) → isset( $this->data[ null ] ).
Call stack:
includes/class-acf-data.php:159 ACF_Data::is()
includes/local-fields.php:588 acf_is_local_field_key()
includes/local-fields.php:679 _acf_apply_is_local_field_key()
includes/acf-field-functions.php:209 acf_is_field_key() (apply_filters 'acf/is_field_key')
includes/local-meta.php:90 ACF_Local_Meta::is_request()
includes/local-meta.php:62 ACF_Local_Meta::add()
includes/local-meta.php:249 acf_setup_meta()
pro/blocks.php:1131 / 1285 acf_rendered_block_v3() / acf_render_block()
To Reproduce
- PHP 8.5, ACF PRO 6.8.10, WordPress 7.1.3,
WP_DEBUGon. - Register an ACF block with no field group (or no saved values) whose template renders
<InnerBlocks />. - Insert it into a post; the saved markup has no
dataattribute, e.g.<!-- wp:my-plugin/feature {"name":"my-plugin/feature","mode":"preview"} -->. - View the post or open it in the editor: the deprecation above is emitted.
Expected behavior
No deprecation. E.g. in is_request(), return false when $meta is empty (or pass key( $meta ) ?? ''); alternatively make ACF_Data::is() / acf_is_local_field_key() null-safe.
Workaround
Swap ACF's filter callback for a null-safe one (behaviour is unchanged, since PHP already treated a null key as ''):
if ( remove_filter( 'acf/is_field_key', '_acf_apply_is_local_field_key', 20 ) ) {
add_filter( 'acf/is_field_key', fn( $bool, $id ) => acf_is_local_field_key( $id ?? '' ), 20, 2 );
}
Version Information
- WordPress 7.1.3
- PHP 8.5.7
- ACF PRO 6.8.10
Related: #1034 (same PHP 8.5 deprecation, different code path in pro/options-page.php).
- Lenguaje dominante
- PHP
- Estrellas
- 947
- Forks
- 197
- Métricas de merge de PR
- Sin PR fusionados en 30 d
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 AdvancedCustomFields/acf
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
AdvancedCustomFields/acf#1041 · 1 comentario ·
-
True / False ACF control bug on RTL layoutsPosiblemente ocupada @elicohenator la tomó hace 19 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
AdvancedCustomFields/acf#1038 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
AdvancedCustomFields/acf#1034 · 2 comentarios ·
-
Bug: class-acf-field-icon_picker.php calls wp_get_attachment_image_url() with string Dashicon slug, causing fatal TypeError on PHP 8+Posiblemente ocupada @antwonw la tomó hace 30 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
AdvancedCustomFields/acf#1032 · 1 comentario ·
-
Drop trap, acf_after_titlePosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
AdvancedCustomFields/acf#1029 · 1 comentario ·
Todos los issues de AdvancedCustomFields/acf
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
hawk-digital-environments/HAWKI#443 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 78/100
crazy-goat/rabbit-stream#799 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
-
Code Quality
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Automattic/safe-publish#708 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
WordPress/two-factor#1022 · 1 comentario ·
Los mantenedores suelen responder en 1 día