pragmaMx Support Forum

pragmaMx => Allgemeines => Thema gestartet von: keiang am 09 März 2007, 14:59:19

Titel: kleinere Code Fehler gefunden
Beitrag von: keiang am 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> -&gt;" . ($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.
Titel: Re: kleinere Code Fehler gefunden
Beitrag von: keiang am 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\">&nbsp;&nbsp;"
neu:         . _SEARCHONTOPIC . ": <input type=\"text\" name=\"query\" size=\"30\">&nbsp;&nbsp;"
Titel: Re: kleinere Code Fehler gefunden
Beitrag von: Andi am 12 März 2007, 09:53:46
Hi :)

vielen Dank für die Hinweise  :thumbup: :thumbup:


Habe alle Vorschläge eingearbeitet, updates kommen....
Titel: Re: kleinere Code Fehler gefunden
Beitrag von: keiang am 12 März 2007, 12:38:37
Search Modul
Datei: "index.php"
Zeile: 181
alt:      <input type="radio" name="type" value="stories"' . (($type == 'stories') ? 'checked' : '') . '> ' . _SSTORIES . '
neu:    <input type="radio" name="type" value="stories"' . (($type == 'stories') ? ' checked' : '') . '> ' . _SSTORIES . '

( Leerzeichen fehlt vor checked )
Titel: Re: kleinere Code Fehler gefunden
Beitrag von: keiang am 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");
Titel: Re: kleinere Code Fehler gefunden
Beitrag von: Andi am 12 März 2007, 23:30:44
Jop, weiter so, gibs uns ....  :puzzled: :gruebel: :thumbup: :thumbup: :)
Titel: Re: kleinere Code Fehler gefunden
Beitrag 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?
Titel: Re: kleinere Code Fehler gefunden
Beitrag von: keiang am 21 März 2007, 16:16:06
Feedback Modul

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

alt:     
Zitat. '<!-- <p class="content">' . _PLEASEPROVIDE . '</p> -->'
. '<p align="right">' . $req . '&nbsp;' . _INDICATESREQ . '</p>';

neu:     
Zitat. '<p class="content" align="center">' . _PLEASEPROVIDE . '&nbsp;&nbsp;' . $req . '&nbsp;' . _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.");

Titel: Re: kleinere Code Fehler gefunden
Beitrag von: Andi am 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...
Titel: Re: kleinere Code Fehler gefunden
Beitrag von: Andi am 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  :thumbup: :thumbup: