Dependency Graph | [ View Issue ] [ Relation Graph ] [ Horizontal ] | |||
|
||||
|
View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0000164 | Branch 0.2.1-FINAL | Bug | public | 2009-12-08 00:37 | 2016-11-23 11:28 |
Reporter | user13 | ||||
Assigned To | user13 | ||||
Priority | normal | Severity | tweak | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Platform | Apache 2.2.x - mod_php 5.2.x | OS | Windows Vista Ultimate x64 SP1 | OS Version | Vista with SP1 |
Product Version | 0.2.1-FINAL | ||||
Target Version | 0.2.1-FINAL | Fixed in Version | 0.2.1-FINAL | ||
Summary | 0000164: Fehler in searchDirsRecursive() => Falsche Rev.-Ermittlung | ||||
Description | Bedingt durch ein unerwartetes Verhalten von getArrayFromDirectory() werden nicht alle Dateien nach den Rev-Tag durchsucht. Ich glaube ich mappe alle gefundenen Verzeichnisse in ein Cache-Array rein, damit es dann schneller geht. | ||||
Tags | No tags attached. | ||||
Attached Files |
|
Notes | |
Quix0r (administrator) 2009-12-08 00:41 |
Kannst du bitte auch Projection, ETA und 'Product Build' aktualisieren? So pflege ich die Tickets. :) SVN-HEAD reicht als Product-Build bei dir aus, hast ja meistens immer HEAD. ;) |
2009-12-09 03:22 |
So habs noch nicht ganz fertig gefixt aber ich werke noch daran. |
2009-12-09 03:33 |
Quix0r kannst du mal prüfen ob ich da einen Fehler gemacht habe? Nach einer Neuinstallation wollte ich sql_patches registrieren. Da hats dann einen Fehler gegeben, dass es CURR_SVN_REVISION nicht von getConfig-her nicht finden konnte. Ich habe den Cache geleert, dann konnte es irgend eine andere template-variable von getConfig-her nicht finden. Kann es sein dass da irgendwie eine Race-Condition mit dem Config-System und der registrierung von sql-patches vorhanden ist? |
Quix0r (administrator) 2009-12-09 22:00 |
Oder wollen wir nicht eher den ganzen Revisonskram rauswerfen, wenn der so viele Probleme macht? Nur dass ein Webmaster aka. Mailtauschbetreiber nicht die Finger krum machen will und die Revisionsnummer z.B. mit Hilfe von TortoiseSVN holen koennte, haben wir nun so viele Probleme und muessen die gesamte Verzeichnisstruktur einscannen, was viel Performance kostet? Ich glaube, PHP bietet hier wo eine Extension, mit der wir auch Revisionsinformationen uns holen koennen. |
Quix0r (administrator) 2009-12-09 22:01 |
Meiner Meinung nach sollten wir uns um das Wesendliche des Mailtausches kuemmern und nicht, welche Revision gerade beim User drauf ist. Die sehen die ohnehin, wenn die auschecken bzw. aktualisieren. :) |
Quix0r (administrator) 2009-12-15 16:58 |
Ich schaue mir aber dennoch deinen Patch an. Irgentwie kann ich mich dennoch nicht davon trennen. :( |
Quix0r (administrator) 2009-12-15 18:44 |
Ich hab den Bug jetzt selber finden koennen, ist aber nicht leicht zu erklaeren und nur durch viele logDebugMessage()-Zeilen findbar. :) Erstmal hattest du in searchDirsRecursive() die Funktion getArrayFromDirectory() mit zwei vertauschten Parametern aufgerufen und zudem selbst rekursiv gesucht, was nicht noetig ist, da getArrayFromDirectory() schon recursiv sucht - ausser du willst es nicht. Dann hattest du immer nach der zuletzt geaenderte Datei gesucht, was beim Entwicklen dazu fuehrte, dass nur die korrekte Revisonsnummer angezeigt wurde, wenn du die Datei als letzttes editiert hast, was selten vorkommt. searchDirsRecursive() habe ich nun so modifiziert, dass es nach neueren Dateien suchen kann und dabei die Revisionsnummer verwenden kann. Standart ist noch immer 'Date' (was vielleicht klappt?). Dann hattest du in getActualVersion() ein [0] beim Verwenden des Cache-Arrays $GLOBALS['cache_array']['revision'] verwendet, was nicht noetig ist. Ist alles beim naechsten Update mit dabei. Dein Patch in deiner Branch habe ich mir auch angesehen und bemerkt, dass du nicht gekapselst hast, sowie gute Funktionen wie getArrayFromDirectory() nicht mehr verwendet hattest (war irgentwo der Fall). Ich hoffe, du bist dann mit meinem (kommenden!) Commit zufrieden. :) |
Quix0r (administrator) 2009-12-15 18:53 |
Vergiss das mit [0], ist wohl doch wegen des Caches noetig. :( |
Quix0r (administrator) 2010-06-23 15:55 |
Ich denke, dies sollte bereits gefixt sein. Sollte es wieder vorkommen, bitte neues Ticket aufmachen, da der Code starken Aenderungen unterliegt. |
Quix0r (administrator) 2016-11-23 11:28 |
Und zu damit. |
Issue History | |||
Date Modified | Username | Field | Change |
---|---|---|---|
2009-12-08 00:37 |
|
New Issue | |
2009-12-08 00:37 |
|
Status | new => assigned |
2009-12-08 00:37 |
|
Assigned To | => user13 |
2009-12-08 00:41 | Quix0r | Note Added: 0000410 | |
2009-12-08 23:19 |
|
Projection | none => minor fix |
2009-12-08 23:19 |
|
ETA | none => 2-3 days |
2009-12-08 23:19 |
|
Build | => SVN-HEAD |
2009-12-09 03:22 |
|
Note Added: 0000411 | |
2009-12-09 03:33 |
|
Note Added: 0000412 | |
2009-12-09 22:00 | Quix0r | Note Added: 0000413 | |
2009-12-09 22:01 | Quix0r | Note Added: 0000414 | |
2009-12-15 16:58 | Quix0r | Note Added: 0000420 | |
2009-12-15 18:44 | Quix0r | Note Added: 0000421 | |
2009-12-15 18:53 | Quix0r | Note Added: 0000422 | |
2010-05-28 18:27 | Quix0r | Build | SVN-HEAD => 0.2.1-FINAL |
2010-05-28 18:36 | Quix0r | Product Version | => 0.2.1-FINAL |
2010-06-23 15:55 | Quix0r | Note Added: 0000462 | |
2010-06-23 15:55 | Quix0r | Status | assigned => resolved |
2010-06-23 15:55 | Quix0r | Fixed in Version | => 0.2.1-FINAL |
2010-06-23 15:55 | Quix0r | Resolution | open => fixed |
2016-11-23 11:28 | Quix0r | Note Added: 0000797 | |
2016-11-23 11:28 | Quix0r | Status | resolved => closed |