[cvs] [Wiki] changed: MichaelRubinsky

[Michael Rubinsky]

mrubinsk  Sun, 30 May 2010 19:59:05 -0400

Modified page: http://wiki.horde.org/MichaelRubinsky
New Revision:  1.29
Change log:  remove some things that are done, add some stuff

@@ -10,15 +10,16 @@
  * PHP5 native
  * rewrite tagger to use content_tagger
  * use pretty autocompleter for tags / update the image view and  
review placement of tag widget.
  * Use a decorator pattern for the different gallery display modes  
(Date/Normal/?)
-* Use custom binders/factories for the various global objects such as  
ansel_storage, cache etc... use a null cache driver instead of  
checking for the cache existence.
+* Deal with Ansel_Storage being able to specify different scopes and  
how that affects injector usage.
  * Rewrite google maps integration to use horde_map javascript
  * Add a way to easily perform image actions such as downloading from  
the gallery view...something along the lines of either a small  
contextual menu or simply a link on the image tile, also a textual  
link to display the full image view would be very helpful for  
galleries using a lightbox - so the user could see the exif  
data/comments/image actions etc...
+* Implement a more MVC-ish way of handling the different  
gallery/image views. Instead of checking for api usage in all the  
views, use a dedicated view. This will make it easier for others to  
implement their own specialized view?
+* Clean up the very long parameter lists on most of the list methods  
- take parameter arrays instead of a dozen different paramters.

  ++++Random notes
-Change Ansel_Storage::listImages() to accept a parameter array  
instead of the gallery_id and the WHERE parameter. In addition, .need  
to refactor usage of this method so that client code doesn't have to  
know anything about the SQL table structure. i.e.  get rid of the  
WHERE parameter. All non Ansel_Gallery code should be calling  
Ansel_Gallery::listImages and not Ansel_Stroage::listImages:
-
+All non Ansel_Gallery code should be calling  
Ansel_Gallery::listImages and not Ansel_Stroage::listImages:
  list* methods should return ids, get* methods should return objects.
  Review all ansel js, output all strings in js and consolidate all  
the gallery_view js into a single *.js file.

  --