serendipity_loadGlobalThemeOptions und Constants

[yellowled]

Gesetzt den Fall, dass ich diesen Patch manuell korrekt angewendet habe, tut sich da überhaupt nichts.

YL

[Timbalu]

Ja, aber auch im svn/trunk ändert sich mal was, insofern ist das Datum entscheidend...

Ist in deinem template_config array amount enthalten?
Ist deine options tabelle leer (t_global & t_"template")?
Sonst einmal komplett leeren.

[yellowled]

Ja, aber auch im svn/trunk ändert sich mal was, insofern ist das Datum entscheidend...

Mein dev-Blog läuft immer mit dem letzten Stand von svn/trunk und wird stetig aktualisiert.

Ist in deinem template_config array amount enthalten?

Keine Ahnung, wie finde ich das heraus?

YL

[Timbalu]

Keine Ahnung, wie finde ich das heraus?

reinschauen! (config.inc.php) Hast du überhaupt eines? Ich war bisher immer davon ausgegangen du würdest bp verwenden.

[yellowled]

reinschauen! (config.inc.php) Hast du überhaupt eines? Ich war bisher immer davon ausgegangen du würdest bp verwenden.

Man sollte nie von sich auf andere schließen.

Ich verwende hier in einem nicht auf BP basierenden Template den vorgegebenen Code für die globale Navigation in der config.inc.php, also

Code: Select all

$template_global_config = array('navigation' => true);
$template_loaded_config = serendipity_loadThemeOptions($template_config, $serendipity['smarty_vars']['template_option']);
serendipity_loadGlobalThemeOptions($template_config, $template_loaded_config, $template_global_config);

Willst Du mir sagen, ich muss die amount-Variable zusätzlich in die config.inc.php einbauen? Das kann so nicht beabsichtigt sein und ist laut http://board.s9y.org/viewtopic.php?p=10419813 auch zumindest ursprünglich nicht notwendig gewesen.

YL

[Timbalu]

Nee, ich wollte nur wissen ob sie eingebaut ist und ob es überhaupt ein Konfigurationsarray gibt. Wenn nicht, könnte das die Ursache des vorläufig vorliegenden Problems sein.
Also nochmal: Verwendest du ein $template_config array?

[yellowled]

Verwendest du ein $template_config array?

Ja, sicher – Garvin hatte im ursprünglichen Thread erklärt, dass das Array mit mindestens einem (zur Not Dummy-) Element vorhanden sein muss. Das sieht hier so aus:

Code: Select all

<?php
if (IN_serendipity !== true) { die ("Don't hack!"); }

$probelang = dirname(__FILE__) . '/' . $serendipity['charset'] . 'lang_' . $serendipity['lang'] . '.inc.php';
if (file_exists($probelang)) { include $probelang; }
include dirname(__FILE__) . '/lang_en.inc.php';

$serendipity['smarty']->assign(array('currpage'=> "http://".$_SERVER['HTTP_HOST'].$_SERVER['REQUEST_URI']));
$serendipity['smarty']->assign(array('currpage2'=> $_SERVER['REQUEST_URI']));

function serendipity_plugin_api_event_hook($event, &$bag, &$eventData, $addData = null) {
    global $serendipity;    
    switch($event) { case 'frontend_footer': echo '<!--PLUGIN API-->'; }
    return true;
}

$template_config = array(
    array(
        'var'           => 'date_format',
        'name'          => GENERAL_PLUGIN_DATEFORMAT . " (http://php.net/strftime)",
        'type'          => 'select',
        'default'       => DATE_FORMAT_ENTRY,
        'select_values' => array(DATE_FORMAT_ENTRY => DATE_FORMAT_ENTRY,
                                 '%a, %e. %B %Y' => '%a, %e. %B %Y',
                                 '%d-%m-%y' => '%d-%m-%y',
                                 '%m-%d-%y' => '%m-%d-%y',
                                 '%a %d-%m-%y' => '%a %d-%m-%y',
                                 '%a %m-%d-%y' => '%a %m-%d-%y',
                                 '%b %d' => '%b %d',
                                 "%b %d '%y" => "%b %d '%y")
    ),
);

$template_global_config = array('navigation' => true);
$template_loaded_config = serendipity_loadThemeOptions($template_config, $serendipity['smarty_vars']['template_option']);
serendipity_loadGlobalThemeOptions($template_config, $template_loaded_config, $template_global_config);

Das ist die komplette config.inc.php.

YL

[Timbalu]

Ich wollte nur sichergehen. Und dein option table ist voll oder leer?
Besonders $template_loaded_config['amount'] ist dort wichtig.
wie genau sieht jetzt deine function serendipity_loadGlobalThemeOptions() in der functions_config.inc.php aus?

[yellowled]

Okay, sorry – die letzte Message hier war bei mir irgendwie untergegangen.

Neue Erkenntnis: Ich habe gerade ein neues Blog mit 1.7-alpha1 (svn-Snapshot von heute) aufgesetzt und ein gewisses Template installiert, dass die globale Navi nutzt. Nach dem erstmaligen Aktivieren des Templates habe ich ein Feld für die Anzahl der Items in der Navigation, dieses „verschwindet“ erst nach dem ersten Speichern der theme options.

Hoffe, das hilft irgendwie weiter – leider kann ich aufgrund des extrem dämlichen Rechtekonzeptes im Webspace dort nichts testen, ohne mir mit überflüssiger Arbeit beim Hin- und Herschalten der Rechte die Nerven zu brechen. Ich kann aber gerne lokal testen.

YL

[Timbalu]

Ich hatte mich schon etwas gewundert.... glaube aber das funktioniert so...:

Das erste Mal ist das Feld da (weil options leer ist) und schreibt anscheinend einen Amount Wert (5) in die options Tabelle (nicht aber die (5) nav links etc). Daraufhin kann aber kein request die globale Geschichte erreichen, weil der zuletzt genannte Patch (der die Funktion fast auf den urspünglichen Ausgangszustand zurücksetzt) noch nicht darinnen ist und im Moment ein gesetzter option table amount Wert den Anhang an das Array verhindert, der wiederum nötig für dessen Anzeige ist.

Insgesamt hängt dieses Verhalten aber mit dem verspätetem Schreiben und der doppelten Reload Geschichte zusammen. In serendipity_loadGlobalThemeOptions() muss das dann so aussehen

Code: Select all

        $navlinks = array();

        $conf_amount = array(
                'var'           => 'amount',
                'name'          => NAVLINK_AMOUNT,
                'type'          => 'string',
                'default'       => '5',
                'scope'         => 'global'
        );

        if (!isset($template_loaded_config['amount']) || empty($template_loaded_config['amount'])) {
            $template_config[] = $conf_amount;
            $template_loaded_config['amount'] = $conf_amount['default'];
        }
    
        for ($i = 0; $i < $template_loaded_config['amount']; $i++) {

Wie gesagt hängt die Patch zusammen mit der Anpassung der core lang files (amount, link, title) im Moment bei Garvin, der ziemlich belastet scheint, aber wohl der Sache erst noch auf den Grund gehen wollte, bevor er das für 1.6 und 1.7 freigibt.

@Garvin
Ich habe die serendipity_loadThemeOptions() ebenfalls gepatcht und serendipity_db_bool für template config vars eingebaut. Wenn du willst, baue ich das alles in einem Patch zusammen.

[yellowled]

Wie gesagt hängt die Patch zusammen mit der Anpassung der core lang files (amount, link, title) im Moment bei Garvin, der ziemlich belastet scheint, aber wohl der Sache erst noch auf den Grund gehen wollte, bevor er das für 1.6 und 1.7 freigibt.

Kinners, ich verlass mich jetzt einfach darauf, dass Ihr das im Auge behaltet und löst, und bastele weiter an 2k11, okay? Prima.

YL

[garvinhicking]

Hi!

Ich habe im Oktober Urlaub (die letzten 2 Wochen), da werde ich die 1.6 pünktlich bis zu meinem Geburtstag releasen und mir auch vorher diese Sache gezielt ansehen. Dann werde ich sicher noch Fragen haben und hoffe dass wir die kurzfristig geklärt kriegen.

Grüße,
Garvin

[yellowled]

Ich habe im Oktober Urlaub (die letzten 2 Wochen), da werde ich die 1.6 pünktlich bis zu meinem Geburtstag releasen und mir auch vorher diese Sache gezielt ansehen. Dann werde ich sicher noch Fragen haben und hoffe dass wir die kurzfristig geklärt kriegen.

Cool. (Planst Du auch noch einen Abend mit Robert und mir ein? Das wäre sensationell.)

Andere Sache: Veit merkte eben (beim Testen von 2k11) richtig an, dass es keine Option gibt, die Kernnavigation abzuschalten. Außerdem sollte das ganze idealerweise eine Art Sicherung haben, dass kein Link generiert wird, wenn kein gültiges Linkziel (href) angegeben wird. Beides finde ich ziemlich sinnvolle Ergänzungen.

Alternativ kannst Du mir auch gern erklären, wie ich dieses Verhalten templateseitig bastele.

YL

[garvinhicking]

Hi!

Das geht garantiert indem man die Smarty-Variablen entsprechend abfragt. Wie kann ich aber leider derzeit nicht vorbereiten.

Und natürlich versuche ich gerne Ende Oktober auch ein Release-Mumbling unterzukriegen.

Viele Grüße,
Garvin

[yellowled]

Das geht garantiert indem man die Smarty-Variablen entsprechend abfragt. Wie kann ich aber leider derzeit nicht vorbereiten.

Hatte ich nicht erwartet. Ich weiß auch nicht, ob das für John-Doe-User reicht bzw. derzeit funktionieren würde – aber es sollte reichen, die Anzahl der Links auf "0" zu setzen, um die Navi abzuklemmen.

Und natürlich versuche ich gerne Ende Oktober auch ein Release-Mumbling unterzukriegen.

Klasse. Wir versuchen auch, das bequem vorzubereiten.

YL