[1.17 TimeSelectorTag] Fehlerhafter Code

Das Forum ist für das Melden von Bugs gedacht. // This forum is intended to report bugs with the APF.
Gesperrt
Benutzeravatar
MrNiceGuy
Beiträge: 749
Registriert: 03.02.2009, 16:49:42
Wohnort: Nienburg / Weser

[1.17 TimeSelectorTag] Fehlerhafter Code

Beitrag von MrNiceGuy » 20.08.2013, 20:50:31

Moin!

Ich glaube ich bin da auf einen Bug gestoßen: In der Klasse TimeSelectorTag fehlt ein import des SelectBoxTags, ohne den vorher manuell einzubinden ergibt das einen Fehler - sehr unschön.

Außerdem ist mir ein Fehler in den Kommentaren aufgefallen, keine Ahnung, ob das irgend wen stört: Zeile 53: "// initialize the year range" - da wurde wohl eine andere Klasse einfach kopiert ;)

Da es sich um Version 1.17 handelt will ich da nicht einfach drin herumfuhrwerken, immerhin ist es bereits ein fertiges Release!?
There are only 10 Types of people in the world:
Those who understand binary and those who don't.

Benutzeravatar
jwlighting
Beiträge: 466
Registriert: 14.07.2010, 14:23:58
Wohnort: LK Oldenburg
Kontaktdaten:

Re: [1.17 TimeSelectorTag] Fehlerhafter Code

Beitrag von jwlighting » 20.08.2013, 21:48:20

Wäre evtl was für eine 1.17.1, die ja im Raum steht.

Magst du dazu mal einen Eintrag im Tracker anlegen?

LG :)
Jan

Menschen irren - Politiker sind Menschen.
Für den Norddeutschen ist 1kW = 2 Pfund Schlick.

jprangenberg
Beiträge: 410
Registriert: 16.08.2010, 22:14:54

Re: [1.17 TimeSelectorTag] Fehlerhafter Code

Beitrag von jprangenberg » 21.08.2013, 07:51:18

Ich bin dafür, dass Fehler auch noch in der Version 1.17 behoben werden. Wieso auch nicht?

Benutzeravatar
MrNiceGuy
Beiträge: 749
Registriert: 03.02.2009, 16:49:42
Wohnort: Nienburg / Weser

Re: [1.17 TimeSelectorTag] Fehlerhafter Code

Beitrag von MrNiceGuy » 22.08.2013, 08:47:36

Achja, der Tracker... Sorry, bin ein bisschen in der Entwicklung der com hinterher ;) sobald ich die Zeit finde mache ich das. Lösung des Problems dann direkt in den 1.17er branch commiten?

Und wegen Behebung in einer Release Version: das macht man normalerweise nicht, da sich der Inhalt oder auch die Funktionsweise ändert und dann nicht mehr der Version entspricht, die ursprünglich released wurde. Das führt zu Inkonsistenzen, deshalb lieber 1.17.1 oder 1.17 update 1 oder irgendwie Sofas. Wusste nur nicht, dass da bereits etwas geplant war.
There are only 10 Types of people in the world:
Those who understand binary and those who don't.

Benutzeravatar
dr.e.
Administrator
Beiträge: 4527
Registriert: 04.11.2007, 16:13:53

Re: [1.17 TimeSelectorTag] Fehlerhafter Code

Beitrag von dr.e. » 22.08.2013, 19:25:18

Wenn du den Fehler behebst, bitte im 2.0er-Branch. Bei Bedarf back-porten wir das dann nach 1.17. Danke dir! :)
Viele Grüße,
Christian

Benutzeravatar
MrNiceGuy
Beiträge: 749
Registriert: 03.02.2009, 16:49:42
Wohnort: Nienburg / Weser

Re: [1.17 TimeSelectorTag] Fehlerhafter Code

Beitrag von MrNiceGuy » 02.09.2013, 05:38:43

Sorry, ich komme da gerade leider nicht zu. Ist zwar nur ein Import-Befehl am Anfang, aber meine SVN-Einrichtung spinnt momentan rum, und so kurz vor Abgabe der Dokumentation für die Fortbildung schaffe ich das leider aktuell nicht mich darum zu kümmern... :(
There are only 10 Types of people in the world:
Those who understand binary and those who don't.

Benutzeravatar
dr.e.
Administrator
Beiträge: 4527
Registriert: 04.11.2007, 16:13:53

Re: [1.17 TimeSelectorTag] Fehlerhafter Code

Beitrag von dr.e. » 02.09.2013, 10:59:45

Poste mal das Delta, dann kann ich das bei Bedarf einchecken.
Viele Grüße,
Christian

Benutzeravatar
MrNiceGuy
Beiträge: 749
Registriert: 03.02.2009, 16:49:42
Wohnort: Nienburg / Weser

Re: [1.17 TimeSelectorTag] Fehlerhafter Code

Beitrag von MrNiceGuy » 13.09.2013, 15:34:03

Delta???
There are only 10 Types of people in the world:
Those who understand binary and those who don't.

Benutzeravatar
dr.e.
Administrator
Beiträge: 4527
Registriert: 04.11.2007, 16:13:53

Re: [1.17 TimeSelectorTag] Fehlerhafter Code

Beitrag von dr.e. » 13.09.2013, 23:00:06

Naja den Diff (=Unterschied) deiner Änderungen. ;)
Viele Grüße,
Christian

Gesperrt

Wer ist online?

Mitglieder in diesem Forum: 0 Mitglieder und 1 Gast