DataOperations::saveRecord() - Code Metrics - Inspection of "Added a way to save related values for belongs to..." - ntentan/nibii - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( efabbe...343d0a )

by James Ekow Abaka

created 2017-05-22 02:00 UTC

DataOperations::saveRecord() C

↳ Parent: DataOperations

Complexity

Conditions	11
Paths	76

Size

Total Lines	72
Code Lines	43

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	38
CRAP Score	11

Importance

Changes	3
Bugs	0	Features	0

Metric	Value
c	3
b	0
f	0
dl	0
loc	72
ccs	38
cts	38
cp	1
rs	5.6331
cc	11
eloc	43
nc	76
nop	2
crap	11

How to fix Long Method Complexity

<?php

/*
 * The MIT License
 *
 * Copyright 2015 ekow.
 *
 * Permission is hereby granted, free of charge, to any person obtaining a copy
 * of this software and associated documentation files (the "Software"), to deal
 * in the Software without restriction, including without limitation the rights
 * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
 * copies of the Software, and to permit persons to whom the Software is
 * furnished to do so, subject to the following conditions:
 *
 * The above copyright notice and this permission notice shall be included in
 * all copies or substantial portions of the Software.
 *
 * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
 * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
 * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
 * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
 * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
 * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
 * THE SOFTWARE.
 */

namespace ntentan\nibii;

/**
 * Description of DataOperations
 *
 * @author ekow
 */
class DataOperations {

    private $wrapper;

    /**
     *
     * @var DriverAdapter
     */
    private $adapter;
    private $data;
    private $invalidFields;
    private $hasMultipleData;
    private $driver;
    private $container;

    const MODE_SAVE = 0;
    const MODE_UPDATE = 1;

    public function __construct(ORMContext $context, RecordWrapper $wrapper, DriverAdapter $adapter) {
        $this->wrapper = $wrapper;
        $this->adapter = $adapter;
        $this->driver = $context->getDbContext()->getDriver();
        $this->container = $context->getContainer();
    }

    public function doSave($hasMultipleData) {
        $this->hasMultipleData = $hasMultipleData;
        $invalidFields = [];
        $data = $this->wrapper->getData();
        $primaryKey = $this->wrapper->getDescription()->getPrimaryKey();
        $singlePrimaryKey = null;
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
        $succesful = true;

        if (count($primaryKey) == 1) {
            $singlePrimaryKey = $primaryKey[0];
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
        }

        // Assign an empty array to force a validation error for empty models
        if (empty($data)) {
            $data = [[]];
        }

        $this->driver->beginTransaction();

        foreach ($data as $i => $datum) {
            $status = $this->saveRecord($datum, $primaryKey);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $data[$i] = $datum;

            if (!$status['success']) {
                $succesful = false;
                $invalidFields[$i] = $status['invalid_fields'];
                $this->driver->rollback();
                break;
            }
        }

        if ($succesful) {
            $this->driver->commit();
        } else {
            $this->assignValue($this->invalidFields, $invalidFields);
        }

        $this->wrapper->setData($hasMultipleData ? $data : $data[0]);

        return $succesful;
    }
    
    public function doValidate() {

        $record = $this->wrapper->getData()[0];
        $primaryKey = $this->wrapper->getDescription()->getPrimaryKey();
        $pkSet = $this->isPrimaryKeySet($primaryKey, $record);
        return $this->validate(
            $record, $pkSet ? DataOperations::MODE_UPDATE : DataOperations::MODE_SAVE
        );
    }

    /**
     * Save an individual record.
     * 
     * @param array $record The record to be saved
     * @param type $primaryKey The primary keys of the record
     * @return boolean

     */
    private function saveRecord(&$record, $primaryKey) {
        $status = [
            'success' => true,
            'pk_assigned' => null,
            'invalid_fields' => []
        ];

        // Determine if the primary key of the record is set.
        $pkSet = $this->isPrimaryKeySet($primaryKey, $record);

        // Reset the data in the model to contain only the data to be saved
        $this->wrapper->setData($record);

        // Run preUpdate or preSave callbacks on models and behaviours
        if ($pkSet) {
            $this->wrapper->preUpdateCallback();
            $record = $this->wrapper->getData();
            $record = reset($record) === false ? [] : reset($record);
            $record = $this->runBehaviours('preUpdateCallback', [$record]);
        } else {
            $this->wrapper->preSaveCallback();
            $record = $this->wrapper->getData();
            $record = reset($record) === false ? [] : reset($record);
            $record = $this->runBehaviours('preSaveCallback', [$record]);
        }

        // Validate the data
        $validity = $this->validate(
            $record, $pkSet ? DataOperations::MODE_UPDATE : DataOperations::MODE_SAVE
        );

        // Exit if data is invalid
        if ($validity !== true) {
            $status['invalid_fields'] = $validity;
            $status['success'] = false;
            return $status;
        }
        
        $relationships = $this->wrapper->getDescription()->getRelationships();
        
        foreach($relationships as $model => $relationship) {
            if(isset($record[$model])) {
                $relationship->preSave($record, $record[$model]);
            }
        }
        
        // Assign the data to the wrapper again
        $this->wrapper->setData($record);

        // Update or save the data and run post callbacks
        if ($pkSet) {
            $this->adapter->update($record);
            $this->wrapper->postUpdateCallback();
            $this->runBehaviours('postUpdateCallback', [$record]);
        } else {
            $this->adapter->insert($record);
            $keyValue = $this->driver->getLastInsertId();
            $this->wrapper->{$primaryKey[0]} = $keyValue;
            $this->wrapper->postSaveCallback($keyValue);
            $this->runBehaviours('postSaveCallback', [$record, $keyValue]);
        }
        
        foreach($relationships as $model => $relationship) {
            if(isset($record[$model])) {
                $relationship->postSave($record, $record[$model]);
            }
        }        

        // Reset the data so it contains any modifications made by callbacks
        $record = $this->wrapper->getData()[0];
        return $status;
    }

    private function validate($data, $mode) {

        $valid = true;
        $validator = $this->container->resolve(
            ModelValidator::class, ['model' => $this->wrapper, 'mode' => $mode]
        );

        if (!$validator->validate($data)) {
            $valid = false;
        }

        if ($valid) {
            $valid = $this->wrapper->onValidate();
        }

        if ($valid === false) {
            $valid = $validator->getInvalidFields();
        }

        return $valid;
    }

    private function isPrimaryKeySet($primaryKey, $data) {
        if (is_string($primaryKey) && ($data[$primaryKey] !== null || $data[$primaryKey] !== '')) {
            return true;
        }
        foreach ($primaryKey as $keyField) {
            if (!isset($data[$keyField]) || $data[$keyField] === null || $data[$keyField] === '') {
                return false;
            }
        }
        return true;
    }

    private function assignValue(&$property, $value) {
        if ($this->hasMultipleData) {
            $property = $value;
        } else {
            $property = $value[0];
        }
    }

    public function getData() {

        return $this->data;
    }

    public function getInvalidFields() {

        return $this->invalidFields;
    }

    public function isItemDeletable($primaryKey, $data) {
        if ($this->isPrimaryKeySet($primaryKey, $data)) {
            return true;
        } else {
            return false;
        }
    }

    private function runBehaviours($event, $args) {

        foreach ($this->wrapper->getBehaviours() as $behaviour) {
            $args[0] = call_user_func_array([$behaviour, $event], $args);
        }
        return $args[0];
    }

}


1			<?php
2
3			/*
4			* The MIT License
5			*
6			* Copyright 2015 ekow.
7			*
8			* Permission is hereby granted, free of charge, to any person obtaining a copy
9			* of this software and associated documentation files (the "Software"), to deal
10			* in the Software without restriction, including without limitation the rights
11			* to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
12			* copies of the Software, and to permit persons to whom the Software is
13			* furnished to do so, subject to the following conditions:
14			*
15			* The above copyright notice and this permission notice shall be included in
16			* all copies or substantial portions of the Software.
17			*
18			* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
19			* IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
20			* FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
21			* AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
22			* LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
23			* OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
24			* THE SOFTWARE.
25			*/
26
27			namespace ntentan\nibii;
28
29			/**
30			* Description of DataOperations
31			*
32			* @author ekow
33			*/
34			class DataOperations {
35
36			private $wrapper;
37
38			/**
39			*
40			* @var DriverAdapter
41			*/
42			private $adapter;
43			private $data;
44			private $invalidFields;
45			private $hasMultipleData;
46			private $driver;
47			private $container;
48
49			const MODE_SAVE = 0;
50			const MODE_UPDATE = 1;
51
52	35	View Code Duplication	public function __construct(ORMContext $context, RecordWrapper $wrapper, DriverAdapter $adapter) {
53	35		$this->wrapper = $wrapper;
54	35		$this->adapter = $adapter;
55	35		$this->driver = $context->getDbContext()->getDriver();
56	35		$this->container = $context->getContainer();
57	35		}
58
59	10		public function doSave($hasMultipleData) {
60	10		$this->hasMultipleData = $hasMultipleData;
61	10		$invalidFields = [];
62	10		$data = $this->wrapper->getData();
63	10		$primaryKey = $this->wrapper->getDescription()->getPrimaryKey();
64	10		$singlePrimaryKey = null;
			0 ignored issues – show Unused Code introduced 2015-11-24 20:03 UTC by Report Bug Copy Issue Report `$singlePrimaryKey` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
65	10		$succesful = true;
66
67	10		if (count($primaryKey) == 1) {
68	10		$singlePrimaryKey = $primaryKey[0];
			0 ignored issues – show Unused Code introduced 2015-11-24 20:03 UTC by Report Bug Copy Issue Report `$singlePrimaryKey` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
69			}
70
71			// Assign an empty array to force a validation error for empty models
72	10		if (empty($data)) {
73	2		$data = [[]];
74			}
75
76	10		$this->driver->beginTransaction();
77
78	10		foreach ($data as $i => $datum) {
79	10		$status = $this->saveRecord($datum, $primaryKey);
			0 ignored issues – show Documentation introduced 2017-05-02 16:26 UTC by Report Bug Copy Issue Report `$primaryKey` is of type `array`, but the function expects a `object<ntentan\nibii\type>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
80	10		$data[$i] = $datum;
81
82	10		if (!$status['success']) {
83	4		$succesful = false;
84	4		$invalidFields[$i] = $status['invalid_fields'];
85	4		$this->driver->rollback();
86	10		break;
87			}
88			}
89
90	10		if ($succesful) {
91	6		$this->driver->commit();
92			} else {
93	4		$this->assignValue($this->invalidFields, $invalidFields);
94			}
95
96	10		$this->wrapper->setData($hasMultipleData ? $data : $data[0]);
97
98	10		return $succesful;
99			}
100
101			public function doValidate() {
			0 ignored issues – show Documentation introduced 2017-05-21 02:04 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
102			$record = $this->wrapper->getData()[0];
103			$primaryKey = $this->wrapper->getDescription()->getPrimaryKey();
104			$pkSet = $this->isPrimaryKeySet($primaryKey, $record);
105			return $this->validate(
106			$record, $pkSet ? DataOperations::MODE_UPDATE : DataOperations::MODE_SAVE
107			);
108			}
109
110			/**
111			* Save an individual record.
112			*
113			* @param array $record The record to be saved
114			* @param type $primaryKey The primary keys of the record
115			* @return boolean
			0 ignored issues – show Documentation introduced 2015-11-24 20:03 UTC by Report Bug Copy Issue Report Should the return type not be `array`? This check compares the return type specified in the `@return` annotation of a function or method doc comment with the types returned by the function and raises an issue if they mismatch. Loading history...
116			*/
117	10		private function saveRecord(&$record, $primaryKey) {
118			$status = [
119	10		'success' => true,
120			'pk_assigned' => null,
121			'invalid_fields' => []
122			];
123
124			// Determine if the primary key of the record is set.
125	10		$pkSet = $this->isPrimaryKeySet($primaryKey, $record);
126
127			// Reset the data in the model to contain only the data to be saved
128	10		$this->wrapper->setData($record);
129
130			// Run preUpdate or preSave callbacks on models and behaviours
131	10		if ($pkSet) {
132	2		$this->wrapper->preUpdateCallback();
133	2		$record = $this->wrapper->getData();
134	2		$record = reset($record) === false ? [] : reset($record);
135	2		$record = $this->runBehaviours('preUpdateCallback', [$record]);
136			} else {
137	8		$this->wrapper->preSaveCallback();
138	8		$record = $this->wrapper->getData();
139	8		$record = reset($record) === false ? [] : reset($record);
140	8		$record = $this->runBehaviours('preSaveCallback', [$record]);
141			}
142
143			// Validate the data
144	10		$validity = $this->validate(
145	10		$record, $pkSet ? DataOperations::MODE_UPDATE : DataOperations::MODE_SAVE
146			);
147
148			// Exit if data is invalid
149	10		if ($validity !== true) {
150	4		$status['invalid_fields'] = $validity;
151	4		$status['success'] = false;
152	4		return $status;
153			}
154
155	6		$relationships = $this->wrapper->getDescription()->getRelationships();
156
157	6		foreach($relationships as $model => $relationship) {
158	6		if(isset($record[$model])) {
159	6		$relationship->preSave($record, $record[$model]);
160			}
161			}
162
163			// Assign the data to the wrapper again
164	6		$this->wrapper->setData($record);
165
166			// Update or save the data and run post callbacks
167	6		if ($pkSet) {
168	2		$this->adapter->update($record);
169	2		$this->wrapper->postUpdateCallback();
170	2		$this->runBehaviours('postUpdateCallback', [$record]);
171			} else {
172	4		$this->adapter->insert($record);
173	4		$keyValue = $this->driver->getLastInsertId();
174	4		$this->wrapper->{$primaryKey[0]} = $keyValue;
175	4		$this->wrapper->postSaveCallback($keyValue);
176	4		$this->runBehaviours('postSaveCallback', [$record, $keyValue]);
177			}
178
179	6		foreach($relationships as $model => $relationship) {
180	6		if(isset($record[$model])) {
181	6		$relationship->postSave($record, $record[$model]);
182			}
183			}
184
185			// Reset the data so it contains any modifications made by callbacks
186	6		$record = $this->wrapper->getData()[0];
187	6		return $status;
188			}
189
190	10		private function validate($data, $mode) {
			0 ignored issues – show Documentation introduced 2015-11-24 20:03 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
191	10		$valid = true;
192	10		$validator = $this->container->resolve(
193	10		ModelValidator::class, ['model' => $this->wrapper, 'mode' => $mode]
194			);
195
196	10		if (!$validator->validate($data)) {
197	4		$valid = false;
198			}
199
200	10		if ($valid) {
201	6		$valid = $this->wrapper->onValidate();
202			}
203
204	10		if ($valid === false) {
205	4		$valid = $validator->getInvalidFields();
206			}
207
208	10		return $valid;
209			}
210
211	10		private function isPrimaryKeySet($primaryKey, $data) {
212	10		if (is_string($primaryKey) && ($data[$primaryKey] !== null \|\| $data[$primaryKey] !== '')) {
213			return true;
214			}
215	10		foreach ($primaryKey as $keyField) {
216	10		if (!isset($data[$keyField]) \|\| $data[$keyField] === null \|\| $data[$keyField] === '') {
217	10		return false;
218			}
219			}
220	2		return true;
221			}
222
223	4		private function assignValue(&$property, $value) {
224	4		if ($this->hasMultipleData) {
225			$property = $value;
226			} else {
227	4		$property = $value[0];
228			}
229	4		}
230
231			public function getData() {
			0 ignored issues – show Documentation introduced 2015-11-24 20:03 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
232			return $this->data;
233			}
234
235	10		public function getInvalidFields() {
			0 ignored issues – show Documentation introduced 2015-11-24 20:03 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
236	10		return $this->invalidFields;
237			}
238
239			public function isItemDeletable($primaryKey, $data) {
240			if ($this->isPrimaryKeySet($primaryKey, $data)) {
241			return true;
242			} else {
243			return false;
244			}
245			}
246
247	10		private function runBehaviours($event, $args) {
			0 ignored issues – show Documentation introduced 2015-11-24 20:03 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
248	10		foreach ($this->wrapper->getBehaviours() as $behaviour) {
249			$args[0] = call_user_func_array([$behaviour, $event], $args);
250			}
251	10		return $args[0];
252			}
253
254			}
255

ntentan / nibii

Push — master ( efabbe...343d0a )

DataOperations::saveRecord() C

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like