Passed
Push — master ( 24644b...f0a5ce )
by Andreas
04:06 queued 22s
created
src/command/schema.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -55,7 +55,7 @@  discard block
 block discarded – undo
55 55
 
56 56
         $mgd_config = midgard_connection::get_instance()->config;
57 57
         $mgdschema_file = $mgd_config->vardir . '/mgdschema_classes.php';
58
-        if (   file_exists($mgdschema_file)
58
+        if (file_exists($mgdschema_file)
59 59
             && !unlink($mgdschema_file)) {
60 60
             throw new \RuntimeException('Could not unlink ' . $mgdschema_file);
61 61
         }
@@ -72,7 +72,7 @@  discard block
 block discarded – undo
72 72
         $cms = $em->getMetadataFactory()->getAllMetadata();
73 73
 
74 74
         // create storage
75
-        if (    !midgard_storage::create_base_storage()
75
+        if (!midgard_storage::create_base_storage()
76 76
              && midgard_connection::get_instance()->get_error_string() != 'MGD_ERR_OK') {
77 77
             throw new \Exception("Failed to create base database structures" . midgard_connection::get_instance()->get_error_string());
78 78
         }
Please login to merge, or discard this patch.
api/midgard/storage.php 1 patch
Doc Comments   +8 added lines patch added patch discarded remove patch
@@ -56,6 +56,9 @@  discard block
 block discarded – undo
56 56
         return true;
57 57
     }
58 58
 
59
+    /**
60
+     * @param string $classname
61
+     */
59 62
     public static function create_class_storage($classname)
60 63
     {
61 64
         $em = connection::get_em();
@@ -87,6 +90,11 @@  discard block
 block discarded – undo
87 90
         $generator->generateProxyClass($cm, $filename);
88 91
     }
89 92
 
93
+    /**
94
+     * @param Doctrine\ORM\EntityManager $em
95
+     *
96
+     * @return ClassMetadata
97
+     */
90 98
     private static function get_cm($em, $classname)
91 99
     {
92 100
         if (!class_exists($classname)) {
Please login to merge, or discard this patch.
api/midgard/connection.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -29,6 +29,9 @@
 block discarded – undo
29 29
 
30 30
     private $replication_enabled = true;
31 31
 
32
+    /**
33
+     * @return integer
34
+     */
32 35
     public static function get_instance()
33 36
     {
34 37
         if (self::$instance === null) {
Please login to merge, or discard this patch.
api/midgard/object/class.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -96,6 +96,9 @@
 block discarded – undo
96 96
         return true;
97 97
     }
98 98
 
99
+    /**
100
+     * @param string $guid
101
+     */
99 102
     public static function get_object_by_guid($guid)
100 103
     {
101 104
         if (!mgd_is_guid($guid)) {
Please login to merge, or discard this patch.
api/midgard/query/builder.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -19,6 +19,9 @@
 block discarded – undo
19 19
         }
20 20
     }
21 21
 
22
+    /**
23
+     * @param string $operator
24
+     */
22 25
     public function add_constraint($name, $operator, $value)
23 26
     {
24 27
         try {
Please login to merge, or discard this patch.
src/storage/objectmanager.php 1 patch
Doc Comments   +4 added lines patch added patch discarded remove patch
@@ -65,6 +65,7 @@  discard block
 block discarded – undo
65 65
      * This is basically a workaround for some quirks when merging detached entities with changed associations
66 66
      *
67 67
      * @todo: This may or may not be a bug in Doctrine
68
+     * @param dbobject $source
68 69
      */
69 70
     private function copy_associations($source, $target)
70 71
     {
@@ -222,6 +223,9 @@  discard block
 block discarded – undo
222 223
         return $entity;
223 224
     }
224 225
 
226
+    /**
227
+     * @param dbobject $target
228
+     */
225 229
     private function copy_metadata($source, $target, $action = 'update')
226 230
     {
227 231
         if (!$source instanceof metadata) {
Please login to merge, or discard this patch.
src/xmlreader.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -95,6 +95,9 @@
 block discarded – undo
95 95
         $type->add_property($property);
96 96
     }
97 97
 
98
+    /**
99
+     * @param string $name
100
+     */
98 101
     private function add_mixin($name, type $type)
99 102
     {
100 103
         if (empty($this->mixins[$name])) {
Please login to merge, or discard this patch.
api/midgard/collector.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
63 63
         $parsed = $this->parse_constraint_name($property);
64 64
 
65 65
         // for properties like up.name
66
-        if (   strpos($property, ".") !== false
66
+        if (strpos($property, ".") !== false
67 67
             && !(strpos($property, "metadata") === 0)) {
68 68
             return $parsed['name'] . " as " . str_replace(".", "_", $property);
69 69
         }
@@ -134,7 +134,7 @@  discard block
 block discarded – undo
134 134
      */
135 135
     public function get($key)
136 136
     {
137
-        if (   !$this->_has_results()
137
+        if (!$this->_has_results()
138 138
             || !isset($this->_results[$key])) {
139 139
             return false;
140 140
         }
Please login to merge, or discard this patch.
Doc Comments   +7 added lines patch added patch discarded remove patch
@@ -25,6 +25,10 @@  discard block
 block discarded – undo
25 25
 
26 26
     private $value_properties = array();
27 27
 
28
+    /**
29
+     * @param string $class
30
+     * @param string $field
31
+     */
28 32
     public function __construct($class, $field = null, $value = null)
29 33
     {
30 34
         parent::__construct($class);
@@ -68,6 +72,9 @@  discard block
 block discarded – undo
68 72
         return true;
69 73
     }
70 74
 
75
+    /**
76
+     * @param string $property
77
+     */
71 78
     protected function build_property_select($property)
72 79
     {
73 80
         $parsed = $this->parse_constraint_name($property);
Please login to merge, or discard this patch.
api/bootstrap.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -9,7 +9,7 @@
 block discarded – undo
9 9
 use midgard\portable\mgdschema\translator;
10 10
 use midgard\portable\api\error\exception;
11 11
 
12
-if (   extension_loaded('midgard')
12
+if (extension_loaded('midgard')
13 13
     || extension_loaded('midgard2')) {
14 14
     //TODO: Print some error?
15 15
     return;
Please login to merge, or discard this patch.