ECB SHOP: Fehlerunterdrückung bei mysql_connect()

[markxs]

Hallo

Kann mir bitte jemand erklären, wieso in inc/xtc_db_connect.inc.php die Zeile(n)

29**** if (USE_PCONNECT == 'true') {
30******** $$link = @mysql_pconnect($server, $username, $password) or die ('can not connect to Database!');
31**** } else {
32******** $$link = @mysql_connect($server, $username, $password) or die("can not connect to Database!");
33**** }

die Fehlermeldungen unterdrücken? Es kostete mich eben 1h um herauszufinden, dass mysql_connect() einen Fehler wirft und zwar dass die Funktion undefined ist (= kein mysql installiert), aber diese Fehlermeldung wurde unterdrückt.

Ich finde die beiden @ an dieser Stelle fahrlässig und total unangebracht. Wäre die Fehlermeldung sofort erschienen, hätte ich nach wenigen Minuten MySQL installiert gehabt.

Oder gibt es einen wirklich guten Grund für @ an der Stelle?


mfg

[Xantiva]

Da fragst Du am besten mal die XTC - Entwickler ... :smiley_emoticons_aufsmaul:

Sauber wäre hier wohl mit try-catch zu arbeiten und eine lesbare Fehlermeldung herauszugeben.

[markxs]

''>ZITAT(Ex-Entwickler @ Nov 13 2009, 14:36) 203692
Da fragst Du am besten mal die XTC - Entwickler ... :smiley_emoticons_aufsmaul:

Sauber wäre hier wohl mit try-catch zu arbeiten und eine lesbare Fehlermeldung herauszugeben.
[/b]
So ungefähr. Ich dachte die Entwickler lesen hier mit? Wie sollte ich ihnen besser die Frage stellen?

[SMT WebServices]

''>ZITAT(markxs @ Nov 13 2009, 16:35) 203711
''>ZITAT(Ex-Entwickler @ Nov 13 2009, 14:36) 203692[/snapback]
Da fragst Du am besten mal die XTC - Entwickler ... :smiley_emoticons_aufsmaul:

Sauber wäre hier wohl mit try-catch zu arbeiten und eine lesbare Fehlermeldung herauszugeben.
[/b]
So ungefähr. Ich dachte die Entwickler lesen hier mit? Wie sollte ich ihnen besser die Frage stellen?
[/b][/quote]
also, ob die xtc entwickler hier mitlesen, keine Ahnung, wenn ja vermutlich eher um doch noch was zu finden wie sie volker doch noch drankriegen könnten und ihre gerichtlichen Schlappen wettmachen, aber mit 100%iger Sicherheit nicht, um userprobleme zu lösen (das tun sie ja schon in ihrem eigenen forum nicht)

was Ex-Entwickler vermutlich meinte (der ja zu den ECB Entwicklern zählt) ist das dies noch ein erbe des ecb vom xtc ist, auf dem ecb ja basiert, und man kann nunmal nicht alles auf einmal fixen (dazu ist die summe an bugs und unzulänglichkeiten im original xtc code einfach zu hoch)

[markxs]

''>ZITAT(SMT WebServices @ Nov 13 2009, 17:11) 203726
also, ob die xtc entwickler hier mitlesen, keine Ahnung, wenn ja vermutlich eher um doch noch was zu finden wie sie volker doch noch drankriegen könnten und ihre gerichtlichen Schlappen wettmachen, aber mit 100%iger Sicherheit nicht, um userprobleme zu lösen (das tun sie ja schon in ihrem eigenen forum nicht)

was Ex-Entwickler vermutlich meinte (der ja zu den ECB Entwicklern zählt) ist das dies noch ein erbe des ecb vom xtc ist, auf dem ecb ja basiert, und man kann nunmal nicht alles auf einmal fixen (dazu ist die summe an bugs und unzulänglichkeiten im original xtc code einfach zu hoch)
[/b]
Achso, jetzt verstehe ich das. Ich wollte es auch den Entwicklern von ECB mitteilen, dass ich diese Stelle für total unsauber halte. Ex-Entwicklers Antwort bezog sich auf mein "Warum ist das dort so?". Damit ist das für mich geklärt. Ein ECB-Entwickler weiß davon und teilt meine Meinung ;)

[Xantiva]

Hi markxs,

sorry, meine Antwort war missverständlich.

Diese "Unschönheit" ist eine von unzähligen, die noch im Code schlummern. Wir haben schon überlegt ein "Best of ..." zu machen, so als kleines Andenken. :rolleyes:

Ich habe es in den BugTracker mit aufgenommen, aber erst sind "echte" Bugs dran, sofern sie denn wenigstens hier gemeldet werden.

http://bugs.ecb-shop.de/view.php?id=210 (http://anonym.to/?http://bugs.ecb-shop.de/view.php?id=210)

Ciao,
Mike

[markxs]

Hallo Mike

Klar sind zuerst Bugs dran. Das hier ist ja mehr ein Hinweis, kein Fehler.

Wurde denn mein hier beschriebener Bug http://www.ecombase.de/forum/index.php?sho...c=50490&hl= (http://www.ecombase.de/forum/index.php?showtopic=50490&hl=) ausreichend von den Entwicklern aufgenommen? Er wurde nur geschlossen ohne irgendeinen Kommentar. Ich habe im erwähnten Bugtracker das http://bugs.ecb-shop.de/view.php?id=206 (http://anonym.to/?http://bugs.ecb-shop.de/view.php?id=206) gefunden. Dort wird zumind. einer von zwei Fehlern erwähnt.

Möchtet ihr denn, dass die Nutzer hier selbstständig dort Fehler eintragen? Entstehen dann dort nicht zuviele Einträge, von denen viel zu viele keine echten Bugs sind?

mfg

[Xantiva]

Wer dort "sauber" dokumentierte, nachvollziehbare Bugs einträgt, ist gerne gesehen. Ist eine Lösung dabei, um so besser.

Wenn einer nur nicht nachvollziehbaren Schrott einträgt, kann man ihm auch schnell wieder die Rechte entziehen!

Zu dem zweiten Teil: Da wir die Tabelle erzeugt, ich würde dafür die Ausgabe von SHOW CREATE TABLE nehmen, wie beschrieben ...

Ciao,
Mike