tidal / PEIP

src/Base/GenericBuilder.php

Doc Comments +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
      * 
      * @access public
      * @param string $className class-name to create objects for
-     * @param ReflectionClass $reflectionClass reflection-class for class. default: NULL 
+     * @param \ReflectionClass $reflectionClass reflection-class for class. default: NULL 
      * @param boolean $storeRef wether to store a reference for new instance. default: true 
      * @return 
      */

src/Event/EventBuilder.php

Doc Comments -1 removed lines

@@ -83,7 +83,6 @@
      * @param ObjectEventDispatcher $dispatcher the dispatcher to dispatch the event 
      * @param mixed $subject the subject for the event
      * @param string $name the name of the event
-     * @param array $parameters the headers for the event
      * @return 
      */
     public function buildAndDispatch(ObjectEventDispatcher $dispatcher, $subject, $name, array $headers = array()){

src/Util/Test.php

Braces +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
             class_exists($className);
             $res =  ReflectionPool::getInstance($className)
                 ->implementsInterface($interfaceName);            
-        }  catch (\Exception $e){
+        } catch (\Exception $e){
             $res = false; 
         }
         return $res;;

src/ABS/Base/Connectable.php

Doc Comments +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
     
     /**
      * @access public
-     * @param $name
+     * @param string $name
      * @param Callable|PEIP\INF\Handler\Handler $listener
      * @return
      */

src/ABS/Dispatcher/MapDispatcher.php

Doc Comments -1 removed lines

@@ -80,7 +80,6 @@
      *
      * @access public
      * @param string $name name of the event
-     * @param Callable|PEIP\INF\Handler\Handler $listener listener to connect
      * @return
      */
     public function disconnectAll($name){

src/ABS/Handler/MessageHandler.php

Doc Comments +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
      * @see PEIP\ABS\Handler\MessageHandler::doSetInputChannel
      * @access public
      * @param \PEIP\INF\Channel\Channel $inputChannel the input-channel
-     * @return PEIP\ABS\Handler\MessageHandler $this;
+     * @return MessageHandler $this;
      */
     public function setInputChannel(\PEIP\INF\Channel\Channel $inputChannel){
         $this->doSetInputChannel($inputChannel);

Braces +1 added lines, -1 removed lines

@@ -77,7 +77,7 @@
         $this->inputChannel = $inputChannel;    
         if($this->inputChannel instanceof \PEIP\INF\Channel\SubscribableChannel){
                 $this->inputChannel->subscribe($this);
-        }else{          
+        } else{          
             $this->unwrapEvents = true;
             $this->inputChannel->connect('postSend', $this);
         }  

src/ABS/Handler/ReplyProducingMessageHandler.php

Doc Comments +3 added lines, -3 removed lines

@@ -51,7 +51,7 @@ 
      * Connects the handler to the output-channel. 
      * 
      * @access protected
-     * @param \PEIP\INF\Channel\Channel $inputChannel the output-channel to connect the handler to
+     * @param \PEIP\INF\Channel\Channel $outputChannel the output-channel to connect the handler to
      * @return 
      */
     protected function doSetOutputChannel(\PEIP\INF\Channel\Channel $outputChannel){
@@ -160,7 +160,7 @@ 
      * 
      * @access public
      * @param string $messageClass name of the message-class to create reply-messages from. 
-     * @return PEIP\ABS\Handler\ReplyProducingMessageHandler $this
+     * @return ReplyProducingMessageHandler $this
      */
     public function setMessageClass($messageClass){
         $this->messageClass = $messageClass;
@@ -193,7 +193,7 @@ 
      * 
      * @access public
      * @param array $headerNames array of message-header names to look for a reply-channel
-     * @return PEIP\ABS\Handler\ReplyProducingMessageHandler $this
+     * @return ReplyProducingMessageHandler $this
      */
     public function setReplyChannelHeaders(array $headerNames){
         $this->replyChannelHeaders = $headerNames;

src/Context/XMLContext.php

Doc Comments +11 added lines, -7 removed lines

@@ -135,7 +135,7 @@ 
      * 
      * @implements \PEIP\INF\Context\Context
      * @access public
-     * @param \PEIP\INF\Context\Context_Plugin $plugin a plugin instance
+     * @param \PEIP\INF\Context\ContextPlugin $plugin a plugin instance
      */
     public function addPlugin(\PEIP\INF\Context\ContextPlugin $plugin){ 
         $plugin->init($this);   
@@ -167,7 +167,6 @@ 
      * </config>     
      * 
      * @access public
-     * @param XMLContext $config the config to include
      */    
     public function includeContext(XMLContext $context){
         $this->services = array_merge($this->services, $context->getServices());
@@ -178,7 +177,7 @@ 
      * 
      * @see XMLContext::includeContext
      * @access public
-     * @param XMLContext $context the config to include
+     * @param string $filePath
      */    
     public function includeContextFromFile($filePath){
         if(file_exists($filePath)){
@@ -357,7 +356,7 @@ 
      * 
      * @access public
      * @param $config 
-     * @return object the initialized service instance
+     * @return \PEIP\INF\Context\ContextPlugin the initialized service instance
      */
     public function createService($config){
         return ServiceFactory::createService($config);      
@@ -685,7 +684,7 @@ 
      * 
      * @access protected
      * @param object $config configuration object to create arguments from.  
-     * @return mixed build arguments 
+     * @return \PEIP\INF\Channel\Channel[] build arguments 
      */
     protected function getReplyHandlerArguments($config){
         $args = array(
@@ -730,7 +729,8 @@ 
      * 
      * @access protected
      * @param string the configuration type ofthe channel (e.g.: 'reply', 'request')
-     * @param object $config configuration object to return channel from. 
+     * @param object $config configuration object to return channel from.
+     * @param string $type 
      * @return \PEIP\INF\Channel\Channel reply-channel
      */
     public function doGetChannel($type, $config){
@@ -793,7 +793,7 @@ 
      * @see GenericBuilder
      * @static
      * @access protected
-     * @param object $className name of class to build instance for. 
+     * @param string $className name of class to build instance for. 
      * @param array $arguments arguments for the constructor 
      * @return object build and modified srvice instance
      */     
@@ -801,6 +801,10 @@ 
         return GenericBuilder::getInstance($className)->build($arguments);
     }
 
+    /**
+     * @param string $type
+     * @param string $name
+     */
     protected static function hasPublicProperty($service, $type, $name){
         $reflection = GenericBuilder::getInstance(get_class($service))->getReflectionClass();
         if($reflection->{'has'.$type}($name) && $reflection->{'get'.$type}($name)->isPublic()){

Braces +13 added lines, -13 removed lines

@@ -94,7 +94,7 @@ 
     public static function createFromFile($file){
         if(file_exists($file)){
             return self::createFromString(file_get_contents($file));
-        }else{
+        } else{
             throw new \RuntimeException('Cannot open file  "'.$file.'".');
         }
     }
@@ -385,7 +385,7 @@ 
                     $setter = self::getSetter($property);               
                     if($setter &&  self::hasPublicProperty($service, 'Method', $setter)){
                         $service->{$setter}($arg);  
-                    }elseif(in_array($property, self::hasPublicProperty($service, 'Property', $setter))){
+                    } elseif(in_array($property, self::hasPublicProperty($service, 'Property', $setter))){
                         $service->$setter = $arg;
                     }                   
                 }
@@ -618,7 +618,7 @@ 
         $ref = trim((string)$config['ref']);
         if($ref != ''){
             $service = $this->getService($ref); 
-        }else{
+        } else{
             $service = $this->createService($config);
         }
         return $service;
@@ -638,7 +638,7 @@ 
     protected static function getSetter($config){
         if($config['setter']){
             $setter = (string)$config['setter'];
-        }elseif($config['name']){
+        } elseif($config['name']){
             $setter = 'set'.ucfirst((string)$config['name']);   
         }
         return $setter;     
@@ -654,26 +654,26 @@ 
     protected function buildArg($config){ 
         if(trim((string)$config['value']) != ''){
             $arg = (string)$config['value'];
-        }elseif($config->getName() == 'value'){
+        } elseif($config->getName() == 'value'){
             $arg = (string)$config;
-        }elseif($config->getName() == 'list'){
+        } elseif($config->getName() == 'list'){
             $arg = array();
             foreach($config->children() as $entry){ 
                 if($entry->getName() == 'value'){
                     if($entry['key']){
                         $arg[(string)$entry['key']] = (string)$entry;   
-                    }else{
+                    } else{
                         $arg[] = (string)$entry;
                     }
-                }elseif($entry->getName() == 'service'){
+                } elseif($entry->getName() == 'service'){
                     $arg[] = $this->provideService($entry);
                 }
             }
-        }elseif($config->getName() == 'service'){
+        } elseif($config->getName() == 'service'){
             $arg = $this->provideService($config);
-        }elseif($config->list){
+        } elseif($config->list){
             $arg = $this->buildArg($config->list);
-        }elseif($config->service){
+        } elseif($config->service){
             $arg = $this->buildArg($config->service);
         } 
         return $arg; 
@@ -774,10 +774,10 @@ 
                 $constructor = (string)$config["constructor"];
         if($constructor != ''){
             $service = call_user_func_array(array($cls, $constructor), $arguments); 
-        }else{
+        } else{
             $service = self::build($cls, $arguments); 
         }        
-            }catch(\Exception $e){
+            } catch(\Exception $e){
                 throw new \RuntimeException('Could not create Service "'.$cls.'" -> '.$e->getMessage());
             }           
         }

src/Dispatcher/ClassDispatcher.php

Doc Comments -1 removed lines

@@ -71,7 +71,6 @@
      * notifies all listeners on a event on a subject
      *
      * @access public
-     * @param string $name name of the event
      * @param mixed $subject the subject
      * @return
      */

Braces +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
         $name = is_object($name) ? get_class($name) : (string)$name;
         if(Test::assertClassOrInterfaceExists($name)){
             parent::connect($name, $listener);
-        }else{
+        } else{
             throw new \InvalidArgumentException($name." is not an Class nor Interface");
         }
     }