Conditional Tags mit eval prüfen

Was genau ist denn $cond? Du gibst ja die Variable $cond als Return zurück, aber zumindest in deinem gezeigten Code wird diese Variable nirgendwo definiert, oder?

Edit:
Gib doch an der Stelle einfach mal einen String als Test zurück.

Zitat von Azrael_0815

Nein, $cond soll nur als 1 zurückgegeben werden. mittlerweile habe ich eine Antwort, allerdings bekomme ich immer is_home als Antwort, auch auf Page 2.
[ATTACH=CONFIG]18014[/ATTACH]

Die Beispielseite sollte eigentlich bei is_page(2) wahr liefern. Keine Ahnung, was eval da tut:

[ATTACH=CONFIG]18015[/ATTACH]

Bei is_category(1) stimmt es komischerweise.

Ich verstehe an deinem Code mehrere Dinge nicht. Aber es ist natürlich auch schwer, etwas dazu zu sagen, wenn man nur den kurzen Ausschnitt und kaum Kontext kennt.

Müsste die untere Funktion nicht sinnvollerweise einen Boolean zurückgeben? Im Moment gibt sie ja die Bedingung bzw. das Conditional Tag zurück. Oder nichts. Ist das so gewollt?

Warum gibt es oben den "break"? Es können ja auch mehrere Bedingungen zutreffen...

Mach doch mal in der oberen Funktionen einen var_dump von $logic um zu sehen, ob $logic überhaupt einen Wert enthält, den du erwartest. Weil in deinem vorherigen Beitrag schreibst du ja, dass 1 zurückgegeben werden soll. Das kann ich mir aber schwer vorstellen, dass das so ist.

Übrigens:
Jenachdem, wo $conditional herkommt unbedingt den Hinweis von b3317133 in diesem Thread berücksichtigen:

http://forum.wpde.org/plugins-und-wi…html#post740469

Ggf. ist es absolut notwendig, zu prüfen, was da in $conditional enthalten ist und im Zweifelsfall die Ausführung abbrechen.

Eval ist evil

Die untere Funktion gibt keinen Boolean zurück. Sondern entweder einen String ($conditional) oder null.

Ich würde erstmal sicherstellen, dass die untere Funktion wirklich korrekt ist. Für mich sieht das aktuell nicht so aus. Ich verstehe auch nicht, wie sie true zurückgeben soll, wenn dein Rückgabewert ein String ist. Das was du willst bzw. beschreibst, würde dann stimmen, wenn dein Rückgabewert $logic wäre. Aber du gibst ja $conditional zurück. Und selbst dann würde ich lieber ein zusätzliches else ergänzen und wirklich false zurückgeben. True oder Null ist irgendwie ... Naja.

Ansonsten kann man das generell schon mit eval() machen. Dann sollte man nur sicher stellen, dass $conditional nur die Conditional Tags enthalten darf. Wenn $conditional etwas anderes enthält, sollte man die Ausführung sicherheitshalber abbrechen.

Ich persönlich würde den User Optionen (z.B. in Form von Checkboxen) zur Verfügung stellen, wo ich die möglichen Bedingungen vorgeben und dann die Conditional Tags im PHP Code prüfen. So kann der User keinen "Quatsch" eingeben und ich bin sicher, dass ich alle Fälle auch sauber prüfen kann. Zudem halte ich die Usability für besser.

Was genau verstehst Du unter is_page(2) ? Die Seite mit der id=2 oder die zweite Seite einer Auflistung bzw. "Pagination"?