anmerkung: snapshot contenido-cvs-2004-04-09.tar

[emergence]

ich gehs mal auf die schnelle durch...

conlib/local.php
kommt dreimal

Code: Alles auswählen

global $cfgPathImg;

vor...

kann entfernt werden... (wird wirklich nicht mehr gebraucht...)

ähm

Code: Alles auswählen

$saveLoginTime = "1";

sollte das nicht

Code: Alles auswählen

$saveLoginTime = true;

sein...

ne nette modifikation für die
conlib\db_mysql.inc

Code: Alles auswählen

$this->halt("Invalid SQL: ".$Query_String."<br><br>");

ersetzen durch

Code: Alles auswählen

$this->halt("Invalid SQL: ".$Query_String);

und bei function halt($msg)
nach

Code: Alles auswählen

$this->haltmsg($_SERVER["REQUEST_URI"]. " ". $msg);

sollte

Code: Alles auswählen

$this->haltmsg("MySQL-Error: ".$this->Error." (".$this->Errno.")");

ergänzt werden
vorteile:
-> das sql statement hat keine <br><br> mehr im errorlog.txt
-> der mysql fehler den die db meldet wird mit protokolliert... ein testen bei phpmyadmin -> welchen fehler das statement liefert kann man sich dann sparen...
ach ja das $_SERVER["REQUEST_URI"] wird das wirklich gebraucht ?

noch ne frage zu:
ach ja function nextid($seq_name)
findet sich ja

Code: Alles auswählen

return $nextid;

sollte das nicht:

Code: Alles auswählen

return $nextid - 1;

sein ?

[emergence]

ähm in
contenido\classes\class.plugin.php
sollte
global $cfg_con in global $cfg und
$con_cfg['PathPlugins'] in $cfg['path']['plugins']
geändert werden...

in
contenido\classes\class.backend.php
bei function select($area)
kann man bei global
$con_cfg, $cfgPathInc, entfernen...

bei
contenido\classes\class.navigation.php
bei function Contenido_Navigation()
bei global $con_cfg entfernen...
und etwas weiter unten findet man...

Code: Alles auswählen

$name = $plugxml->valueOf($xpath);

sollte

Code: Alles auswählen

$name = $plugxml->valueOf($cfgPath['xmlroot'] . $xpath);

sein...

[emergence]

zwei kleinigkeiten sozusagen als feature request
(ich weiss schon ich solls in den bugtracker schreiben)
eine möglichkeit beim actionlog es komplett zu leeren wäre ganz brauchbar... (mit eigener berechtigung)
ebenso eine möglichkeit die con_code zu leeren sollte allgemein gegeben sein...

[HerrB]

Von meinen Anmerkungen sind noch die folgenden übrig oder neu hinzugekommen (kursiv). Sachen, die im Bugtracker bereits eingetragen sind, habe ich rausgenommen.

MyContenido:
Symbolhilfe enthält ein "Empfönger" im Inhaltsverzeichnis.

Content -> Artikel:
Eigenschaften eines Artikels: Funktion "Article link" zeigt im IE keine Funktion (es erscheint kein Link).

Content -> Artikel -> Editor -> SPAW
- Bild einfügen: Leicht derangiert, verkleinert sich bei Wechsel des Ordners einmal um ca. 10 Pixel (Dialogfenster entspricht der 4.4.3/4-Version). Verzeichnisliste ("Bibliothek") zeigt leeren Eintrag zusätzlich zu den anderen Verzeichnissen.
- Link einfügen: Alle Felder sehr schmal, "Externer Link"-Feld sehr schmal -> Änderung aus V4.4.3 wurde nicht übernommen.

Content -> Kategorie:
Versuch der Template-Änderung für eine Kategorie führt zu '"obj" ist undefiniert (Zeilen und Zeichen variieren).

Content -> Dateiverwaltung:
Tabelle mit den Informationen zu den Dateien ist für 1024x768 eine Spur zu breit (ohne das entspechend breiter Inhalt vorhanden wäre).

Size zeigt immer 0 B an.

Style -> Layouts:
Hat man sich ein Layout angesehen (Vorschau) und fügt ein neues Template hinzu, vergibt einen Namen und speichert es, verbleibt die Markierung des Submenüs bei 'Vorschau'. Klickt man 'Vorschau' erneut an, erscheint die Vorschau des vorher angesehenen Layouts.

Style -> Templates:
Wird bei einem neuen Template das Layout von -- None -- auf einen andern Wert gewechselt, wird das Template ohne zutun des Nutzers gespeichert (damit kann man sogar Templates ohne Namen erzeugen). Templates ohne Name kann man dann nicht mehr editieren (aber löschen).

Hat man den Namen vor Auswahl des Layouts festgelegt, wird das Template mit dem Namen erzeugt, allerdings ist in diesem Template der Aufruf der Vorkonfiguration zwar möglich; sie enthält jedoch keine Optionen. Ausserdem werden die falschen Menüpunkte farbig unterlegt.

Style allgemein (Style -> Layouts, Module, Templates, usw.):
Die Liste der Elemente wird aktualisiert, wenn zwischen den Menüpunkten (z.B. Vorschau und Editieren; Historie und Bearbeiten) gewechselt wird (ohne Änderungen zu speichern). Vielleicht ist es möglich, darauf zu verzichten (Performance)?

Extras -> Empfänger:
Werden die Eigenschaften eines Empfängers angezeigt und der Empfänger links gelöscht, werden die Eigenschaften noch immer angezeigt. Änderungen in diesen Eigenschaften werden naturgemäß nicht mehr gespeichert.

Administration -> Benutzer/Gruppen -> Bereiche:
Der Bereich enthält noch Trümmer aus der Frontend-Berechtigung (ganz unten).

Ggf. vorhandene Feature-Requests habe ich im Bug-Tracker ergänzt.

Und weil ich es im Bugtracker gefunden habe (0000200): Toggle geht nicht in Style -> Styleeditor, Extras -> Empfänger, Statistik -> Zugriffe (mit JS-Fehler), Administration -> Benutzer.

Gruß
HerrB

[timo]

ach ja das $_SERVER["REQUEST_URI"] wird das wirklich gebraucht ?

Ja, denn es ist ungemein praktisch, wenn man auf einer produktiven Website Fehler im errorlog.txt findet, aber nicht weiß, mit welcher URL man sie reproduzieren kann.

[emergence]

@all

bevor ich es vergesse...
frohe ostern
(bin schon etwas spät dran)

[timo]

ja dito - um den Rest kümmere ich morgen

[timo]

noch ne frage zu:
ach ja function nextid($seq_name)
findet sich ja

Code: Alles auswählen

return $nextid;

sollte das nicht:

Code: Alles auswählen

return $nextid - 1;

sein ?

Das habe ich schon woanders gelöst, ich glaube, ich schreibe minus eins in die DB mit rein, d.h. die Funktion sollte jetzt so richtig funktionieren.

[timo]

ähm in
contenido\classes\class.plugin.php

Die Plugin-Klasse habe ich mal rausgeworfen, ich glaube nicht, daß die jemals so verwendet wird.

Den rest habe ich geändert.

[emergence]

geschlossen...