flack / midgard-portable

src/api/blob.php

Doc Comments +3 added lines

@@ -31,6 +31,9 @@
         return null;
     }
 
+    /**
+     * @param string $content
+     */
     public function write_content($content)
     {
         return file_put_contents($this->get_path(), $content) !== false;

api/midgard/storage.php

Doc Comments +8 added lines

@@ -56,6 +56,9 @@ 
         return true;
     }
 
+    /**
+     * @param string $classname
+     */
     public static function create_class_storage($classname)
     {
         $em = connection::get_em();
@@ -87,6 +90,11 @@ 
         $generator->generateProxyClass($cm, $filename);
     }
 
+    /**
+     * @param Doctrine\ORM\EntityManager $em
+     *
+     * @return ClassMetadata
+     */
     private static function get_cm($em, $classname)
     {
         if (!class_exists($classname)) {

api/midgard/connection.php

Doc Comments +3 added lines

@@ -29,6 +29,9 @@
 
     private $replication_enabled = true;
 
+    /**
+     * @return integer
+     */
     public static function get_instance()
     {
         if (self::$instance === null) {

api/midgard/object/class.php

Doc Comments +3 added lines

@@ -96,6 +96,9 @@
         return true;
     }
 
+    /**
+     * @param string $guid
+     */
     public static function get_object_by_guid($guid)
     {
         if (!mgd_is_guid($guid)) {

api/midgard/replicator.php

Doc Comments +11 added lines, -2 removed lines

@@ -240,7 +240,7 @@ 
     }
 
     /**
-     * @return boolean Indicating success
+     * @return boolean|null Indicating success
      */
     public static function import_from_xml($xml, $force = false)
     {
@@ -264,6 +264,9 @@ 
         $blob->write_content($blob->content);
     }
 
+    /**
+     * @return string|null
+     */
     private static function resolve_link_id(ClassMetadata $cm, dbobject $object, $name)
     {
         if ($object->$name == 0) {
@@ -339,7 +342,7 @@ 
      *
      * @param SimpleXMLElement $node
      * @param boolean $force
-     * @return dbobject
+     * @return blob
      */
     private static function blob_from_xml(SimpleXMLElement $node, $force)
     {
@@ -350,6 +353,9 @@ 
         return $blob;
     }
 
+    /**
+     * @param string $guid
+     */
     private static function get_object_action($guid)
     {
         $result = connection::get_em()
@@ -376,6 +382,9 @@ 
         }
     }
 
+    /**
+     * @return string|null
+     */
     private static function convert_value($value)
     {
         if ($value instanceof midgard_datetime) {

src/api/config.php

Doc Comments +6 added lines

@@ -30,6 +30,9 @@ 
     public $cachedir = '/var/cache/midgard2';
     public $gdathreads = false;
 
+    /**
+     * @param string $path
+     */
     public function read_file_at_path($path)
     {
         if (   !file_exists($path)
@@ -122,6 +125,9 @@ 
         return true;
     }
 
+    /**
+     * @param string $key
+     */
     private function convert_to_storage($key, $value)
     {
         if (is_bool($value)) {

src/query.php

Doc Comments +6 added lines

@@ -205,6 +205,9 @@ 
         }
     }
 
+    /**
+     * @param string $targetclass
+     */
     protected function add_collection_join($current_table, $targetclass)
     {
         if (!array_key_exists($targetclass, $this->join_tables)) {
@@ -215,6 +218,9 @@ 
         return $this->join_tables[$targetclass];
     }
 
+    /**
+     * @param \midgard_reflection_property $mrp
+     */
     protected function add_join($current_table, $mrp, $property)
     {
         $targetclass = $mrp->get_link_name($property);

src/storage/connection.php

Doc Comments +4 added lines

@@ -123,6 +123,7 @@ 
 
     /**
      * Toggle autostart
+     * @param boolean $autostart
      */
     public static function set_autostart($autostart)
     {
@@ -156,6 +157,9 @@ 
         }
     }
 
+    /**
+     * @param string $name
+     */
     public static function get_parameter($name)
     {
         if (!array_key_exists($name, self::$parameters)) {

src/storage/objectmanager.php

Doc Comments +4 added lines

@@ -65,6 +65,7 @@ 
      * This is basically a workaround for some quirks when merging detached entities with changed associations
      *
      * @todo: This may or may not be a bug in Doctrine
+     * @param dbobject $source
      */
     private function copy_associations($source, $target)
     {
@@ -222,6 +223,9 @@ 
         return $entity;
     }
 
+    /**
+     * @param dbobject $target
+     */
     private function copy_metadata($source, $target, $action = 'update')
     {
         if (!$source instanceof metadata) {