Salida de un logotipo de la WordPress Personalizer -- beginner campo con php campo con wordpress camp codereview Relacionados El problema

Outputting a logo from the Wordpress Customiser


1
vote

problema

Español

Me las arreglé para resolver un pequeño problema, me había hecho leyendo y modificando.
Mi código funciona, pero como novato, me gustaría preguntar si podría aconsejarme de una manera más limpia de lograr lo mismo.

Las cosas se ven un poco desordenadas para mí, así que estoy seguro de que probablemente haya una mejor manera de hacer esto.

Este código simplemente emite un logotipo de WordPress PersonSiser.

Cualquier consejo o dirección será muy apreciado.

  <?php              $custom_logo_id = get_theme_mod('custom_logo');              $logo = wp_get_attachment_image_src($custom_logo_id, 'full');              if (has_custom_logo()) {                  echo '<img src="' . esc_url($logo[0]) . '">';              } else { ?>                  <img src='<?php echo get_template_directory_uri(); ?>/img/logo.png'> <?php }   ?>   
Original en ingles

I managed to solve a little problem I had myself by reading and modifying.
My code works, but as a novice I would like to ask if you could advise me of a cleaner way to achieve the same thing.

Things look a bit messy to me, so I'm sure there is probably a better way to do this.

This code simply outputs a logo from the Wordpress Customiser.

Any advice or direction will be greatly appreciated.

<?php              $custom_logo_id = get_theme_mod('custom_logo');              $logo = wp_get_attachment_image_src($custom_logo_id, 'full');              if (has_custom_logo()) {                  echo '<img src="' . esc_url($logo[0]) . '">';              } else { ?>                  <img src='<?php echo get_template_directory_uri(); ?>/img/logo.png'> <?php }   ?> 
        

Lista de respuestas

2
 
vote

Aquí están mis sugerencias:

No hay un punto de "recuperar" la URL de la imagen si 99887776655443354 devuelve Falso.

En su lugar, definiría slider5 con un valor predeterminado, entonces, si 99887776655443356 es 99887776655443357 , simplemente reescribiría su valor. < / p>

  slider8  

Estoy usando slider9 MÉTODO , eso

Rebobina el puntero interno de la matriz al primer elemento y devuelve el Valor del primer elemento de matriz.

Puede convertir esto a un método y usarlo en varios lugares.

 

Here are my suggestions:

There is no point of "fetching" the image url if has_custom_logo returns false.

Instead, I would define $logo with a default value, then, if has_custom_logo is true, I would just rewrite it's value.

$logo = get_template_directory_uri() . '/img/logo.png';  if(has_custom_logo()){   $logos = wp_get_attachment_image_src(get_theme_mod('custom_logo'), 'full');   $logo  = reset($logos); }  echo sprintf('<img src="%s" />', esc_url($logo)); 

I'm using reset method, it

rewinds array's internal pointer to the first element and returns the value of the first array element.

You can convert this to a method and use it in multiple places.

 
 
 
 

Relacionados problema

2  Visualización de campos en una barra lateral de una plantilla de página personalizada  ( Displaying fields in a sidebar of a custom page template ) 
Estoy comenzando con la programación de PHP y apreciaría una crítica en mi código y cómo podría hacerlo más eficiente, en lugar de repetir las declaraciones ...

1  Script de cuatro líneas para hacer una copia de seguridad inmediata de DB y Document Root  ( Four lines script to make an immediate db and document root zip backup ) 
¿Qué piensas de mi script para hacer una copia de seguridad inmediata de DB y DOCUMENTE ROOT ROOT? Cualquier falla que puedas encontrar. Por favor, infórmeme ...

5  Mostrando todas las publicaciones individuales  ( Displaying all single posts ) 
Creo que hay algunos problemas de sangría confusa y extraños en este archivo de WordPress (single.php). ¿Alguna sugerencia sobre la mejora de la sangría y la ...

6  ¿Modificar los bucles personalizados para mejorar la legibilidad y la eficiencia para WordPress?  ( Modify custom loops to improve readability and efficiency for wordpress ) 
El siguiente código recupere los tipos de publicaciones personalizadas con su taxonomía personalizada. Solo soy un principiante en PHP y me gustaría saber c...

1  Botón para seguir / dejar de seguir un usuario  ( Button to follow unfollow a user ) 
Como principiante en la codificación, acabo de escribir este código. El código debe agregar un formulario con un botón a cada perfil de usuario, llamado Sig...

3  Cree un sitio de sitio de WordPress con los datos de la aplicación NGINX automáticamente  ( Create a wordpress site data with nginx app data automatically ) 
El siguiente script está dirigido a Ubuntu >=16.04 LEMP entornos con PHP-FPM 9988777665544335 , MySQL, WP-CLI, CertBot y WordPress Apps. El script Obteng...

1  Conversión CSV a MySQL con PHP en mi complemento de WordPress personalizado  ( Converting csv to mysql with php in my custom wordpress plugin ) 
Tengo una función que convierte un archivo de CSV a MYSQL El archivo podría ser de 5,000 filas de largo hasta 90,000 filas de largo. Sobre la base de qu...

4  Añadir campos de perfil de usuario  ( Add user profile fields ) 
Tengo esta función PHP que agrega campos de perfil de usuario personalizados a WordPress. Guarda el campo a través del archivo WOOCOMMERCE FORM-EDIT-MY-CUENTA...

3  Exportación de datos en un widget de WordPress  ( Exporting data in a wordpress widget ) 
Tengo algunas tablas personalizadas en una base de datos de WordPress que necesito para exportar a CSV. Creé un widget de Dashboard para ayudarlo con esto en ...

6  Determinar el valor en el rango, inyectar a HTML que se utilizará en CSS  ( Determine value in range inject to html to be used in css ) 
Quiero mostrar a los usuarios qué paquete están, según cuántos artículos tienen en su carrito. El código funciona, pero me pregunto si algo puede mejorarse. ...




© 2022 respuesta.top Reservados todos los derechos. Centro de preguntas y respuestas reservados todos los derechos