[dev] [commits] Horde branch master updated. 40316ec5290525a65065aa6edc3d3c166c21dff5

Gunnar Wrobel wrobel at horde.org
Mon Aug 8 05:34:52 UTC 2011


Quoting "Michael J. Rubinsky" <mrubinsk at horde.org>:

> The branch "master" has been updated.
> The following is a summary of the commits.
>
> from: 076904da083ba705201bb5464baed149571a0df2
>
> 40316ec Add Horde_Routes_Matcher
>
> -----------------------------------------------------------------------
>
> commit 40316ec5290525a65065aa6edc3d3c166c21dff5
> Author: Michael J Rubinsky <mrubinsk at horde.org>
> Date:   Sat Aug 6 11:58:45 2011 -0400
>
>     Add Horde_Routes_Matcher
>
>     Helper class for managing match dictionaries. Avoids having to
>     match the route multiple times in a MVC environment.
>     Thanks Gunnar :)
>
>  framework/Routes/lib/Horde/Routes/Matcher.php |   88  
> +++++++++++++++++++++++++
>  framework/Routes/package.xml                  |   20 +++---
>  2 files changed, 99 insertions(+), 9 deletions(-)
>  create mode 100644 framework/Routes/lib/Horde/Routes/Matcher.php
>
> http://git.horde.org/horde-git/-/commit/40316ec5290525a65065aa6edc3d3c166c21dff5

Looks good. I just wondered if this adds an optional dependency on  
Horde_Controller in Horde_Routes as the package is now at least aware  
of the Request class from the Controller package? Might alternatively  
be bridged by an interface declaration in Horde_Routes that just  
defined getPath() or some such.

Cheers,

Gunnar

>
>
> __
> commits mailing list
> Frequently Asked Questions: http://horde.org/faq/
> To unsubscribe, mail: commits-unsubscribe at lists.horde.org

-- 
Core Developer
The Horde Project

e: wrobel at horde.org
t: +49 700 6245 0000
w: http://www.horde.org

pgp: 9703 43BE
tweets: http://twitter.com/pardus_de
blog: http://log.pardus.de



More information about the dev mailing list