MyXoops Forum

Aw:: Modul-Versionsnummern??? [Bug-Report für XOOPS 2.3] - myXOOPS German Support

frankblack

"myXOOPs"-Profi
Gepostet am:24.03.2009 11:14
frankblack
frankblack Offline (Show more)
"myXOOPs"-Profi
Posts: 747
Since: 19.04.2006
#9

Aw:: Modul-Versionsnummern???

Alles klar! Aber einen habe ich noch: themes / zetagenesis / content.css Zeile 315. Da ist der Punkt vom 25em weggerückt.

... is gone with the wind ...

alfred

Administrator
Gepostet am:24.03.2009 11:06
alfred
alfred Offline (Show more)
Administrator
Posts: 7619
Since: 06.12.2004
#8

Aw:: Modul-Versionsnummern???

Zitat:

frankblack schrieb:
Hab da noch ein paar Korinthen! Da schwirren im Code ein paar Semikola zu viel herum. Könnte man doch glatt 9 Byte am Code einsparen.
...

Reicht das oder soll ich noch mehr suchen? LOL


Nein, weitermachen.

Zitat:

EDIT: ??? getVar->$version sollte doch eigentlich nur die Versionsnummer aus der xoops_version.php popeln? Also wird aus einem String eine Zahl gemacht? Komisch...


nein, so wirds aus der DB (integer) ausgelesen.
aus der xoops_version.php kommt das mit ner anderen Funktion (getinfo)

Advertisement

Gepostet am:0
Advertisement (Show more)
Posts: 0
Since: 0


Hier könnten Sie Ihre Anzeige aufgeben!
Bitte kontaktieren Sie uns, um mehr darüber zu erfahren.

frankblack

"myXOOPs"-Profi
Gepostet am:24.03.2009 11:01
frankblack
frankblack Offline (Show more)
"myXOOPs"-Profi
Posts: 747
Since: 19.04.2006
#7

Aw:: Modul-Versionsnummern???

Hab da noch ein paar Korinthen! Da schwirren im Code ein paar Semikola zu viel herum. Könnte man doch glatt 9 Byte am Code einsparen.

class / textsanitizer / syntaxhighlight / geshi.php Zeile 2493

xoopseditor / tinymce in den CSS-Dateien für die Plugins xoopsemotions und xoopsmlcontent jeweils Zeile 7

modules / profile / admin / step.php Zeilen 37 und 44

modules / system / admin / comments / admin_header.php Zeile 36

modules / system / class / gui / default / css / style.css Zeile 111

themes / zetagenesis / css / content.css Zeile 321
themes / zetagenesis / css / xoopseditor.css Zeile 4

Reicht das oder soll ich noch mehr suchen? LOL

EDIT: ??? getVar->$version sollte doch eigentlich nur die Versionsnummer aus der xoops_version.php popeln? Also wird aus einem String eine Zahl gemacht? Komisch...
... is gone with the wind ...

alfred

Administrator
Gepostet am:24.03.2009 10:58
alfred
alfred Offline (Show more)
Administrator
Posts: 7619
Since: 06.12.2004
#6

Aw:: Modul-Versionsnummern???

Zitat:

frankblack schrieb:
Es geht mir ja auch nicht um die Kosmetik. $module->getVar('version') liest aus den Modulen doch die Versionsnummer aus? Sollte das nicht dann schon reichen? Warum muss man da noch eine Funktion um die Variable "herumlegen"?


nöö, reicht eben nicht, weil dann nur 163 stehen würde
wenn es nur ganzzahlige versionen geben würde, wäre das einfacher.

Zitat:

Ich kack ja gerne Korinthen!

Klare Regeln setzen, wie in der Kindererziehung.


frankblack

"myXOOPs"-Profi
Gepostet am:24.03.2009 10:38
frankblack
frankblack Offline (Show more)
"myXOOPs"-Profi
Posts: 747
Since: 19.04.2006
#5

Aw:: Modul-Versionsnummern???

Es geht mir ja auch nicht um die Kosmetik. $module->getVar('version') liest aus den Modulen doch die Versionsnummer aus? Sollte das nicht dann schon reichen? Warum muss man da noch eine Funktion um die Variable "herumlegen"? Wer Versionsnummern in seinen Modulen á la "debaser 3#+1_die Version Ist supi" verwendet, sollte dann doch lieber ins kleine Modulentwicklerhandbuch schauen?

Ich kack ja gerne Korinthen!

Klare Regeln setzen, wie in der Kindererziehung.
... is gone with the wind ...

alfred

Administrator
Gepostet am:24.03.2009 10:25
alfred
alfred Offline (Show more)
Administrator
Posts: 7619
Since: 06.12.2004
#4

Aw:: Modul-Versionsnummern???

mh, welche Funktion wandelt das 'extra' um ?

Du kannst in der /modules/system/admin/modulesadmin/modulesadmin.php Zeile 66, das

round($module->getVar('version') / 100 , 2)

mit

number_format(($module->getVar('version') / 100) , 2)

ersetzen, dann ist der Punkt wieder da

frankblack

"myXOOPs"-Profi
Gepostet am:24.03.2009 09:34
frankblack
frankblack Offline (Show more)
"myXOOPs"-Profi
Posts: 747
Since: 19.04.2006
#3

Aw:: Modul-Versionsnummern???

Ist aber unnötig, da in den Modulen die Punktnotation genutzt wird. Es wird also extra eine Funktion darauf verplempert aus dem Punkt ein Komma zu machen. Das hätte man sich doch eigentlich sparen können?!?
... is gone with the wind ...

Micha578

"myXOOPs"-Könner
Gepostet am:24.03.2009 09:31
Micha578
Micha578 Offline (Show more)
"myXOOPs"-Könner
Posts: 426
Since: 02.07.2007
#2

Aw:: Modul-Versionsnummern???

Das ist mir auch schon aufgefallen auf meiner Live Seite... Kann ich aber mit leben
Mfg, Micha

frankblack

"myXOOPs"-Profi
Gepostet am:24.03.2009 09:28
frankblack
frankblack Offline (Show more)
"myXOOPs"-Profi
Posts: 747
Since: 19.04.2006
#1

Modul-Versionsnummern???

Witzig! Ist mir noch gar nicht so aufgefallen, vielleicht liegt es auch am Mac/XAMPP wo ich die Testversion von 2.3.3 installiert habe: Die Modulversions-Nummern sind jetzt mit einem Komma versehen. Userprofile 1,52?

Ist zwar nur kosmetische Frage, aber Versionsnummern werden doch IMMER mit einem Punkt abgetrennt? Wie gesagt, vielleicht ist es auch nur beim Mac so? Das XAMPP für Mac hat eh ne Meise. In der Templatesadministration wird auch nicht mehr das Datum angezeigt, sondern nur die Unix-Timestamp-Zahl.
... is gone with the wind ...
Diesen Thread durchsuchen:  1 Anonyme Benutzer

Aktuell aus dem Forum

Forum Thema Antworten Views Letzter Beitrag
Modulentwicklung New Xoops Xcreate Module 0 4208 26.11.2025 15:22
efkan Gehe zum letzten Beitrag
Modulentwicklung XOOPS Custom Field Module 1 1586 24.11.2025 11:21
Goffy Gehe zum letzten Beitrag