DataOperations::saveRecord() - Code Metrics - Inspection of "Removing php version limits to allow building on h..." - ntentan/nibii - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 7f591d...75938c )

by James Ekow Abaka

created 2017-06-26 16:34 UTC

DataOperations::saveRecord() C

↳ Parent: DataOperations

Complexity

Conditions	10
Paths	52

Size

Total Lines	69
Code Lines	40

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	32
CRAP Score	10.0629

Importance

Changes	3
Bugs	0	Features	0

Metric	Value
c	3
b	0
f	0
dl	0
loc	69
ccs	32
cts	35
cp	0.9143
rs	6.0493
cc	10
eloc	40
nc	52
nop	2
crap	10.0629

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;

    /**
     * Private instance of driver adapter
     *
     * @var DriverAdapter
     */
    private $adapter;

    /**
     *
     * @var array
     */
    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);
        } else {
            $this->wrapper->preSaveCallback();
            $record = $this->wrapper->getData();
            $record = reset($record) === false ? [] : reset($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;
        }
        
        // Save any relationships that are attached to the data
        $relationships = $this->wrapper->getDescription()->getRelationships();
        $presentRelationships = [];
        
        foreach($relationships ?? [] as $model => $relationship) {
            if(isset($record[$model])) {
                $relationship->preSave($record, $record[$model]);
                $presentRelationships[$model] = $relationship;
            }
        }
        
        // 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();
        } else {
            $this->adapter->insert($record);
            $keyValue = $this->driver->getLastInsertId();
            $this->wrapper->{$primaryKey[0]} = $keyValue;
            $this->wrapper->postSaveCallback($keyValue);
        }
        
        // Reset the data so it contains any modifications made by callbacks
        $record = $this->wrapper->getData()[0];
        foreach($presentRelationships as $model => $relationship) {
            $relationship->postSave($record);
        }        

        return $status;
    }

    private function validate($data, $mode) {

        $validator = $this->container->resolve(
            ModelValidator::class, ['model' => $this->wrapper, 'mode' => $mode]
        );
        $errors = [];
        
        if (!$validator->validate($data)) {
            $errors = $validator->getInvalidFields();
        }
        $errors = $this->wrapper->onValidate($errors);
        return empty($errors) ? true : $errors;
    }

    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;
        }
    }

}


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			* Private instance of driver adapter
40			*
41			* @var DriverAdapter
42			*/
43			private $adapter;
44
45			/**
46			*
47			* @var array
48			*/
49			private $data;
50			private $invalidFields;
51			private $hasMultipleData;
52			private $driver;
53			private $container;
54
55			const MODE_SAVE = 0;
56			const MODE_UPDATE = 1;
57
58	35	View Code Duplication	public function __construct(ORMContext $context, RecordWrapper $wrapper, DriverAdapter $adapter) {
59	35		$this->wrapper = $wrapper;
60	35		$this->adapter = $adapter;
61	35		$this->driver = $context->getDbContext()->getDriver();
62	35		$this->container = $context->getContainer();
63	35		}
64
65	10		public function doSave($hasMultipleData) {
66	10		$this->hasMultipleData = $hasMultipleData;
67	10		$invalidFields = [];
68	10		$data = $this->wrapper->getData();
69
70	10		$primaryKey = $this->wrapper->getDescription()->getPrimaryKey();
71	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...
72	10		$succesful = true;
73
74	10		if (count($primaryKey) == 1) {
75	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...
76			}
77
78			// Assign an empty array to force a validation error for empty models
79	10		if (empty($data)) {
80	2		$data = [[]];
81			}
82
83	10		$this->driver->beginTransaction();
84
85	10		foreach ($data as $i => $datum) {
86	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...
87	10		$data[$i] = $datum;
88
89	10		if (!$status['success']) {
90	4		$succesful = false;
91	4		$invalidFields[$i] = $status['invalid_fields'];
92	4		$this->driver->rollback();
93	4		break;
94			}
95			}
96
97	10		if ($succesful) {
98	6		$this->driver->commit();
99			} else {
100	4		$this->assignValue($this->invalidFields, $invalidFields);
101			}
102
103	10		$this->wrapper->setData($hasMultipleData ? $data : $data[0]);
104
105	10		return $succesful;
106			}
107
108			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...
109			$record = $this->wrapper->getData()[0];
110			$primaryKey = $this->wrapper->getDescription()->getPrimaryKey();
111			$pkSet = $this->isPrimaryKeySet($primaryKey, $record);
112			return $this->validate(
113			$record, $pkSet ? DataOperations::MODE_UPDATE : DataOperations::MODE_SAVE
114			);
115			}
116
117			/**
118			* Save an individual record.
119			*
120			* @param array $record The record to be saved
121			* @param type $primaryKey The primary keys of the record
122			* @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...
123			*/
124	10		private function saveRecord(&$record, $primaryKey) {
125			$status = [
126	10		'success' => true,
127			'pk_assigned' => null,
128			'invalid_fields' => []
129			];
130
131			// Determine if the primary key of the record is set.
132	10		$pkSet = $this->isPrimaryKeySet($primaryKey, $record);
133
134			// Reset the data in the model to contain only the data to be saved
135	10		$this->wrapper->setData($record);
136
137			// Run preUpdate or preSave callbacks on models and behaviours
138	10		if ($pkSet) {
139	2		$this->wrapper->preUpdateCallback();
140	2		$record = $this->wrapper->getData();
141	2		$record = reset($record) === false ? [] : reset($record);
142			} else {
143	8		$this->wrapper->preSaveCallback();
144	8		$record = $this->wrapper->getData();
145	8		$record = reset($record) === false ? [] : reset($record);
146			}
147
148			// Validate the data
149	10		$validity = $this->validate(
150	10		$record, $pkSet ? DataOperations::MODE_UPDATE : DataOperations::MODE_SAVE
151			);
152
153			// Exit if data is invalid
154	10		if ($validity !== true) {
155	4		$status['invalid_fields'] = $validity;
156	4		$status['success'] = false;
157	4		return $status;
158			}
159
160			// Save any relationships that are attached to the data
161	6		$relationships = $this->wrapper->getDescription()->getRelationships();
162	6		$presentRelationships = [];
163
164	6		foreach($relationships ?? [] as $model => $relationship) {
165	6		if(isset($record[$model])) {
166			$relationship->preSave($record, $record[$model]);
167			$presentRelationships[$model] = $relationship;
168			}
169			}
170
171			// Assign the data to the wrapper again
172	6		$this->wrapper->setData($record);
173
174			// Update or save the data and run post callbacks
175	6		if ($pkSet) {
176	2		$this->adapter->update($record);
177	2		$this->wrapper->postUpdateCallback();
178			} else {
179	4		$this->adapter->insert($record);
180	4		$keyValue = $this->driver->getLastInsertId();
181	4		$this->wrapper->{$primaryKey[0]} = $keyValue;
182	4		$this->wrapper->postSaveCallback($keyValue);
183			}
184
185			// Reset the data so it contains any modifications made by callbacks
186	6		$record = $this->wrapper->getData()[0];
187	6		foreach($presentRelationships as $model => $relationship) {
188			$relationship->postSave($record);
189			}
190
191	6		return $status;
192			}
193
194	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...
195	10		$validator = $this->container->resolve(
196	10		ModelValidator::class, ['model' => $this->wrapper, 'mode' => $mode]
197			);
198	10		$errors = [];
199
200	10		if (!$validator->validate($data)) {
201	4		$errors = $validator->getInvalidFields();
202			}
203	10		$errors = $this->wrapper->onValidate($errors);
204	10		return empty($errors) ? true : $errors;
205			}
206
207	10		private function isPrimaryKeySet($primaryKey, $data) {
208	10		if (is_string($primaryKey) && ($data[$primaryKey] !== null \|\| $data[$primaryKey] !== '')) {
209			return true;
210			}
211	10		foreach ($primaryKey as $keyField) {
212	10		if (!isset($data[$keyField]) \|\| $data[$keyField] === null \|\| $data[$keyField] === '') {
213	8		return false;
214			}
215			}
216	2		return true;
217			}
218
219	4		private function assignValue(&$property, $value) {
220	4		if ($this->hasMultipleData) {
221			$property = $value;
222			} else {
223	4		$property = $value[0];
224			}
225	4		}
226
227			public function getData() {
228			return $this->data;
229			}
230
231	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...
232	10		return $this->invalidFields;
233			}
234
235			public function isItemDeletable($primaryKey, $data) {
236			if ($this->isPrimaryKeySet($primaryKey, $data)) {
237			return true;
238			} else {
239			return false;
240			}
241			}
242
243			}
244

ntentan / nibii

Push — master ( 7f591d...75938c )

DataOperations::saveRecord() C

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like