Passed
Pull Request — master (#21)
by
unknown
01:35
created

DataLayer::setData()   A

Complexity

Conditions 1
Paths 1

Size

Total Lines 4
Code Lines 2

Duplication

Lines 0
Ratio 0 %

Importance

Changes 2
Bugs 0 Features 0
Metric Value
eloc 2
c 2
b 0
f 0
dl 0
loc 4
rs 10
cc 1
nc 1
nop 1
1
<?php
2
3
namespace CoffeeCode\DataLayer;
4
5
use Exception;
6
use PDO;
7
use PDOException;
8
use stdClass;
9
10
/**
11
 * Class DataLayer
12
 * @package CoffeeCode\DataLayer
13
 */
14
abstract class DataLayer
15
{
16
    use CrudTrait;
17
18
    /** @var string $entity database table */
19
    private $entity;
20
21
    /** @var string $primary table primary key field */
22
    private $primary;
23
24
    /** @var array $required table required fields */
25
    private $required;
26
27
    /** @var string $timestamps control created and updated at */
28
    private $timestamps;
29
30
    /** @var string */
31
    protected $statement;
32
33
    /** @var string */
34
    protected $params;
35
36
    /** @var string */
37
    protected $group;
38
39
    /** @var string */
40
    protected $order;
41
42
    /** @var int */
43
    protected $limit;
44
45
    /** @var int */
46
    protected $offset;
47
48
    /** @var \PDOException|null */
49
    protected $fail;
50
51
    /** @var object|null */
52
    protected $data;
53
54
    /**
55
     * DataLayer constructor.
56
     * @param string $entity
57
     * @param array $required
58
     * @param string $primary
59
     * @param bool $timestamps
60
     */
61
    public function __construct(string $entity, array $required, string $primary = 'id', bool $timestamps = true)
62
    {
63
        $this->entity = $entity;
64
        $this->primary = $primary;
65
        $this->required = $required;
66
        $this->timestamps = $timestamps;
0 ignored issues
show
Documentation Bug introduced by
The property $timestamps was declared of type string, but $timestamps is of type boolean. Maybe add a type cast?

This check looks for assignments to scalar types that may be of the wrong type.

To ensure the code behaves as expected, it may be a good idea to add an explicit type cast.

$answer = 42;

$correct = false;

$correct = (bool) $answer;
Loading history...
67
    }
68
69
    /**
70
     * @param $name
71
     * @param $value
72
     */
73
    public function __set($name, $value)
74
    {
75
        if (empty($this->data)) {
76
            $this->data = new stdClass();
77
        }
78
79
        $this->data->$name = $value;
80
    }
81
82
    /**
83
     * @param $name
84
     * @return bool
85
     */
86
    public function __isset($name)
87
    {
88
        return isset($this->data->$name);
89
    }
90
91
    /**
92
     * @param $name
93
     * @return string|null
94
     */
95
    public function __get($name)
96
    {
97
        return ($this->data->$name ?? null);
98
    }
99
100
    /**
101
     * @return object|null
102
     */
103
    public function data(): ?object
104
    {
105
        return $this->data;
106
    }
107
    
108
    /**
109
     * @param array|null $data
110
     *
111
     * @return object|null
112
     */
113
    public function setData(?array $data): ?object
114
    {
115
        $array = array_replace_recursive((array)$this->data, $data);
116
        return $this->data = (object)$array;
117
    }
118
119
    /**
120
     * @return PDOException|Exception|null
121
     */
122
    public function fail()
123
    {
124
        return $this->fail;
125
    }
126
127
    /**
128
     * @param string|null $terms
129
     * @param string|null $params
130
     * @param string $columns
131
     * @return DataLayer
132
     */
133
    public function find(?string $terms = null, ?string $params = null, string $columns = "*"): DataLayer
134
    {
135
        if ($terms) {
136
            $this->statement = "SELECT {$columns} FROM {$this->entity} WHERE {$terms}";
137
            parse_str($params, $this->params);
0 ignored issues
show
Bug introduced by
$this->params of type string is incompatible with the type array|null expected by parameter $arr of parse_str(). ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-type  annotation

137
            parse_str($params, /** @scrutinizer ignore-type */ $this->params);
Loading history...
138
            return $this;
139
        }
140
141
        $this->statement = "SELECT {$columns} FROM {$this->entity}";
142
        return $this;
143
    }
144
145
    /**
146
     * @param int $id
147
     * @param string $columns
148
     * @return DataLayer|null
149
     */
150
    public function findById(int $id, string $columns = "*"): ?DataLayer
151
    {
152
        $find = $this->find($this->primary . " = :id", "id={$id}", $columns);
153
        return $find->fetch();
0 ignored issues
show
Bug Best Practice introduced by
The expression return $find->fetch() could return the type array which is incompatible with the type-hinted return CoffeeCode\DataLayer\DataLayer|null. Consider adding an additional type-check to rule them out.
Loading history...
154
    }
155
156
    /**
157
     * @param string $column
158
     * @return DataLayer|null
159
     */
160
    public function group(string $column): ?DataLayer
161
    {
162
        $this->group = " GROUP BY {$column}";
163
        return $this;
164
    }
165
166
    /**
167
     * @param string $columnOrder
168
     * @return DataLayer|null
169
     */
170
    public function order(string $columnOrder): ?DataLayer
171
    {
172
        $this->order = " ORDER BY {$columnOrder}";
173
        return $this;
174
    }
175
176
    /**
177
     * @param int $limit
178
     * @return DataLayer|null
179
     */
180
    public function limit(int $limit): ?DataLayer
181
    {
182
        $this->limit = " LIMIT {$limit}";
0 ignored issues
show
Documentation Bug introduced by
The property $limit was declared of type integer, but ' LIMIT '.$limit is of type string. Maybe add a type cast?

This check looks for assignments to scalar types that may be of the wrong type.

To ensure the code behaves as expected, it may be a good idea to add an explicit type cast.

$answer = 42;

$correct = false;

$correct = (bool) $answer;
Loading history...
183
        return $this;
184
    }
185
186
    /**
187
     * @param int $offset
188
     * @return DataLayer|null
189
     */
190
    public function offset(int $offset): ?DataLayer
191
    {
192
        $this->offset = " OFFSET {$offset}";
0 ignored issues
show
Documentation Bug introduced by
The property $offset was declared of type integer, but ' OFFSET '.$offset is of type string. Maybe add a type cast?

This check looks for assignments to scalar types that may be of the wrong type.

To ensure the code behaves as expected, it may be a good idea to add an explicit type cast.

$answer = 42;

$correct = false;

$correct = (bool) $answer;
Loading history...
193
        return $this;
194
    }
195
196
    /**
197
     * @param bool $all
198
     * @return array|mixed|null
199
     */
200
    public function fetch(bool $all = false)
201
    {
202
        try {
203
            $stmt = Connect::getInstance()->prepare($this->statement . $this->group . $this->order . $this->limit . $this->offset);
204
            $stmt->execute($this->params);
0 ignored issues
show
Bug introduced by
$this->params of type string is incompatible with the type array expected by parameter $input_parameters of PDOStatement::execute(). ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-type  annotation

204
            $stmt->execute(/** @scrutinizer ignore-type */ $this->params);
Loading history...
205
206
            if (!$stmt->rowCount()) {
207
                return null;
208
            }
209
210
            if ($all) {
211
                return $stmt->fetchAll(PDO::FETCH_CLASS, static::class);
212
            }
213
214
            return $stmt->fetchObject(static::class);
215
        } catch (PDOException $exception) {
216
            $this->fail = $exception;
217
            return null;
218
        }
219
    }
220
221
    /**
222
     * @return int
223
     */
224
    public function count(): int
225
    {
226
        $stmt = Connect::getInstance()->prepare($this->statement);
227
        $stmt->execute($this->params);
0 ignored issues
show
Bug introduced by
$this->params of type string is incompatible with the type array expected by parameter $input_parameters of PDOStatement::execute(). ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-type  annotation

227
        $stmt->execute(/** @scrutinizer ignore-type */ $this->params);
Loading history...
228
        return $stmt->rowCount();
229
    }
230
231
    /**
232
     * @return bool
233
     */
234
    public function save(): bool
235
    {
236
        $primary = $this->primary;
237
        $id = null;
238
239
        try {
240
            if (!$this->required()) {
241
                throw new Exception("Preencha os campos necessários");
242
            }
243
244
            /** Update */
245
            if (!empty($this->data->$primary)) {
246
                $id = $this->data->$primary;
247
                $this->update($this->safe(), $this->primary . " = :id", "id={$id}");
248
            }
249
250
            /** Create */
251
            if (empty($this->data->$primary)) {
252
                $id = $this->create($this->safe());
253
            }
254
255
            if (!$id) {
256
                return false;
257
            }
258
259
            $this->data = $this->findById($id)->data();
260
            return true;
261
        } catch (Exception $exception) {
262
            $this->fail = $exception;
0 ignored issues
show
Documentation Bug introduced by
$exception is of type Exception, but the property $fail was declared to be of type PDOException|null. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

Our type inference engine has found a suspicous assignment of a value to a property. This check raises an issue when a value that can be of a given class or a super-class is assigned to a property that is type hinted more strictly.

Either this assignment is in error or an instanceof check should be added for that assignment.

class Alien {}

class Dalek extends Alien {}

class Plot
{
    /** @var  Dalek */
    public $villain;
}

$alien = new Alien();
$plot = new Plot();
if ($alien instanceof Dalek) {
    $plot->villain = $alien;
}
Loading history...
263
            return false;
264
        }
265
    }
266
267
    /**
268
     * @return bool
269
     */
270
    public function destroy(): bool
271
    {
272
        $primary = $this->primary;
273
        $id = $this->data->$primary;
274
275
        if (empty($id)) {
276
            return false;
277
        }
278
279
        $destroy = $this->delete($this->primary . " = :id", "id={$id}");
280
        return $destroy;
281
    }
282
283
    /**
284
     * @return bool
285
     */
286
    protected function required(): bool
287
    {
288
        $data = (array)$this->data();
289
        foreach ($this->required as $field) {
290
            if (empty($data[$field])) {
291
                return false;
292
            }
293
        }
294
        return true;
295
    }
296
297
    /**
298
     * @return array|null
299
     */
300
    protected function safe(): ?array
301
    {
302
        $safe = (array)$this->data;
303
        unset($safe[$this->primary]);
304
305
        return $safe;
306
    }
307
}
308