Comments on: Moving VuFind to Zend Framework 2: Part 5 — The Dreaded forward() Bug http://blog.library.villanova.edu/libtech/2012/08/13/moving-vufind-to-zend-framework-2-part-5-the-dreaded-forward-bug/ Knowledge Sharing from Falvey's Technology Team Thu, 14 Mar 2013 19:32:00 +0000 hourly 1 http://wordpress.org/?v=3.5.1 By: dkatz http://blog.library.villanova.edu/libtech/2012/08/13/moving-vufind-to-zend-framework-2-part-5-the-dreaded-forward-bug/#comment-88 dkatz Mon, 20 Aug 2012 18:10:01 +0000 http://blog.library.villanova.edu/libtech/?p=124#comment-88 Another update: this is now a non-issue; the framework has been updated so that the forward() plug-in behaves more logically. See the notes added to the top of the post for more details.

]]> By: dkatz http://blog.library.villanova.edu/libtech/2012/08/13/moving-vufind-to-zend-framework-2-part-5-the-dreaded-forward-bug/#comment-86 dkatz Tue, 14 Aug 2012 16:37:05 +0000 http://blog.library.villanova.edu/libtech/?p=124#comment-86 One little bit of followup to this. Since using the forward() plugin is quite verbose, and since all of VuFind’s forward actions follow the same basic pattern, I have created a helper method called forwardTo() in the base controller that all of VuFind’s other controllers inherit from. The new method allows this lengthy and unattractive code:

$this->forward()->dispatch(‘controller’, array(‘action’ => ‘action’));

return false;

to be simplified down to:

return $this->forwardTo(‘controller’, ‘action’);

If ZF2 eventually incorporates its own convenience method, we might consider switching to that. But for now, this makes the code more readable and gives us a centralized place where we can add extra forward-related processing if we run into a new forwarding problem similar to the one described above.

In case anyone’s interested, here are the relevant diffs:

http://vufind.git.sourceforge.net/git/gitweb.cgi?p=vufind/vufind;a=commitdiff;h=d78ecd58680ba60f6f66bd905afa2abe8af7e38d

]]>