Bugreports für SP 3.36.4-beta1 hier rein!

[falk]

Hallo,

ich habe nun alle Änderungen der letzten Wochen endlich in eine SP Version 3.36.4 gepakt und online gestellt.
Wie immer erstmal auf http://s9y-cms.fadoe.de/plugins/serendi ... taticpages.

Folgende Änderungen sind drin:

Stefan: RSS-Kategorien
Ian: angepasster Link zum Gästebuch

Ich habe das Plugin noch nicht ganz auf Herz und Nieren durchgetestet. Außerdem stehen evtl. noch einige Schönheitskorrekturen an. Unter anderem werde ich die RSS-Kategorien in Kategorien umbenennen. Außerdem fehlt noch ein Template Namens "static-entries.tpl" -> Stefan?

Ich werde die nächsten Tage das alles noch durchtesten. Wer Bugs findet, darf sie hierher schreiben .

Ich hoffe nächste, spätestens übernächste Woche eine stabile Version 3.36.4 zu haben und diese dann als Version 3.37 ins CVS schicken zu können.

Tschau,

Falk

[stm999999999]

ich habe nun alle Änderungen der letzten Wochen endlich in eine SP Version 3.36.4 gepakt und online gestellt.
Wie immer erstmal auf http://s9y-cms.fadoe.de/plugins/serendi ... taticpages.

Der Link lautet korrekt http://s9y-cms.fadoe.de/plugins/serendi ... staticpage

Folgende Änderungen sind drin:

Stefan: RSS-Kategorien
Ian: angepasster Link zum Gästebuch

Das heißt, Deine 3.36.4 basiert noch auf der alten staticpage-Version? Weil, ja zwischenzeitlich einige Änderungen im CVS eingebaut wurden: Die "Brotkrumen"-Navigation siehe http://php-blog.cvs.sourceforge.net/php ... taticpage/

Und: Hast Du für meine Änderungen die beiden Probleme beheben können?

a) Tabelle wurde in meinem Code noch nicht automatisch erstellt beim Installieren
b) Beim Löschen einer Kategorie wurde der Tabelleneintrag nicht gelöscht

siehe http://www.s9y.org/forums/viewtopic.php ... t=15#38136



Und noch eines: Sind die Probleme aus http://www.s9y.org/forums/viewtopic.php?t=7056 auch schon behoben?

Ich habe das Plugin noch nicht ganz auf Herz und Nieren durchgetestet. Außerdem stehen evtl. noch einige Schönheitskorrekturen an. Unter anderem werde ich die RSS-Kategorien in Kategorien umbenennen. Außerdem fehlt noch ein Template Namens "static-entries.tpl" -> Stefan?

no problem:

Code: Select all

<!-- ENTRIES START -->

{if $entries}
{$CONST.STATICPAGE_NEW_HEADLINES}

<ul>
    {foreach from=$entries item="dategroup"}
            {foreach from=$dategroup.entries item="entry"}
			<li class="static-entries">
        		({$dategroup.date|date_format:"%d.%m.%Y"}) <a href="{$entry.link}">{$entry.title|@default:$entry.id}</a>

<!-- if you want to see the first letters of the article
            <div>
                {$entry.body|truncate:160:' ...'}
            </div>
 -->
			</li>
            {/foreach}
   {/foreach}
</ul>

» <a href="{$serendipityBaseURL}{getCategoryLinkByID cid=$staticpage_rss_category_id}
">{$CONST.STATICPAGE_ARTICLE_OVERVIEW}</a><br />

{/if}
<!-- ENTRIES END -->

Ist natürlich nur ein Vorschlag, das kann und sollte man ggf. selber anpassen.

In der lang-Datei des Plugins muß dann noch hinzugefügt werden:

Code: Select all

STATICPAGE_NEW_HEADLINES = "aktuelle Schlagzeilen" / "new headlines"

und

Code: Select all

STATICPAGE_ARTICLE_OVERVIEW = "Artikelübersicht" / "overview" 

Wobei das natürlich nur Vorschläge sind.

Möglicherweise sollte man noch eine static-entries-frontpage.tpl anlegen, wenn man nämlich eine statische Seite als Frontpage verwendet, dann muß der Link unten anders sein:

Code: Select all

» <a href="{$serendipityBaseURL}index.php?frontpage">{$CONST.ARTICLE_OVERVIEW}</a><br />

[falk]

Der Link lautet korrekt http://s9y-cms.fadoe.de/plugins/serendi ... staticpage

Tschuldigung.

Das heißt, Deine 3.36.4 basiert noch auf der alten staticpage-Version? Weil, ja zwischenzeitlich einige Änderungen im CVS eingebaut wurden: Die "Brotkrumen"-Navigation siehe http://php-blog.cvs.sourceforge.net/php ... taticpage/

Ich muss erst mal wieder einen Überblick über die Änderungen bekommen. Diese fleißen noch mit ein. Außerdem hatte ich mal wieder keinen Zugriff auf den CVS-Server.

Und: Hast Du für meine Änderungen die beiden Probleme beheben können?

Bin noch am schaun. Man, dies ist eine BETA!

a) Tabelle wurde in meinem Code noch nicht automatisch erstellt beim Installieren

Done.

b) Beim Löschen einer Kategorie wurde der Tabelleneintrag nicht gelöscht

Noch nicht erledigt.

Und noch eines: Sind die Probleme aus http://www.s9y.org/forums/viewtopic.php?t=7056 auch schon behoben?

Das sind die Momente, wo ich Foren anfange zu hassen. So viele Topics.
Antwort: Ne.

no problem:

Code: Select all

<!-- ENTRIES START -->

{if $entries}
{$CONST.STATICPAGE_NEW_HEADLINES}

<ul>
    {foreach from=$entries item="dategroup"}
            {foreach from=$dategroup.entries item="entry"}
			<li class="static-entries">
        		({$dategroup.date|date_format:"%d.%m.%Y"}) <a href="{$entry.link}">{$entry.title|@default:$entry.id}</a>

<!-- if you want to see the first letters of the article
            <div>
                {$entry.body|truncate:160:' ...'}
            </div>
 -->
			</li>
            {/foreach}
   {/foreach}
</ul>

» <a href="{$serendipityBaseURL}{getCategoryLinkByID cid=$staticpage_rss_category_id}
">{$CONST.STATICPAGE_ARTICLE_OVERVIEW}</a><br />

{/if}
<!-- ENTRIES END -->

Okay, wird eingefügt.

Ist natürlich nur ein Vorschlag, das kann und sollte man ggf. selber anpassen.

In der lang-Datei des Plugins muß dann noch hinzugefügt werden:

Code: Select all

STATICPAGE_NEW_HEADLINES = "aktuelle Schlagzeilen" / "new headlines"

und

Code: Select all

STATICPAGE_ARTICLE_OVERVIEW = "Artikelübersicht" / "overview" 

Wobei das natürlich nur Vorschläge sind.

Wird alles gemacht.

Möglicherweise sollte man noch eine static-entries-frontpage.tpl anlegen, wenn man nämlich eine statische Seite als Frontpage verwendet, dann muß der Link unten anders sein:

Code: Select all

» <a href="{$serendipityBaseURL}index.php?frontpage">{$CONST.ARTICLE_OVERVIEW}</a><br />

Gute Idee.

Danke für die Hinweise.

[stm999999999]

Der Link lautet korrekt http://s9y-cms.fadoe.de/plugins/serendi ... staticpage

Tschuldigung.

Ich helfe ja gerne

Die "Brotkrumen"-Navigation siehe http://php-blog.cvs.sourceforge.net/php ... taticpage/

Ich muss erst mal wieder einen Überblick über die Änderungen bekommen. Diese fleißen noch mit ein. Außerdem hatte ich mal wieder keinen Zugriff auf den CVS-Server.

Das sieht nach mehr Änderungen aus, als es wirklich sind, jede der neuen CVS-Einträge modifiziert nur den neuen Code-Block. Ich habe es testweise in meine staticpage-Version per Hand eingebaut - einfach den aktuellsten Code-Block einfügen und es lief wunderbar.

Und: Hast Du für meine Änderungen die beiden Probleme beheben können?

Bin noch am schaun. Man, dies ist eine BETA!

Tschulligung, ich frag ja nur

Und noch eines: Sind die Probleme aus http://www.s9y.org/forums/viewtopic.php?t=7056 auch schon behoben?

Das sind die Momente, wo ich Foren anfange zu hassen. So viele Topics.
Antwort: Ne.

Dafür bin ich ja dann da: Ich erinnere Dich dran!

[stm999999999]

Äh, aus anderem Anlaß guckte ich gerade in die config.inc.php meines templates rein und fand dort noch folgendes:

Code: Select all

$serendipity['smarty']->register_function('getCategoryLinkByID', 'smarty_getCategoryLinkByID');

function smarty_getCategoryLinkByID ($data, &$smarty) {

	$cat = serendipity_fetchCategoryInfo($data['cid']);
	$result=serendipity_getPermalink($cat,'category');
  return $result;
} ;

genutzt wird das in der static-page.tpl für den Link auf die Artikelübersichtsseite:

Code: Select all

» <a href="{$serendipityBaseURL}{getCategoryLinkByID cid=$staticpage_rss_category_id}
">{$CONST.ARTICLE_OVERVIEW}</a><br />

Ich denke mal, das wird im plugin noch benötigt, oder?

[falk]

Äh, aus anderem Anlaß guckte ich gerade in die config.inc.php meines templates rein und fand dort noch folgendes:

Code: Select all

$serendipity['smarty']->register_function('getCategoryLinkByID', 'smarty_getCategoryLinkByID');

function smarty_getCategoryLinkByID ($data, &$smarty) {

	$cat = serendipity_fetchCategoryInfo($data['cid']);
	$result=serendipity_getPermalink($cat,'category');
  return $result;
} ;

genutzt wird das in der static-page.tpl für den Link auf die Artikelübersichtsseite:

Code: Select all

» <a href="{$serendipityBaseURL}{getCategoryLinkByID cid=$staticpage_rss_category_id}
">{$CONST.ARTICLE_OVERVIEW}</a><br />

Ich denke mal, das wird im plugin noch benötigt, oder?

Ich wollte mich gerade deswegen beschweren . Danke. Ich werde diese Funktion ins Plugin integrieren, müsste dann also aus der config.inc.php wieder verschwinden. Leider habe ich heute keine Zeit mehr für eingehendere Tests. Aber in den Plugincode bei mir kommts schon rein.

[stm999999999]

Wie sieht denn der aktuelle Stand aus?

Was ist in der aktuellen Version auf Deiner Download-Seite drin und was kann / sollte man mal austesten?

[stm999999999]

ich habe nochmal in meine Änderungen reingeschaut:

unter der Liste der Einträge auf der Frontseite kann natürlich kein Link auf die zugehörige Kategorieseite liegen, weil von der Frontpage auf die das allgemeine Archiv verwiesen werden muß.

Deshalb hatte ich ja dafür auch eine eigene static-entries-frontpage.tpl vorgeschlagen. In dieser heißt es nun:

<!-- ENTRIES START -->

{if $entries}
{$CONST.NEW_HEADLINES}

<ul>
{foreach from=$entries item="dategroup"}
{foreach from=$dategroup.entries item="entry"}
<li class="static-entries">
({$dategroup.date|date_format:"%d.%m.%Y"}) <a href="{$entry.link}">{$entry.title|@default:$entry.id}</a>
</li>
{/foreach}
{/foreach}
</ul>

» <a href="{$serendipityArchiveURL}/P1.html">{$CONST.ARTICLE_OVERVIEW}</a><br />

{/if}
<!-- ENTRIES END -->

Und da fehlt dann natürlich (so wie letztens die getCategoryLinkByID) noch die zugehörige Funktion, die bislang in meiner config.inc.php rumliegt:

Code: Select all

$serendipity['smarty']->assign('serendipityArchiveURL', getArchiveURL());

function getArchiveURL() {
	global $serendipity;
	$uriArguments = $serendipity['uriArguments'];
	return serendipity_rewriteURL(implode('/', $uriArguments));
};