search

gnuheidix / api.piratenpartei-bw.de

Ein API, um auf freie Inhalte der Piratenpartei Deutschland zugreifen zu können.
http://api.piratenpartei-bw.de/
8 stars 2 forks source link

Review #6

Closed gnuheidix closed 12 years ago

gnuheidix commented 12 years ago

Es ist ein Kurzreview des Konzepts und des Codes durch mindestens drei Software-Entwickler durchzuführen, um mögliche bisher unerkannte Fehler und zukünftig notwendige Features besser identifizieren zu können.

gnuheidix commented 12 years ago

Nahezu die gesamte Datenzugriffslogik, die momentan in Controllern ist, gehört imho in die Modelle. Diese Aufräumerei gehe ich demnächst an.

gnuheidix commented 12 years ago

Die Datenzugriffe sind nun in den Modellen gekapselt und die Controller verschlankt. So dürfte es nun viel besser sein. :-)

gnuheidix commented 12 years ago

Ich habe im AppModel zwei Methoden fürs Locking gebaut. Diese benötige ich, um beim Extrahieren von Bilddateien auf einer gerade zum Updaten geladenen Wikiseite zu verhindern, dass mögliche andere Threads den gleichen Prozess durchlaufen und am Ende nicht mehr referenzierte Bilddateien auf der Platte liegen, weil die Prozesse die referenzierten Dateien immer wieder anders setzen. Caching ist ohne Page-Image-Mappingtabelle umgesetzt, was bedeutet, dass gleiche Bilder mehrfach gespeichert werden können. Imho ist das aber vorerst nicht so tragisch.

Bitte mal die Geschichten reviewen, danke. :-)

gnuheidix commented 12 years ago

@KnairdA was meinst Du. Sind wir bereit fürs erste Release, wenn die beiden letzten Bugs aus den Issues raus sind? Was würdest Du noch anders machen?

gnuheidix commented 12 years ago

Ich schalte den Release einfach mal frei. Im nächsten können wir ja weiter dran herumfeilen.

KnairdA commented 12 years ago

Bin jetzt dazu gekommen mir die aktuelle Version auszuchecken und anzuschauen. Habe nichts gefunden was man fürs erste Release noch anders hätte machen müssen - tolle Arbeit :)

gnuheidix commented 12 years ago

Klasse, dankeschön :-)