Any feedback would be appreciated.

The script retrieves images from flickr based on the url a user submits.

Here it is working: http://aotf.com/gallery

* view/scripts/gallery/add.phtml - page url gets submitted with
http://pastebin.com/YQ2PMq3u

* view/scripts/gallery/index.phtml - page for viewing pictures already in
the db
http://pastebin.com/XeAEEe2i

* controllers/Gallery.php - code that retrieves data from user or sends data
to user
http://pastebin.com/513qW9qH

* controllers/helper/Gallery.php - code that validates and retrieves flickr
specific data
http://pastebin.com/MCMwGkkF

* models/DbTable/Gallery.php - code that inserts data retrieved from flickr
into the db
http://pastebin.com/UPJL2etA

* view/scripts/flickrItem.phtml - code that gets called by a partial to
display flickr image
http://pastebin.com/qQsSenF2

Some general questions.

It seems odd that I would have to retrieve the database resource inside of
the extended ZendDbTable_Abstract class and not have it locally:

$db =
ZendControllerFront::getInstance()->getParam('bootstrap')->getResource('db');
$db->query($str);

It feels odd to have the Gallery helper not be able to access the view
directly and set error messages. It would be nice to set an error message
and return false to the controller. http://pastebin.com/MCMwGkkF

In the Gallery model, I do a bulk insert that is proprietary to mysql. Is
there something similar available that could be done in the framework that
is not proprietary? I don't want to loop over 100+ queries.
http://pastebin.com/UPJL2etA
-- 
View this message in context: 
http://zend-framework-community.634137.n4.nabble.com/code-review-of-a-image-gallery-tp2275768p2275768.html
Sent from the Zend Framework mailing list archive at Nabble.com.

Reply via email to