Development

Changeset 6490

You must first sign up to be able to contribute.

Changeset 6490

Show
Ignore:
Timestamp:
12/13/07 22:44:39 (1 year ago)
Author:
fabien
Message:

removed sfEvent dependency on sfParameterHolder

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved
  • trunk/lib/command/sfCommandLogger.class.php

    r5250 r6490  
    3939  public function listenToLogEvent(sfEvent $event) 
    4040  { 
    41     $priority = $event->getParameterHolder()->remove('priority', self::INFO)
     41    $priority = isset($event['priority']) ? $event['priority'] : self::INFO
    4242 
    4343    $prefix = ''; 
     
    5050    } 
    5151 
    52     foreach ($event->getParameterHolder()->getAll() as $message) 
     52    foreach ($event->getParameters() as $message) 
    5353    { 
    5454      $this->log(sprintf('%s%s', $prefix, $message), $priority); 
  • trunk/lib/event/sfEvent.class.php

    r5016 r6490  
    1717 * @version    SVN: $Id$ 
    1818 */ 
    19 class sfEvent 
     19class sfEvent implements ArrayAccess 
    2020{ 
    2121  protected 
    22     $value           = null, 
    23     $processed       = false, 
    24     $subject         = null, 
    25     $name            = '', 
    26     $parameterHolder = null; 
     22    $value      = null, 
     23    $processed  = false, 
     24    $subject    = null, 
     25    $name       = '', 
     26    $parameters = null; 
    2727 
    2828  /** 
     
    3838    $this->name = $name; 
    3939 
    40     $this->parameterHolder = new sfParameterHolder(); 
    41     $this->parameterHolder->add($parameters); 
     40    $this->parameters = $parameters; 
    4241  } 
    4342 
     
    102101  } 
    103102 
    104   public function getParameterHolder() 
     103  public function getParameters() 
    105104  { 
    106     return $this->parameterHolder
     105    return $this->parameters
    107106  } 
    108107 
    109   public function getParameter($name, $default = null) 
     108  /** 
     109   * Returns true if the parameter exists (implements the ArrayAccess interface). 
     110   * 
     111   * @param  string  The parameter name 
     112   * 
     113   * @return Boolean true if the parameter exists, false otherwise 
     114   */ 
     115  public function offsetExists($name) 
    110116  { 
    111     return $this->parameterHolder->get($name, $default); 
     117    return isset($this->parameters[$name]); 
    112118  } 
    113119 
    114   public function hasParameter($name) 
     120  /** 
     121   * Returns a parameter value (implements the ArrayAccess interface). 
     122   * 
     123   * @param  string The parameter name 
     124   * 
     125   * @return mixed  The parameter value 
     126   */ 
     127  public function offsetGet($name) 
    115128  { 
    116     return $this->parameterHolder->has($name)
     129    return isset($this->parameters[$name]) ? $this->parameters[$name] : null
    117130  } 
    118131 
    119   public function setParameter($name, $value) 
     132  /** 
     133   * Sets a parameter (implements the ArrayAccess interface). 
     134   * 
     135   * @param string The parameter name 
     136   * @param mixed   
     137   */ 
     138  public function offsetSet($name, $value) 
    120139  { 
    121     return $this->parameterHolder->set($name, $value); 
     140    $this->parameters[$name] = $value; 
     141  } 
     142 
     143  /** 
     144   * Removes a parameter (implements the ArrayAccess interface). 
     145   * 
     146   * @param string The parameter name 
     147   */ 
     148  public function offsetUnset($name) 
     149  { 
     150    unset($this->parameters[$name]); 
    122151  } 
    123152} 
  • trunk/lib/i18n/sfI18N.class.php

    r4957 r6490  
    266266  { 
    267267    // change the message format object with the new culture 
    268     $this->setCulture($event->getParameter('culture')); 
     268    $this->setCulture($event['culture']); 
    269269  } 
    270270 
     
    278278  { 
    279279    // change message source directory to our module 
    280     $this->setMessageSource(sfLoader::getI18NDirs($event->getParameter('module'))); 
     280    $this->setMessageSource(sfLoader::getI18NDirs($event['module'])); 
    281281  } 
    282282} 
  • trunk/lib/log/sfLogger.class.php

    r5232 r6490  
    213213   * 
    214214   * @param sfEvent An sfEvent instance 
    215    * 
    216215   */ 
    217216  public function listenToLogEvent(sfEvent $event) 
    218217  { 
    219     $priority = $event->getParameterHolder()->remove('priority', self::INFO)
     218    $priority = isset($event['priority']) ? $event['priority'] : self::INFO
    220219    $subject  = $event->getSubject(); 
    221220    $subject  = is_object($subject) ? get_class($subject) : (is_string($subject) ? $subject : 'main'); 
    222     foreach ($event->getParameterHolder()->getAll() as $message) 
     221    foreach ($event->getParameters() as $message) 
    223222    { 
    224223      $this->log(sprintf('{%s} %s', $subject, $message), $priority); 
  • trunk/lib/plugin/sfSymfonyPluginManager.class.php

    r5250 r6490  
    110110  public function ListenToPluginPostInstall($event) 
    111111  { 
    112     $this->installWebContent($event->getParameter('plugin')); 
     112    $this->installWebContent($event['plugin']); 
    113113  } 
    114114 
     
    120120  public function ListenToPluginPostUninstall($event) 
    121121  { 
    122     $this->uninstallWebContent($event->getParameter('plugin')); 
     122    $this->uninstallWebContent($event['plugin']); 
    123123  } 
    124124 
  • trunk/lib/response/sfWebResponse.class.php

    r5735 r6490  
    709709  public function listenToChangeActionEvent(sfEvent $event) 
    710710  { 
    711     $moduleName = $event->getParameter('module')
    712     $actionName = $event->getParameter('action')
     711    $moduleName = $event['module']
     712    $actionName = $event['action']
    713713 
    714714    if ($moduleName == sfConfig::get('sf_error_404_module') && $actionName == sfConfig::get('sf_error_404_action')) 
  • trunk/lib/routing/sfRouting.class.php

    r4961 r6490  
    162162  { 
    163163    // change the culture in the routing default parameters 
    164     $this->setDefaultParameter('sf_culture', $event->getParameter('culture')); 
     164    $this->setDefaultParameter('sf_culture', $event['culture']); 
    165165  } 
    166166 
     
    173173  public function listenToLoadParametersInfoEvent(sfEvent $event, $parameters) 
    174174  { 
    175     return array_merge($parameters, $this->parse($event->getParameter('path_info'))); 
     175    return array_merge($parameters, $this->parse($event['path_info'])); 
    176176  } 
    177177 
  • trunk/test/unit/sfEventDispatcherTest.class.php

    r4951 r6490  
    4141  static public function newMethod(sfEvent $event) 
    4242  { 
    43     if ($event->getParameter('method') == 'newMethod') 
     43    if ($event['method'] == 'newMethod') 
    4444    { 
    45       $arguments = $event->getParameter('arguments')
     45      $arguments = $event['arguments']
    4646      $event->setReturnValue($arguments[0]); 
    4747