Page 3 of 4

Re: serendipity_loadGlobalThemeOptions und Constants

Posted: Sat Aug 20, 2011 7:40 pm
by yellowled
Gesetzt den Fall, dass ich diesen Patch manuell korrekt angewendet habe, tut sich da überhaupt nichts.

YL

Re: serendipity_loadGlobalThemeOptions und Constants

Posted: Sun Aug 21, 2011 10:57 am
by 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.

Re: serendipity_loadGlobalThemeOptions und Constants

Posted: Sun Aug 21, 2011 3:22 pm
by yellowled
Timbalu wrote: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.
Timbalu wrote:Ist in deinem template_config array amount enthalten?
Keine Ahnung, wie finde ich das heraus?

YL

Re: serendipity_loadGlobalThemeOptions und Constants

Posted: Sun Aug 21, 2011 5:24 pm
by Timbalu
yellowled wrote:Keine Ahnung, wie finde ich das heraus?
reinschauen! :wink: (config.inc.php) Hast du überhaupt eines? Ich war bisher immer davon ausgegangen du würdest bp verwenden.

Re: serendipity_loadGlobalThemeOptions und Constants

Posted: Sun Aug 21, 2011 5:41 pm
by yellowled
Timbalu wrote:reinschauen! :wink: (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. :wink:

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

Re: serendipity_loadGlobalThemeOptions und Constants

Posted: Sun Aug 21, 2011 5:56 pm
by 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?

Re: serendipity_loadGlobalThemeOptions und Constants

Posted: Sun Aug 21, 2011 6:17 pm
by yellowled
Timbalu wrote: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

Re: serendipity_loadGlobalThemeOptions und Constants

Posted: Mon Aug 22, 2011 10:53 am
by 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?

Re: serendipity_loadGlobalThemeOptions und Constants

Posted: Wed Sep 07, 2011 10:33 pm
by 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

Re: serendipity_loadGlobalThemeOptions und Constants

Posted: Thu Sep 08, 2011 9:45 am
by 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. :P

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.

Re: serendipity_loadGlobalThemeOptions und Constants

Posted: Thu Sep 08, 2011 1:39 pm
by yellowled
Timbalu wrote: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

Re: serendipity_loadGlobalThemeOptions und Constants

Posted: Fri Sep 09, 2011 11:26 am
by 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

Re: serendipity_loadGlobalThemeOptions und Constants

Posted: Wed Sep 14, 2011 11:45 pm
by yellowled
garvinhicking wrote: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. :wink:

YL

Re: serendipity_loadGlobalThemeOptions und Constants

Posted: Thu Sep 15, 2011 3:37 pm
by 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

Re: serendipity_loadGlobalThemeOptions und Constants

Posted: Thu Sep 15, 2011 5:33 pm
by yellowled
garvinhicking wrote: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.
garvinhicking wrote:Und natürlich versuche ich gerne Ende Oktober auch ein Release-Mumbling unterzukriegen.
Klasse. Wir versuchen auch, das bequem vorzubereiten.

YL