anmerkung: contenido-cvs-2005-11-18.tar

Beitrag von **HerrB** » Mi 23. Nov 2005, 23:44

Tja, wenn man die neueste Version einsetzt... Wird ein Modul gespeichert, werden " durch \" ersetzt. Beim nächsten Speichern durch \\", beim nächsten \\\\\"... na, usw.

Bin mir gerade nicht sicher, ob das ein Bug ist oder mehr ein Problem mit der Providerkonfiguration (Hosteurope).

Gruß
HerrB

Beitrag von **emergence** » Do 24. Nov 2005, 09:57

ähm schaut nach bug aus...

Beitrag von **HerrB** » Do 24. Nov 2005, 23:17

Ist ein Bug, bei V4.6.2 tritt es nicht auf. Vermutlich bei allen Edit-Bereichen (Module, Layouts, woanders habe ich es nicht mehr getestet).

Gruß
HerrB

Beitrag von **HerrB** » Sa 26. Nov 2005, 19:27

Der Fehler in der class.newsletter.php, der in V4.6.0 eingebaut wurde und in V4.6.1 nicht ganz gefixt war, macht auch in der V4.6.2 noch Probleme. Die letzte Korrektur hat zwar dazu geführt, dass kein Parse Error mehr auftritt, aber bei der Gelegenheit ist auch die Funktion flöten gegangen.

Den Fix (und ich wäre waaaaahnsinnig dankbar, wenn er - so, wie er ist - Eingang in die nächste Version finden könnte) findet sich [hier]

Ich verwende auch gerne in Zukunft

Code: Alles auswählen

if bla
{
   foo
} else
{
   foo2
}

wobei ich

Code: Alles auswählen

if bla {
   foo
} else {
   foo2
}

definitiv besser lesbar finde (und es spart Platz).

Gruß
HerrB

timo · Beitrag von **timo** » Sa 26. Nov 2005, 20:06

zu den Coding Conventions sag ich an der stelle nichts

das mit den händisch einzubauenden Fixes ist halt ein Problem...kannst du eventuell einen Patch erzeugen und mir schicken?

Beitrag von **HerrB** » Sa 26. Nov 2005, 20:17

Sage doch was zu den Conventions - wenn ihr es immer so macht, mache ich es auch...

Das mit dem Patch: Ähm, wenn ich jetzt wüsste, wie? Es genügt mir, wenn Du mir die Komponenten nennst (z.B. verwendetes CVS/Version), um sowas zu machen...

Gruß
HerrB

Beitrag von **HerrB** » Sa 26. Nov 2005, 20:17

Zwei Bugs im Modul News (V4.6.2): http://contenido.org/forum/viewtopic.php?p=62600#62600

Gruß
HerrB

timo · Beitrag von **timo** » Sa 26. Nov 2005, 20:45

HerrB hat geschrieben:Sage doch was zu den Conventions - wenn ihr es immer so macht, mache ich es auch...

Das mit dem Patch: Ähm, wenn ich jetzt wüsste, wie? Es genügt mir, wenn Du mir die Komponenten nennst (z.B. verwendetes CVS/Version), um sowas zu machen...

Am einfachsten ist es mit Eclipse - du entpackst das Zip, holst dir einen Workspace, machst die Änderungen und dann erzeugst du einen Patch.

Das mit den Conventions muß ich dir am Montag mal raussuchen, im Prinzip so wie bei Java: http://java.sun.com/docs/codeconv/html/ ... 6.html#449

Beitrag von **Halchteranerin** » Sa 26. Nov 2005, 21:02

timo hat geschrieben:Das mit den Conventions muß ich dir am Montag mal raussuchen, im Prinzip so wie bei Java: http://java.sun.com/docs/codeconv/html/ ... 6.html#449

Die Conventions entsprechen doch HerrBs Notation, ihr scheint etwas anderes benutzt zu haben, wenn ich sein Posting richtig interpretiere.

Beitrag von **HerrB** » Sa 26. Nov 2005, 21:04

Ja, na dann kann ich ja weitermachen ...

Gruß
HerrB

timo · Beitrag von **timo** » Sa 26. Nov 2005, 21:20

sorry mein fehler. Ich poste die richtigen nochmal am Montag - bitte nochmal dran erinnern

pit · Beitrag von **pit** » So 27. Nov 2005, 11:22

Hallo HerrB,

bei mir tritt der Fehler auch noch bei der cvs vom 25.11. auf, allerdings bei einer Installation mit PHP 4.2.3

bei Installation auf PHP 4.4.0 funktioniert es.

Irgendwas mit preg in class.module.php Funktion parseModulForStrings???

Beitrag von **HerrB** » So 27. Nov 2005, 12:14

Hier war es PHP 5.0.5. Da das Problem in verschiedenen Bereichen vorkommt (nicht nur Module), würde ich eine Änderung an zentraler Stelle vermuten - aber ich nehme an, das rührt daher, dass an der Art und Weise, wie Formulardaten verarbeitet werden, gearbeitet wird. Also erst mal abwarten...

Gruß
HerrB

pit · Beitrag von **pit** » So 27. Nov 2005, 15:16

ok, abwarten und bei PHP 4.2.3 mit 4.6.2 arbeiten?

Beitrag von **HerrB** » Mo 28. Nov 2005, 00:15

Ja.

Gruß
HerrB