anmerkung: snapshot contenido-cvs-2004-01-23.tar

[emergence]

irgendwie hab ich das problem, das ich im snapshot das erste mal etwas nicht anfinde...

und zwar
http://bugs.contenido.de/bug_view_page. ... id=0000141

und
http://bugs.contenido.de/bug_view_page. ... id=0000148

kann das sein das dies im CVS_HEAD nicht enthalten ist... oder hab ich was übersehen ?

die änderung für function isStartArticle ist vermutlich auch noch nicht drinnen...

weiteres folgt...

[timo]

ich bin mir sehr sicher, daß ich das eingecheckt habe...

die änderung mit der Sprache ist nur eine zeile irgendwo in der functions.con.php

[emergence]

ich geh mal testen...

das mit sprache hab ich gefunden... (habs beim ersten mal übersehen)
bei function conGenerateCodeForAllArtsUsingTemplate
das b.idlang = '".$lang."' AND -> raus...

lt. cvs wurde dies aber nur einmal entfernt und zwar bei
Search all categories
sollte diese zeile nicht auch bei
Search all articles
raus ?

das mit den cronjob hab ich jetzt auch schon gefunden...
jetzt ist mir auch klar warum das $db objekt in der funktion nicht zur verfügung stand aber egal...

[emergence]

ähm, hab mir jetzt gerade ein wenig die neue is_start_compatible einstellung angesehen und bin da auf ein problem gestossen...

vielleicht irre ich mich ja, aber es kommt mir zeitweise so vor als ob die tabelle con_cat_lang ein problem mit dem updaten des startartikels hat wenn ich unter den eigenschaften des artikels die kategorie ändere...

der eintrag für startidartlang ändert sich leider nicht dabei... sprich wenn ich den startartikel in ne andere kategorie verschiebe bleibt der eintrag dennoch weiterhin unverändert...

ich muss das jetzt nochmals genauer ansehen...

[emergence]

und gleich noch etwas gefunden...

hab mir jetzt die function conSyncArticle angesehen...
ist zwar nett nur synkronisiert die nicht alle spalten...
zb free_use_01 etc.. die ja selbst genutzt werden können (und was ich auch tue) bleiben aussen vor...

[emergence]

noch ne kleinigkeit für die front_content.php

bei location reload zeilen zb
header ($errsite);
sollte immer ein
die() um den header gesetzt werden... sonst läuft der rest des scripts noch durch -> was ja dann nicht benötigt wird und zeit frisst...
oder nachher gleich ein exit;

[emergence]

die neue is_start_compatible auf false zu lassen, ergibt bei mir momentan auch ne menge probleme bei den standard navigations modulen...

in der hauptnavigation verschwindet der menüpunkt dann ganz einfach...
setze ich den wert startidartlang in der db jedoch wieder auf 0 (weil ja momentan keiner vorhanden ist) erscheint der menüpunkt wieder....

[timo]

bei einer neuinstallation darfst du nicht is_start_compatible auf false setzen. das flag ist nur dafür da, bei einem upgrade dafür zu sorgen, daß alle contenido-funktionen so wie vorher funktionieren.

ich weiß nicht, inwieweit die standardmodule das flag abfragen...

[emergence]

beim CVS HEAD ist es aber standard mäßig in der config.php.tpl auf false...

mittlerweile hab ich das schon etwas ausgibig getestet, funktioniert eigentlich auch ganz gut bis auf die aktualisierung von startidartlang....

[timo]

ja, ist ja korrekt - bei neuinstallationen soll der neue modus verwendet werden.

welche aktualisierung von startidartlang? bei uns funktioniert das alles sehr gut...habe da bisher keine probleme feststellen können.

[emergence]

hey das forum scheint wieder normal zu gehen...

okay beispiel
is_start_compatible = false

in contenido auf content - artikel - bei einem startartikel in die eigenschaften
des startartikels - >
dann weisst du einfach den artikel in eine andere kategorie zu und siehst dir dann die tabelle con_cat_lang spalte startidartlang den wert der kategorie an... der sollte auf 0 stehen -> bleibt aber auf die id des verschobenen artikels und das ist ein bug.

[timo]

okay, akzeptiert

ich dachte, daß du meintest, daß die funktion generell nicht geht.

ich habe einen fix gebaut (functions.con.php, nähe zeile 430, nach dem $db->query für "delete from cat_art table"):

Code: Alles auswählen

/* Remove startidartlang */
$sql = "UPDATE ".$cfg["tab"]["cat_lang"]." SET startidartlang='0' WHERE idcat='$value' AND idlang='$lang'";
$db->query($sql);

[emergence]

noch ne kleinigkeit... (?) - habs gerade getestet
wenn in der anderen kategorie ein startartikel bereits vorhanden ist, wird diese zuweisung ebenfalls auf 0 gesetzt... was leider auch ein bug ist...

beispiel:
eine kategorie : 2 artikel, einer startartikel der andere ein normaler
ich weise den normalen einer anderen kategorie zu -> wird in der con_cat_lang startidartlang auf 0 gesetzt...

[emergence]

damit das wirklich korrekt funktioniert sollte dieses query verwendet werden:

Code: Alles auswählen

$sql = "UPDATE ".$cfg["tab"]["cat_lang"]." SET startidartlang='0' WHERE idcat='$value' AND idlang='$lang' AND startidartlang='$idart'";

[timo]

neuer code:

Code: Alles auswählen

                /* Remove startidartlang */
                if (isStartArticle($idartlang, $idcat, $lang))
                {
                	$sql = "UPDATE ".$cfg["tab"]["cat_lang"]." SET startidartlang='0' WHERE idcat='$value' AND idlang='$lang'";
                	$db->query($sql);
                }

Das ganze zeug ist leider total verwirrend...ich bin eigentlich ganz froh, die richtige stelle gefunden zu haben