kleinere Code Fehler gefunden

keiang · 09 März 2007, 14:59:19

Hallo zusammen,
ich habe ein paar kleinere Fehler gefunden, wäre schön wenn die Korrekturen in die nächsten Updates übernehmen könnte.

SMF-Bridge:
Datei: "index.template.php" in Ordner "mx-port"
alt: "echo $txt['welcome_guest'],"
neu: "echo $txt['welcome_guest'], '</span>' ;"

FAQ-Modul:
Datei: "index.php"
alt: " " . _CATEGORY . ": <a href=\"modules.php?name=FAQ\">" . _MAIN . "</a> ->" . ($categories) . "<br>"
neu: " " . _CATEGORY . ": <a href=\"modules.php?name=FAQ\">" . _MAIN . "</a> ->" . ($categories) . "<br>"

SMF-Bride:
Datei: "block-smf_Center.php"
Fehler: Der Wert von "colspan" in den Zeilen 147, 153, 159 ist falsch wenn eine der Spalten deaktiviert wird. ( z.B. "$show_starter = 0;" )
Leider sind meine php-Kenntnisse weniger als "poor" und weiß deswegen auch nicht wie man das automatisch machen lassen könnte.

keiang · 09 März 2007, 19:57:54

Hab noch was gefunden:

News-Modul
Datei: "index.php"
Zeile: 101
alt: . _SEARCHONTOPIC . ": <input type=\"name\" name=\"query\" size=\"30\">  "
neu: . _SEARCHONTOPIC . ": <input type=\"text\" name=\"query\" size=\"30\">  "

Andi · 12 März 2007, 09:53:46

Hi

vielen Dank für die Hinweise

Habe alle Vorschläge eingearbeitet, updates kommen....

keiang · 12 März 2007, 12:38:37

Search Modul
Datei: "index.php"
Zeile: 181
alt:

Code Auswählen

<input type="radio" name="type" value="stories"' . (($type == 'stories') ? 'checked' : '') . '> ' . _SSTORIES . '
neu:

Code Auswählen

<input type="radio" name="type" value="stories"' . (($type == 'stories') ? ' checked' : '') . '> ' . _SSTORIES . '

( Leerzeichen fehlt vor checked )

keiang · 12 März 2007, 13:38:27

News Modul
Datei: "lang-german.php"
Zeile: 123
alt: define("_SEARCHONTOPIC","Thema duchsuchen");
neu: define("_SEARCHONTOPIC","Thema durchsuchen");

Andi · 12 März 2007, 23:30:44

Jop, weiter so, gibs uns ....

keiang · 20 März 2007, 17:48:39

SMF Bridge
Datei: "index.template.php" in Ordner ".../modules/Forum/smf/Themes/mx-port/"

Zeilen: 386, 389, 393, 396

In diesen Zeilen scheint der "footertext" auskommentiert zu sein mit "", ist das Absicht oder evtl. noch vom debuggen drin?

keiang · 21 März 2007, 16:16:06

Feedback Modul

Datei: "index.php"
Zeilen: 71 -7 2

alt:

Zitat. ''
. '<p align="right">' . $req . ' ' . _INDICATESREQ . '</p>';

neu:

Zitat. '<p class="content" align="center">' . _PLEASEPROVIDE . '  ' . $req . ' ' . _INDICATESREQ . '</p>';

Zeilen: 108 - 109 ( Leerzeichen vor "checked" fehlt )

alt:

Zitat<input type=\"radio\" name=\"sendcopy\" value=\"1\"" . (empty($sendcopy) ? '' : 'checked="checked"') . "> " . _YES . "
<input type=\"radio\" name=\"sendcopy\" value=\"0\"" . (empty($sendcopy) ? 'checked="checked"' : '') . "> " . _NO . "

neu:

Zitat<input type=\"radio\" name=\"sendcopy\" value=\"1\"" . (empty($sendcopy) ? '' : ' checked="checked"') . "> " . _YES . "
<input type=\"radio\" name=\"sendcopy\" value=\"0\"" . (empty($sendcopy) ? ' checked="checked"' : '') . "> " . _NO . "

Datei: "lang-german.php" in ".../Feedback/language/"
Zeile: 34

alt:

Zitatdefine("_PLEASEPROVIDE","geben Sie bitte die folgenden Kontaktinformationen an.");

neu:

Zitatdefine("_PLEASEPROVIDE","Geben Sie bitte die folgenden Kontaktinformationen an.");

Andi · 22 März 2007, 17:44:28

Zitat von: keiang am 20 März 2007, 17:48:39
SMF Bridge
Datei: "index.template.php" in Ordner ".../modules/Forum/smf/Themes/mx-port/"

Zeilen: 386, 389, 393, 396

In diesen Zeilen scheint der "footertext" auskommentiert zu sein mit "", ist das Absicht oder evtl. noch vom debuggen drin?

Hi

jop, das ist Absicht.
Auskommentiert ist da auch nicht der eigentliche Copyrighthinweis, sondern die Credits und W3C Bildchen.
Wer die unbedingt wieder haben will, kann sich das wieder aktivieren, deshalb wurde es nur auskommentiert und nicht gelöscht

Aber das W3C Bildchen passt eh nicht, weil das Forum innerhalb der Bridge nicht in xhtml 1.0 erscheint...

Andi · 22 März 2007, 18:09:30

So, der Rest...

Die Sache mit dem auskommentierten _PLEASEPROVIDE ist Absicht. Dieser Beschreibungstext ist unnötig und auch begrifflich falsch. Es sind ja nicht nur Kontaktinformationen, die da reingeschrieben werden sollen. Und wenn man ein Feedback geben will ist es nur logisch, dass man da auch was reinschreiben soll....

fehlendes Leerzeichen vor select >> ist geändert

Danke für deine Mithilfe