Bugs und Fehler in CPO 4.0.8

Dieses Forum enthält Diskussionen zu Bugs in ConPresso 4 und deren Lösungen.
Benutzeravatar
Mr. Magpie
ConPresso-Profi
Beiträge: 1004
Registriert: 01.01.1970 01:00
Wohnort: Wuppertal
Hat sich bedankt: 274 Mal
Danksagung erhalten: 59 Mal

Bugs und Fehler in CPO 4.0.8

Beitrag von Mr. Magpie »

EDIT: Alles in diesem Beitrag hier Aufgeführte ist in den aktuellen 4.0.8-Downloads bereits berücksichtigt.




In der Datei _cfg/common-local.inc.php.dist ($Id: common-local.inc.php.dist 380 2009-05-10 11:45:22Z balu $) fehlt m.E. das schließende "?>".


Gleiches gilt für die Datei _data/index.php ($Id: index.php,v 1.11 2005/07/10 22:26:57 balu Exp $)


Datei _include/lang/deutsch/common.inc.php (korrigierte Version ist angehängt):

in Zeile 385 müsste es richtig "Webserver Schreibrechte" anstatt "Webserber Schweibrechte" heißen,

in Zeile 1061: korrekt wäre "Mitglieder" anstelle von "Member";

Zeile 1717: schlechtes Deutsch/schlechte Interpunktion, d.h. anstelle von "Bitte kontaktieren Sie Ihren Administrator falls Sie der Meinung sind dieses wäre falsch." sollte es besser heißen "Bitte kontaktieren Sie Ihren Administrator, falls, Sie der Meinung sind, dies wäre falsch";

Zeile 1859: "Bitte überprüfen Sie, ob die Größe der Datei Please check if the size of the file exceeds the maximum uploadable file size." ist natürlich Unsinn und hieße korrekt "Bitte überprüfen Sie, ob die Größe der Datei die maximal hochladbare Dateigröße überschreitet.";

Zeile 1995: lustig, aber falsch - "Sie können die Ausewahl nicht an dieser Position einfügen";

Zeile 2783: falsch ist "Richt-Text", richtig wäre "Rich-Text";

Zeile 3868: ich würde anstelle von "Bitte überprüfen Sie ob die Größe der Datei die maximale Dateigröße für hochgeladene Dateien überschreitet" schreiben "Bitte überprüfen Sie, ob die Größe der Datei die maximal hochladbare Dateigröße überschreitet";

Zeile 3065: falsches Weglassen von Bindestrichen in "Administrator Logindaten"; korrekt: "Administrator-Logindaten", noch besser: "Administrator-Login-Daten";

Zeile 3197: falsch: "Es wurden Probleme der Überprüfung der eingegebenen Daten festgestellt. Bevor Sie weitermachen und das Formular erneut absenden beheben Sie bitte die folgenden Probleme:", richtig: "Es wurden Probleme bei der Überprüfung der eingegebenen Daten festgestellt. Bevor Sie weitermachen und das Formular erneut absenden, beheben Sie bitte die folgenden Probleme:";

Zeile 3191: ein "z" zuviel in "PHP unterstzützt die Funktion";

in Zeile 3224 fehlen zwei Kommata: "alle Dateien und Verzeichnisse, die im Archiv enthalten waren, korrekt";

Zeile 3233: statt "oder mache dem Administrator" muss es "oder machen dem Administrator" lauten;

Zeile 3275: falsch ist "Beim überprüfen der Voraussetzungen für %s wurden Probleme gefunden. Bevor Sie weitermachen beheben Sie bitte die folgenden Punkte:", richtig wäre "Beim Überprüfen der Voraussetzungen für %s wurden Probleme gefunden. Bevor Sie weitermachen, beheben Sie bitte die folgenden Punkte:";

Zeile 3299: bitte "Abschluß" in "Abschluss" ändern;

in Zeile 3377 kann sicherlich "zurueckgesetzt" auch als "zurückgesetzt" geschrieben werden;

Zeile 3416: in "Artikel, etc." ist das Komma zuviel;

in Zeile 3581 wäre "Aufzählungs-Liste" besser als "unsortierte Liste" und entsprechend wäre m.E. in Zeile 3584 "Nummerierte Liste" besser als "sortierte Liste", aber das ist Geschmackssache.


Müsste in _include/var_category.php die Zeile 127 anstelle von

if (strtolower($db->v('name'))!=strtolower($directory)) continue;

nicht

if (strtolower($v['name'])!=strtolower($directory)) continue;

lauten? (siehe http://community.conpresso.de/viewtopic ... 0785#20785)
Dateianhänge
common.inc.php.zip
(28.57 KiB) 507-mal heruntergeladen
Zuletzt geändert von Mr. Magpie am 19.05.2009 19:59, insgesamt 4-mal geändert.
Günther Ludwig
Benutzeravatar
Mr. Magpie
ConPresso-Profi
Beiträge: 1004
Registriert: 01.01.1970 01:00
Wohnort: Wuppertal
Hat sich bedankt: 274 Mal
Danksagung erhalten: 59 Mal

Beitrag von Mr. Magpie »

EDIT: Ist in den aktuellen 4.0.8-Downloads bereits berücksichtigt.




In den Dateien index.php der Verzeichnisse

_include/lang/deutsch/ und
_include/lang/english/

muss es anstelle von

Code: Alles auswählen

require_once('../../_include/common.inc.php');
heißen

Code: Alles auswählen

require_once('../../../_include/common.inc.php');
oder

Code: Alles auswählen

require_once('../../common.inc.php');
Zuletzt geändert von Mr. Magpie am 19.05.2009 20:00, insgesamt 1-mal geändert.
Günther Ludwig
Benutzeravatar
balu
ConPresso-Entwickler
Beiträge: 1748
Registriert: 01.01.1970 01:00
Hat sich bedankt: 81 Mal
Danksagung erhalten: 133 Mal

Re: Bugs und Fehler in CPO 4.0.8

Beitrag von balu »

Hallo Magpie,

ich integriere die Sachen heute Abend noch soweit es geht in die 4.0.8.
Mr. Magpie hat geschrieben:In der Datei _cfg/common-local.inc.php.dist ($Id: common-local.inc.php.dist 380 2009-05-10 11:45:22Z balu $) fehlt m.E. das schließende "?>".
Das sieht man heutzutage anders. Der PHP-Interpreter erkennt anhand des Dateiendes, dass auch PHP zuende ist und fügt quasi selber ein ?> ein.

Der Vorteil ist, dass nicht aus Versehen noch eine Leerzeile oder ein Leerzeichen stehen kann.

Aus diesem Grund wird in neueren Coding-"Richtlinien" empfohlen, das schliessende ?> am Ende einer Datei wegzulassen.

Wir werden das jetzt nach und nach machen, wenn wir sowieso gerade an den Dateien sind.

Den Rest schaue ich mir gleich an.

Balu
Bartels.Schöne
ConPresso Support & Development
Benutzeravatar
MarkusR
Handbuchversteher
Beiträge: 7362
Registriert: 01.01.1970 01:00
Hat sich bedankt: 111 Mal
Danksagung erhalten: 934 Mal
Kontaktdaten:

Beitrag von MarkusR »

Ich habe gerade mal das Kopieren von Templates inkl. der Platzhalter getestet und es scheint bis auf einen Schönheitsfehler zu funktionieren (wobei dies nicht unbedingt an der neuen Version liegen muß):

Nach Anklicken des Copy-Schalter erhält man ja die Kopie zur Bearbeitung. Im Quelltext dieser Seite ist mir die Meldung über einen Array-Fehler aufgefallen:
<script type="text/javascript"> var placeholder = new Array();<br />
<b>Warning</b>: Invalid argument supplied for foreach() in <b>/var/www/___/_admin/templates.php</b> on line <b>334</b><br />
</script>
und das ist dann dieser Block in templates.php

Code: Alles auswählen

        if ($editor_allowed) {
            print('<script type="text/javascript"> var placeholder = new Array();');
            foreach ($placeholder as $k=>$v) {
                print("placeholder.push('".$v[type]."', '".$v[name]."');\n");
            }
            print('</script>');
Vielleicht kann man das ja noch ausbügeln... z.B.

Code: Alles auswählen

if ($editor_allowed && is_array($placeholder)) {
Ciao Markus
ConPresso-Module

Kein Support per PN!!! Für Fragen und Diskussionen ist das Forum da!

Succi recentis officinalis
Hochwertige Kräutersäfte und -Öle
Benutzeravatar
Mr. Magpie
ConPresso-Profi
Beiträge: 1004
Registriert: 01.01.1970 01:00
Wohnort: Wuppertal
Hat sich bedankt: 274 Mal
Danksagung erhalten: 59 Mal

Beitrag von Mr. Magpie »

Da ich templates.php leicht geändert habe, ist es bei mir die Zeile 336. Die Fehlermeldung liest sich in der FF-Fehlerkonsole so wie im angefügten Bild.
Also nur ein nicht-gequotetes "/", oder?

Was man aber wirklich auch einmal fixen könnte, sind die "hoover"-Fehler in der style_cpo.css.php!
Dateianhänge
Fehlerkonsole.png
Günther Ludwig
Benutzeravatar
MarkusR
Handbuchversteher
Beiträge: 7362
Registriert: 01.01.1970 01:00
Hat sich bedankt: 111 Mal
Danksagung erhalten: 934 Mal
Kontaktdaten:

Beitrag von MarkusR »

Also nur ein nicht-gequotetes "/", oder?
Quatsch :mrgreen:

Da sollte ein (oder auch kein) Javascriptcode und keine in HTML formatierte Fehlermeldung erscheinen... :wink:
Was man aber wirklich auch einmal fixen könnte, sind die "hoover"-Fehler in der style_cpo.css.php!
*lol* :D
Aber wirklich... :wink:

Dann wäre ein Entsorgen von

Code: Alles auswählen

/* default alignment of table cells */
td {vertical-align: top; text-align: left;}
in der style.css.dist aber auch mal fällig. Sooo viele Webauftritte, in denen das zu Problemen führt, weil das CCS-File ergänzt statt ersetzt wird... aktuell zu sehen auf der Kinderhaus65-Seite...
Ciao Markus
ConPresso-Module

Kein Support per PN!!! Für Fragen und Diskussionen ist das Forum da!

Succi recentis officinalis
Hochwertige Kräutersäfte und -Öle
Benutzeravatar
Mr. Magpie
ConPresso-Profi
Beiträge: 1004
Registriert: 01.01.1970 01:00
Wohnort: Wuppertal
Hat sich bedankt: 274 Mal
Danksagung erhalten: 59 Mal

Beitrag von Mr. Magpie »

MarkusR hat geschrieben: Da sollte ein (oder auch kein) Javascriptcode und keine in HTML formatierte Fehlermeldung erscheinen... :wink:
Auch wieder wahr 8)
Günther Ludwig
Benutzeravatar
balu
ConPresso-Entwickler
Beiträge: 1748
Registriert: 01.01.1970 01:00
Hat sich bedankt: 81 Mal
Danksagung erhalten: 133 Mal

Beitrag von balu »

Hm,

wofür benutzen wir hier eigentlich die ganzen Tools, wenn sie uns sowas wie die hoover-Fehler nicht anzeigen... :-(

Ich schaue mir die Sachen morgen im Detail an. Es wurden auch noch andere Probleme gemeldet, die mit dem erweiterten Debug-System zusammenhängen.

Balu
Bartels.Schöne
ConPresso Support & Development
Benutzeravatar
balu
ConPresso-Entwickler
Beiträge: 1748
Registriert: 01.01.1970 01:00
Hat sich bedankt: 81 Mal
Danksagung erhalten: 133 Mal

Beitrag von balu »

Hi,

ich habe mir das jetzt auch nochmal angeschaut und die folgenden Änderungen vorgenommen:

In Zeile 61 (vor dem while) und in Zeile 90 (vor dem $views =) habe ich jeweils die Definition der Variable ergänzt:

$placeholder = array();

Ich habe mir angewöhnt, die Variablen zu initialisieren und nicht wie in Markus Vorschlag zu prüfen, ob sie gesetzt ist.

Meine Idee dahinter ist, dass man so immer einen definierten Wert hat und nicht evtl. von aussen etwas in dieses Array "eingeschmuggelt" werden kann. Früher gab es sowas durch die register-globals-Probleme relativ häufig.

Und wenn man dann so testet, fällt einem auf, dass die "Vorhandenen Platzhalter" beim Bearbeiten nicht im Editor angezeigt werden und dass templates_modify bzw. templates_copy quasi identisch sind und zusammengefasst werden könnten.

Das Ergebnis ist die _admin/templates.php im Anhang. Bitte testet diese einmal, nach vorheriger Sicherung Eurer aktuellen Datei.

Balu
Dateianhänge
templates.txt
Aktualisierte _admin/templates.php
(25.03 KiB) 520-mal heruntergeladen
Bartels.Schöne
ConPresso Support & Development
Benutzeravatar
MarkusR
Handbuchversteher
Beiträge: 7362
Registriert: 01.01.1970 01:00
Hat sich bedankt: 111 Mal
Danksagung erhalten: 934 Mal
Kontaktdaten:

Beitrag von MarkusR »

Bei mir ist die Fehlermeldung jetzt weg.

Ein anderes Problem kann ich noch nicht nachvollziehen, da es bisher erst zweimal aufgetreten ist und sich nicht rekonstruieren lässt. Dabei wurde ich nach dem Login wieder auf die Login-Maske geleitet, obwohl der Login erfolgreich war. Nach Schließen aller Browser-Fenster konnte ich mich wieder einloggen und sehen, daß alle Logins technisch gesehen erfolgreich waren, da ich nun mehrfach eingeloggt war... werde das verfolgen, wenn es wieder auftritt.
Ciao Markus
ConPresso-Module

Kein Support per PN!!! Für Fragen und Diskussionen ist das Forum da!

Succi recentis officinalis
Hochwertige Kräutersäfte und -Öle
Benutzeravatar
balu
ConPresso-Entwickler
Beiträge: 1748
Registriert: 01.01.1970 01:00
Hat sich bedankt: 81 Mal
Danksagung erhalten: 133 Mal

Beitrag von balu »

Hast Du irgendwo eine Grafik eingebaut, die nicht auf einen Dateinamen, sondern nur auf den _data-Ordner zeigt?

In dem Fall wird der Browser (nicht sichbar, da <img>-Tag) weitergeleitet auf die Loginseite und die Session resettet.

Das Problem hatte ich vor kurzem bei einem Kunden

Eine andere Überlegung ist, dass es was mit dem Sicherheitsupdate zu tun hat, bei dem man nach einem Login, Logout oder Timeout eine neue Session-ID erhält.

Balu
Bartels.Schöne
ConPresso Support & Development
Benutzeravatar
MarkusR
Handbuchversteher
Beiträge: 7362
Registriert: 01.01.1970 01:00
Hat sich bedankt: 111 Mal
Danksagung erhalten: 934 Mal
Kontaktdaten:

Beitrag von MarkusR »

Es ist eine noch unveränderte (außer nun template.php) 4.0.8. Ohne Module, ohne alles.

Keine Änderungen im Backend, nur Inhalte im Frontend, keine weitere Aktion zwischen den mehrfach gescheiterten Logins. Kann es ansonsten noch nicht weiter eingrenzen und werde laut geben, wenn ich mehr weiß.
Ciao Markus
ConPresso-Module

Kein Support per PN!!! Für Fragen und Diskussionen ist das Forum da!

Succi recentis officinalis
Hochwertige Kräutersäfte und -Öle
Benutzeravatar
Mr. Magpie
ConPresso-Profi
Beiträge: 1004
Registriert: 01.01.1970 01:00
Wohnort: Wuppertal
Hat sich bedankt: 274 Mal
Danksagung erhalten: 59 Mal

Beitrag von Mr. Magpie »

Ich hatte heute auch ein komisches Erlebnis!

Ich hatte im Rubriknamen einen Punkt, bspw. Home.nl - daraufhin hat die Bearbeitung der Navigationsstruktur völlig gesponnen: Änderungen wurden mehrfach ausgeführt, immer wieder tauchte ein oberster Navi-Punkt "root" auf, Löschen eines Navi-Punktes löschte ganz andere Navi-Punkte usw.

Wenn ich einmal Zeit finde, teste ich das nochmal mit einer unveränderten v.4.0.8 ohne Module etc.
Günther Ludwig
Benutzeravatar
MarkusR
Handbuchversteher
Beiträge: 7362
Registriert: 01.01.1970 01:00
Hat sich bedankt: 111 Mal
Danksagung erhalten: 934 Mal
Kontaktdaten:

Beitrag von MarkusR »

Möglicherweise ist die Datenbank aus dem Takt...

...benutzt Du "Lock Tables"?

Das wurde zu Anfangszeiten der 4er Versionen schon einmal beobachtet...
Ciao Markus
ConPresso-Module

Kein Support per PN!!! Für Fragen und Diskussionen ist das Forum da!

Succi recentis officinalis
Hochwertige Kräutersäfte und -Öle
Benutzeravatar
Mr. Magpie
ConPresso-Profi
Beiträge: 1004
Registriert: 01.01.1970 01:00
Wohnort: Wuppertal
Hat sich bedankt: 274 Mal
Danksagung erhalten: 59 Mal

Beitrag von Mr. Magpie »

MarkusR hat geschrieben:...benutzt Du "Lock Tables"?
Ja, d.h. die Option in den Systemeinstellungen ist nicht angehakt. Möglicherweise war ich - so wie bei deinem seltsamen Mehrfachlogin - mindestens zweimal eingeloggt. Was Ursache, was Wirkung war, kann ich so leider nicht mehr sagen. Jedenfalls, nachdem ich die Punkte in den Rubriknamen eliminiert hatte, war alles wieder wie gewohnt...
Günther Ludwig
Antworten