HTML Formatierung per IF Anweisung

[BlackShadow]

Moin Leute,

ich möchte in Wordpress und mit dem Plugin ACF (advanced custom fields) ein neues Template schreiben.

Hierzu brauche ich bei einem Wort eine farbliche Kennzeichnung anhand des Status... Das Dropdown-Feld beinhaltet 3 Werte (gruen, gelb, rot)

Leider läuft mein derzeitiger Code in den 500 error. Vermutlich stimmt irgendwo die Symantik nicht...

<?php $field = get_field('status');
    if( $field['value'] == 'gruen' ) {
    echo '<span style="color:green;">', $label , ' </span>'; }
    else ( $field['value'] != 'gruen' ) {
    echo '<span style="color:red;">', $label , ' </span>'; } ?>

Die Variablen stimmen soweit... Status heißt mein Feld, welches die oben genannten drei Werte beinhaltet.

Kann mir jemand auf die Sprünge helfen?

Hier noch die ACF Doku: https://www.advancedcustomfields.com...ces/get_field/

Vielen Dank.

[hausl]

Leider läuft mein derzeitiger Code in den 500 error.

Error Log anschauen, oder Error Reporting hochdrehen, da steht genaueres.

Es geht auch einfacher.. Nimm ein Array. Und es ist besser CSS Klassen zu verwenden statt inline CSS.

[chorn]

Bei Error 500 solltest du erstmal ins error.log deines Webservers gucken.

[protestix]

$field = get_field('status') 

Das ist doch unnötig, warum kopierst du den Wert in eine andere Variable?

$label kommt wo her?
Warum ist der Status eine Farbe und keine beschreibende Eigenschaft?

echo '<span style="color:green;">', $label , ' </span>'; } 

Bist du sicher das du hier echo verwenden darfst?
Warum inlinestyles und keine CSS Klasse?

Die AFC Doku habe ich mir gar nicht erst durchgelesen, das ist deine Aufgabe, wir sind hier bei PHP und weisen dich auf PHP Fehler hin, für Wordpress und deren Plugins fragst du den Herstellen oder gehst in das Wordpress Forum.

[BlackShadow]

Okay ich schau mal rein, aber das heißt auch das mein Code so falsch gar nicht ist?

[jonas3344]

Die Kommas beim echo könnten den Fehler verursachen.

Richtig gehts so: https://www.php.net/manual/de/langua...ors.string.php

[hausl]

Wie gesagt.. Inline CSS ist nicht ideal, nutze doch css-Klassen dafür.

Rein auf deine if-Wurst bezogen .. Nutze Arrays oder warum nicht gleich:

printf(
    '<span style="color:%s;">%s</span>',
    htmlspecialchars($field['value']),
    htmlspecialchars($label)
); 

?

[BlackShadow]

Hier ist der Fehler


Fatal error: Uncaught Error: Call to undefined function mysql_connect() in /home/.../wp-includes/wp-db.php:1564 Stack trace: #0 /home/.../wp-includes/wp-db.php(592): wpdb->db_connect() #1 /home/.../wp-includes/load.php(404): wpdb->__construct('dbname', 'dbpass', 'dbuser', 'localhost') #2 /home/.../wp-settings.php(106): require_wp_db() #3 /home/.../wp-config.php(95): require_once('/home/...') #4 /home/.../wp-load.php(37): require_once('/home/...') #5 /home/.../wp-blog-header.php(13): require_once('/home/...') #6 /home/.../index.php(17): require('/home/...') #7 {main} thrown in /home/.../wp-includes/wp-db.php on line 1564


Für mich ist das aber wieder alles andere als hilfreich, denn er sagt ja nun das ein Fehler in der DB Verbindung bzw. in der wp-db.php Datei steckt aber das ist so nicht richtig. Die Dateien und auch die Verbindungsdaten sind korrekt. Der Fehler entsteht ja durch mein oben geschriebenes Konstrukt. Ohne das läuft alles wunderbar...

[protestix]

Die Funktion im Handbuch von PHP in das Suchfeld kopieren und nachlesen, ich habe das schon mal für dich gemacht klick und lies
https://www.php.net/manual/en/functi...ql-connect.php

[BlackShadow]

protestix

Danke. Weil ich es nicht besser weiß ^^ Aber ob inline oder klasse fand ich erstmal egal... Ich kann natürlich auch class="xy" machen. Ich wollte es erstmal zum laufen bringen und danach optimieren. Label soll meine Feldname sein... auch den sollte ACF mitbringen so dass ich ihn hier abrufen kann.

Vielleicht noch wichtig zu wissen: gruen ist nur mein interner Wert... ACF erlaubt es einen Wert und eine Bezeichnung zu vergeben z.B. gruen : Funktioniert oder rot : Defekt (die Bezeichnung sollte dann mein Label sein)

[BlackShadow]

protestix

Ich weiß und sorry das ich gerade Dich wieder nerve
Aber wie gesagt der Fehler hat mit meinem Code oben eh nicht viel zu tun. Ich habs gelesen... aber das hilft mir bei der If-Abfrage nicht wirklich.

[BlackShadow]

Code sieht nun so aus:

<?php $field = get_field('status');
    if ( $field['value'] == 'gruen' ) {
    echo '<span class="valid">'. $label . ' </span>'; }
    else ( $field['value'] != 'gruen' ) {
    echo '<span class="invalid">'. $label . ' </span>'; } ?>

CSS

.valid {color: green}
.invalid {color:red}

[protestix]

kann man verkürzen zu

if ( get_field('status') === 'gruen' ) {
    echo '<span class="valid">' . $label . ' </span>';
}else {
    echo '<span class="invalid">' . $label . ' </span>';
} 

[BlackShadow]

Okay das hat schon mal fast funktioniert...

Ich habs noch mal ein wenig umgeschrieben... und die Punkte wieder durch die Kommas ersetzt da mit den Punkten der HTML Code hinter meinem Label ausgegeben wurde...

if ( get_field('status') == 'gruen' ) {
    echo '<span class="valid">' , the_field('status') , ' </span>'; }
    else {
    echo '<span class="invalid">' , the_field('status') , ' </span>';
    } 

the_field ist um das Feld direkt abzufragen
Und get_field um dessen Werte (gruen, rot, gelb) abzufragen...

Mein Problem ist nun noch das der Feldwert nicht korrekt berücksichtigt wird... dadurch ist es natürlich immer falsch und somit Klasse "invalid". In dem Falle müsste aber mein Ergebnis true sein...

[hausl]

Dann schau mal rein:

var_dump( get_field('status'), the_field('status') );