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

Bug: class-acf-field-icon_picker.php calls wp_get_attachment_image_url() with string Dashicon slug, causing fatal TypeError on PHP 8+

Abierto Apto para principiantes
  • ¿Algo no está bien?
#1,032 1 comentario 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
86/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
php, wordpress
Área
backend

Línea de trabajo

Empieza en includes/fields/class-acf-field-icon_picker.php, alrededor del bucle de la pestaña media_library, e inspecciona cómo se construye la URL de vista previa. Reproduce el problema mediante wp-admin/post-new.php?post_type=acf-post-type con un valor de Dashicon y el plugin mencionado que enlaza filtros. Se considera terminado cuando los valores de Dashicon ya no provocan el TypeError de PHP 8+ y las vistas previas de media-library siguen funcionando.

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

Descripción

Title:
Bug: class-acf-field-icon_picker.php calls wp_get_attachment_image_url() with string Dashicon slug, causing fatal TypeError on PHP 8+

ACF Version: 6.3.x / 6.4.x (and latest)
WordPress Version: 6.7+ / 7.x
PHP Version: 8.2 / 8.3 / 8.4

Description

In includes/fields/class-acf-field-icon_picker.php, when rendering the tabs for the Icon Picker field (used on post types, taxonomies, and custom fields), ACF executes wp_get_attachment_image_url() unconditionally inside the media_library tab loop, even when the field's active value is a Dashicon string (such as 'dashicons-admin-post').

Because $field['value']['value'] is a string and not an attachment ID, WordPress passes this string through wp_get_attachment_image_src. Any plugin hooking into this filter with strict type hints (such as WP Offload Media 3.4+) immediately crashes with an Uncaught TypeError: Argument must be of type int, string given.

Steps to Reproduce
  1. Run a WordPress site on PHP 8.2+ with ACF and any plugin that hooks into wp_get_attachment_image_src expecting an integer (e.g., WP Offload Media 3.4+).
  2. Navigate to wp-admin/post-new.php?post_type=acf-post-type (ACF > Post Types > Add New).
  3. The page fails with a 500 Fatal Error (TypeError).
Root Cause

In includes/fields/class-acf-field-icon_picker.php (around lines 215–220):

<div class="acf-icon-picker-media-library-preview-img" style="<?php echo esc_attr( 'media_library' !== $field['value']['type'] ? 'display: none;' : '' ); ?>">
    <?php
        $img_url = wp_get_attachment_image_url( $field['value']['value'], 'thumbnail' );
    ?>
    <img src="<?php echo esc_url( $img_url ); ?>" alt="<?php esc_attr_e( 'The currently selected image preview', 'acf' ); ?>" />
</div>

The template attempts to hide the preview using CSS (display: none;), but PHP still evaluates wp_get_attachment_image_url( 'dashicons-admin-post', 'thumbnail' ).

Suggested Fix

Wrap the call so wp_get_attachment_image_url() is only invoked if the field type is actually media_library and the value is numeric:

<div class="acf-icon-picker-media-library-preview-img" style="<?php echo esc_attr( 'media_library' !== $field['value']['type'] ? 'display: none;' : '' ); ?>">
    <?php
        $img_url = '';
        if ( 'media_library' === $field['value']['type'] && is_numeric( $field['value']['value'] ) ) {
            $img_url = wp_get_attachment_image_url( (int) $field['value']['value'], 'thumbnail' );
        }
    ?>
    <img src="<?php echo esc_url( $img_url ); ?>" alt="<?php esc_attr_e( 'The currently selected image preview', 'acf' ); ?>" />
</div>
Lenguaje dominante
PHP
Estrellas
946
Forks
197
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

  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 AdvancedCustomFields/acf

Todos los issues de AdvancedCustomFields/acf

Issues similares

Más issues de PHP

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.