View GitHub Repository midgard-portable

Schedule Inspection
Subscribe
Passed
Push — master ( dd2b6c...573650 )
by Andreas
04:29
created
Status
Category
Toggle Patches
src/api/dbobject.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch 
@@ -59,7 +59,7 @@  discard block
 
 block discarded – undo
59 59 
     {
60 60 
         $this->initialize();
61 61 
         $properties = array_merge($this->cm->getFieldNames(), $this->cm->getAssociationNames(), array_keys($this->cm->midgard['field_aliases']));
62 
-        $properties = array_filter($properties, function ($input) {
62 
+        $properties = array_filter($properties, function($input) {
63 63 
             return strpos($input, 'metadata_') === false;
64 64 
         });
65 65 
         $ret = [];
 
@@ -74,7 +74,7 @@  discard block
 
 block discarded – undo
74 74 
     {
75 75 
         $this->initialize();
76 76
77 
-        if (   !$this->cm->hasField($field)
77 
+        if (!$this->cm->hasField($field)
78 78 
             && isset($this->cm->midgard['field_aliases'][$field])) {
79 79 
             $field = $this->cm->midgard['field_aliases'][$field];
80 80 
         }
 
@@ -84,7 +84,7 @@  discard block
 
 block discarded – undo
84 84 
             if (empty($value)) {
85 85 
                 $value = null;
86 86 
             } else {
87 
-                if (   !\is_object($this->$field)
87 
+                if (!\is_object($this->$field)
88 88 
                     || $this->$field->id != $value) {
89 89 
                     $this->changed_associations[$field] = true;
90 90 
                 }
 
@@ -94,7 +94,7 @@  discard block
 
 block discarded – undo
94 94 
         } elseif ($this->cm->hasField($field)) {
95 95 
             $mapping = $this->cm->getFieldMapping($field);
96 96
97 
-            if (   $mapping['type'] === 'string'
97 
+            if ($mapping['type'] === 'string'
98 98 
                 || $mapping['type'] == 'text') {
99 99 
                 $value = (string) $value;
100 100 
             } elseif ($mapping['type'] === 'integer') {
 
@@ -104,7 +104,7 @@  discard block
 
 block discarded – undo
104 104 
             } elseif ($mapping['type'] === 'float') {
105 105 
                 $value = (float) $value;
106 106 
             } elseif ($mapping['type'] === 'datetime') {
107 
-                if (   \is_string($value)
107 
+                if (\is_string($value)
108 108 
                     && $value !== '0000-00-00 00:00:00') {
109 109 
                     $value = new midgard_datetime($value);
110 110 
                 } elseif (!($value instanceof midgard_datetime)) {
 
@@ -120,7 +120,7 @@  discard block
 
 block discarded – undo
120 120 
     {
121 121 
         $this->initialize();
122 122
123 
-        if (   !$this->cm->hasField($field)
123 
+        if (!$this->cm->hasField($field)
124 124 
             && isset($this->cm->midgard['field_aliases'][$field])) {
125 125 
             $field = $this->cm->midgard['field_aliases'][$field];
126 126 
         }
 
@@ -133,11 +133,11 @@  discard block
 
 block discarded – undo
133 133 
             }
134 134 
             return 0;
135 135 
         }
136 
-        if (   $this->$field === null
136 
+        if ($this->$field === null
137 137 
             && $this->cm->isIdentifier($field)) {
138 138 
             return 0;
139 139 
         }
140 
-        if (   $this->$field instanceof midgard_datetime
140 
+        if ($this->$field instanceof midgard_datetime
141 141 
             && $this->$field->format('U') == -62169984000) {
142 142 
             //This is mainly needed for working with converted Legacy databases. Midgard2 somehow handles this internally
143 143 
             //@todo Find a nicer solution and research how QB handles this
Please login to merge, or discard this patch.