[commits] Horde branch master updated. 3eb04cb5d8972b124aed95ed568d056c8e980dc3

Michael M Slusarz slusarz at horde.org
Mon Feb 6 22:38:21 UTC 2012


The branch "master" has been updated.
The following is a summary of the commits.

from: bb6588af2902fa116ab0e46dd9ef8929a871919d

4686006 Fix phpdoc
3eb04cb Add Horde_Core_Alarm_Handler_Notify class

-----------------------------------------------------------------------

commit 4686006122184c20e284f70b1e9d72514806667b
Author: Michael M Slusarz <slusarz at horde.org>
Date:   Mon Feb 6 15:22:21 2012 -0700

    Fix phpdoc

 framework/Alarm/lib/Horde/Alarm/Handler/Notify.php |   10 ++++++----
 1 files changed, 6 insertions(+), 4 deletions(-)

http://git.horde.org/horde-git/-/commit/4686006122184c20e284f70b1e9d72514806667b

-----------------------------------------------------------------------

commit 3eb04cb5d8972b124aed95ed568d056c8e980dc3
Author: Michael M Slusarz <slusarz at horde.org>
Date:   Mon Feb 6 15:32:05 2012 -0700

    Add Horde_Core_Alarm_Handler_Notify class
    
    The injector passed to the Alarm Notify class requires that create()
    have no arguments.  However, the factory class requires that create()
    have a Horde_Injector argument.
    
    The alternative is to create a wrapper object that wraps the create()
    method and adds the argument.  But at that point, we might as well
    create a local Horde_Core driver.
    
    Wondering if we should do this for the Mail and Desktop drivers also.

 framework/Core/lib/Horde/Core/Alarm/Handler/Notify.php |   79 ++++++++++++++++
 framework/Core/lib/Horde/Core/Factory/Alarm.php        |    8 +-
 framework/Core/package.xml                             |   12 ++-
 3 files changed, 91 insertions(+), 8 deletions(-)
 create mode 100644 framework/Core/lib/Horde/Core/Alarm/Handler/Notify.php

http://git.horde.org/horde-git/-/commit/3eb04cb5d8972b124aed95ed568d056c8e980dc3




More information about the commits mailing list