SebastianBerc /
Repositories
This project does not seem to handle request data directly as such no vulnerable execution paths were found.
include, or for example
via PHP's auto-loading mechanism.
These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more
| 1 | <?php |
||
| 2 | |||
| 3 | namespace SebastianBerc\Repositories\Services; |
||
| 4 | |||
| 5 | use Illuminate\Contracts\Container\Container as Application; |
||
| 6 | use Illuminate\Database\Eloquent\Builder; |
||
| 7 | use Illuminate\Database\Eloquent\Collection; |
||
| 8 | use Illuminate\Database\Eloquent\Model as Eloquent; |
||
| 9 | use Illuminate\Pagination\LengthAwarePaginator; |
||
| 10 | use SebastianBerc\Repositories\Contracts\ServiceInterface; |
||
| 11 | use SebastianBerc\Repositories\Repository; |
||
| 12 | use SebastianBerc\Repositories\Traits\Filterable; |
||
| 13 | use SebastianBerc\Repositories\Traits\Sortable; |
||
| 14 | |||
| 15 | /** |
||
| 16 | * Class DatabaseService. |
||
| 17 | * |
||
| 18 | * @author Sebastian Berć <[email protected]> |
||
| 19 | * @copyright Copyright (c) Sebastian Berć |
||
| 20 | */ |
||
| 21 | class DatabaseService implements ServiceInterface |
||
| 22 | { |
||
| 23 | use Filterable, Sortable; |
||
| 24 | |||
| 25 | /** |
||
| 26 | * Contains Laravel Application instance. |
||
| 27 | * |
||
| 28 | * @var Application |
||
| 29 | */ |
||
| 30 | protected $app; |
||
| 31 | |||
| 32 | /** |
||
| 33 | * Contains a repository instance. |
||
| 34 | * |
||
| 35 | * @var Repository |
||
| 36 | */ |
||
| 37 | protected $repository; |
||
| 38 | |||
| 39 | /** |
||
| 40 | * Contains model instance for fetch, and simple fetch methods. |
||
| 41 | * |
||
| 42 | * @var mixed |
||
| 43 | */ |
||
| 44 | protected $instance; |
||
| 45 | |||
| 46 | /** |
||
| 47 | * Create a new database service instance. |
||
| 48 | * |
||
| 49 | * @param Application $app |
||
| 50 | * @param Repository $repository |
||
| 51 | */ |
||
| 52 | 120 | public function __construct(Application $app, Repository $repository) |
|
| 53 | { |
||
| 54 | 120 | $this->app = $app; |
|
| 55 | 120 | $this->repository = $repository; |
|
| 56 | 120 | } |
|
| 57 | |||
| 58 | /** |
||
| 59 | * Get all of the models from the database. |
||
| 60 | * |
||
| 61 | * @param string[] $columns |
||
| 62 | * |
||
| 63 | * @return Collection |
||
| 64 | */ |
||
| 65 | 12 | public function all(array $columns = ['*']) |
|
| 66 | { |
||
| 67 | 12 | return $this->repository->makeQuery()->get($columns); |
|
| 68 | } |
||
| 69 | |||
| 70 | /** |
||
| 71 | * Create a new basic where query clause on model. |
||
| 72 | * |
||
| 73 | * @param string|array $column |
||
| 74 | * @param string $operator |
||
| 75 | * @param mixed $value |
||
| 76 | * @param string $boolean |
||
| 77 | * @param string[] $columns |
||
| 78 | * |
||
| 79 | * @return mixed |
||
| 80 | */ |
||
| 81 | 20 | public function where($column, $operator = '=', $value = null, $boolean = 'and', array $columns = ['*']) |
|
| 82 | { |
||
| 83 | 20 | return $this->repository->makeQuery()->where($column, $operator, $value, $boolean, $columns)->get($columns); |
|
| 84 | } |
||
| 85 | |||
| 86 | /** |
||
| 87 | * Paginate the given query. |
||
| 88 | * |
||
| 89 | * @param int $perPage |
||
| 90 | * @param string[] $columns |
||
| 91 | * |
||
| 92 | * @return \Illuminate\Contracts\Pagination\LengthAwarePaginator |
||
| 93 | */ |
||
| 94 | 8 | public function paginate($perPage = 15, array $columns = ['*']) |
|
| 95 | { |
||
| 96 | 8 | return $this->repository->makeQuery()->paginate($perPage, $columns); |
|
| 97 | } |
||
| 98 | |||
| 99 | /** |
||
| 100 | * Save a new model and return the instance. |
||
| 101 | * |
||
| 102 | * @param array $attributes |
||
| 103 | * |
||
| 104 | * @return Eloquent |
||
| 105 | */ |
||
| 106 | 6 | public function create(array $attributes = []) |
|
| 107 | { |
||
| 108 | 6 | return $this->repository->makeModel()->create($attributes); |
|
| 109 | } |
||
| 110 | |||
| 111 | /** |
||
| 112 | * Save or update the model in the database. |
||
| 113 | * |
||
| 114 | * @param mixed $identifier |
||
| 115 | * @param array $attributes |
||
| 116 | * |
||
| 117 | * @return Eloquent |
||
| 118 | */ |
||
| 119 | 6 | public function update($identifier, array $attributes = []) |
|
| 120 | { |
||
| 121 | $instance = $identifier instanceof Eloquent |
||
| 122 | 6 | ? $identifier |
|
| 123 | 6 | : $this->repository->makeQuery()->findOrFail($identifier); |
|
| 124 | |||
| 125 | 6 | $instance->fill($attributes); |
|
|
0 ignored issues
–
show
|
|||
| 126 | |||
| 127 | 6 | if ($instance->isDirty()) { |
|
|
0 ignored issues
–
show
The method
isDirty does only exist in Illuminate\Database\Eloquent\Model, but not in Illuminate\Database\Eloq...ase\Eloquent\Collection.
It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A
{
public function foo() { }
}
class B extends A
{
public function bar() { }
}
/**
* @param A|B $x
*/
function someFunction($x)
{
$x->foo(); // This call is fine as the method exists in A and B.
$x->bar(); // This method only exists in B and might cause an error.
}
Available Fixes
Loading history...
|
|||
| 128 | 6 | $instance->save(); |
|
|
0 ignored issues
–
show
The method
save does only exist in Illuminate\Database\Eloquent\Model, but not in Illuminate\Database\Eloq...ase\Eloquent\Collection.
It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A
{
public function foo() { }
}
class B extends A
{
public function bar() { }
}
/**
* @param A|B $x
*/
function someFunction($x)
{
$x->foo(); // This call is fine as the method exists in A and B.
$x->bar(); // This method only exists in B and might cause an error.
}
Available Fixes
Loading history...
|
|||
| 129 | 6 | } |
|
| 130 | |||
| 131 | 6 | return $instance; |
|
| 132 | } |
||
| 133 | |||
| 134 | /** |
||
| 135 | * Delete the model from the database. |
||
| 136 | * |
||
| 137 | * @param int $identifier |
||
| 138 | * |
||
| 139 | * @return bool|null |
||
| 140 | */ |
||
| 141 | 6 | public function delete($identifier) |
|
| 142 | { |
||
| 143 | 6 | return $this->repository->makeQuery() |
|
|
0 ignored issues
–
show
The method
delete does only exist in Illuminate\Database\Eloq...Database\Eloquent\Model, but not in Illuminate\Database\Eloquent\Collection.
It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A
{
public function foo() { }
}
class B extends A
{
public function bar() { }
}
/**
* @param A|B $x
*/
function someFunction($x)
{
$x->foo(); // This call is fine as the method exists in A and B.
$x->bar(); // This method only exists in B and might cause an error.
}
Available Fixes
Loading history...
|
|||
| 144 | 6 | ->findOrFail($identifier, [$this->repository->makeModel()->getKeyName()]) |
|
| 145 | 6 | ->delete(); |
|
| 146 | } |
||
| 147 | |||
| 148 | /** |
||
| 149 | * Find a model by its primary key. |
||
| 150 | * |
||
| 151 | * @param int $identifier |
||
| 152 | * @param string[] $columns |
||
| 153 | * |
||
| 154 | * @return Eloquent |
||
| 155 | */ |
||
| 156 | 28 | public function find($identifier, array $columns = ['*']) |
|
| 157 | { |
||
| 158 | 28 | return $this->repository->makeQuery()->find($identifier, $columns); |
|
| 159 | } |
||
| 160 | |||
| 161 | /** |
||
| 162 | * Find a model by its specified column and value. |
||
| 163 | * |
||
| 164 | * @param mixed $column |
||
| 165 | * @param mixed $value |
||
| 166 | * @param string[] $columns |
||
| 167 | * |
||
| 168 | * @return Eloquent |
||
| 169 | */ |
||
| 170 | 8 | public function findBy($column, $value, array $columns = ['*']) |
|
| 171 | { |
||
| 172 | 8 | return $this->where([$column => $value], '=', null, 'and', $columns)->first(); |
|
| 173 | } |
||
| 174 | |||
| 175 | /** |
||
| 176 | * Find a model by its specified columns and values presented as array. |
||
| 177 | * |
||
| 178 | * @param array $wheres |
||
| 179 | * @param string[] $columns |
||
| 180 | * |
||
| 181 | * @return Eloquent |
||
| 182 | */ |
||
| 183 | 8 | public function findWhere(array $wheres, array $columns = ['*']) |
|
| 184 | { |
||
| 185 | 8 | return $this->where($wheres, '=', null, 'and', $columns)->first(); |
|
| 186 | } |
||
| 187 | |||
| 188 | /** |
||
| 189 | * Returns total count of whole collection. |
||
| 190 | * |
||
| 191 | * @return int |
||
| 192 | */ |
||
| 193 | 4 | public function count() |
|
| 194 | { |
||
| 195 | 4 | $countBy = "{$this->repository->makeModel()->getTable()}.{$this->repository->makeModel()->getKeyName()}"; |
|
| 196 | |||
| 197 | 4 | return $this->repository->makeQuery()->count($countBy); |
|
| 198 | } |
||
| 199 | |||
| 200 | /** |
||
| 201 | * Fetch collection ordered and filtrated by specified columns for specified page as paginator. |
||
| 202 | * |
||
| 203 | * @param int $page |
||
| 204 | * @param int $perPage |
||
| 205 | * @param array $filter |
||
| 206 | * @param array $sort |
||
| 207 | * @param array $columns |
||
| 208 | * |
||
| 209 | * @return LengthAwarePaginator |
||
| 210 | */ |
||
| 211 | 12 | public function fetch($page = 1, $perPage = 15, array $columns = ['*'], array $filter = [], array $sort = []) |
|
| 212 | { |
||
| 213 | 12 | $this->instance = $this->repository->makeQuery(); |
|
| 214 | |||
| 215 | 12 | $this->multiFilterBy($filter)->multiSortBy($sort); |
|
| 216 | |||
| 217 | 12 | $this->parseAliases($this->instance); |
|
| 218 | |||
| 219 | 12 | $count = $this->instance->count(); |
|
| 220 | 12 | $items = $this->instance->forPage($page, $perPage)->get($columns); |
|
| 221 | |||
| 222 | $options = [ |
||
| 223 | 12 | 'path' => $this->app->make('request')->url(), |
|
| 224 | 12 | 'query' => compact('page', 'perPage'), |
|
| 225 | 12 | ]; |
|
| 226 | |||
| 227 | 12 | return new LengthAwarePaginator($items, $count, $perPage, $page, $options); |
|
| 228 | } |
||
| 229 | |||
| 230 | /** |
||
| 231 | * Fetch collection ordered and filtrated by specified columns for specified page. |
||
| 232 | * |
||
| 233 | * @param int $page |
||
| 234 | * @param int $perPage |
||
| 235 | * @param array $columns |
||
| 236 | * @param array $filter |
||
| 237 | * @param array $sort |
||
| 238 | * |
||
| 239 | * @return Collection |
||
| 240 | */ |
||
| 241 | 4 | public function simpleFetch($page = 1, $perPage = 15, array $columns = ['*'], array $filter = [], array $sort = []) |
|
| 242 | { |
||
| 243 | 4 | $this->instance = $this->repository->makeQuery(); |
|
| 244 | |||
| 245 | 4 | $this->multiFilterBy($filter)->multiSortBy($sort)->parseAliases($this->instance); |
|
| 246 | |||
| 247 | 4 | return $this->instance->forPage($page, $perPage)->get($columns); |
|
| 248 | } |
||
| 249 | |||
| 250 | /** |
||
| 251 | * Replace alias name in where closure with sub-queries from select. |
||
| 252 | * |
||
| 253 | * Example: |
||
| 254 | * |
||
| 255 | * SELECT table.*, (SELECT 1 FROM other_table ot WHERE ot.id = table.id) AS exists |
||
| 256 | * WHERE exists = 1; |
||
| 257 | * |
||
| 258 | * Will be converted to: |
||
| 259 | * |
||
| 260 | * SELECT table.*, (SELECT 1 FROM other_table ot WHERE ot.id = table.id) AS exists |
||
| 261 | * WHERE (SELECT 1 FROM other_table ot WHERE ot.id = table.id) = 1; |
||
| 262 | * |
||
| 263 | * @param Builder $query |
||
| 264 | * |
||
| 265 | * @return Builder |
||
| 266 | */ |
||
| 267 | 16 | protected function parseAliases(Builder $query) |
|
| 268 | { |
||
| 269 | 16 | $aliases = []; |
|
| 270 | |||
| 271 | 16 | if (!empty($query->getQuery()->columns)) { |
|
| 272 | 2 | $aliases = $this->getAliases($query); |
|
| 273 | 2 | } |
|
| 274 | |||
| 275 | 16 | if (!empty($aliases) && !empty($query->getQuery()->wheres)) { |
|
| 276 | $this->replaceAliases($query, $aliases); |
||
| 277 | } |
||
| 278 | |||
| 279 | 16 | return $query; |
|
| 280 | } |
||
| 281 | |||
| 282 | /** |
||
| 283 | * Get sub queries and aliases from select statement. |
||
| 284 | * |
||
| 285 | * @param Builder $query |
||
| 286 | * |
||
| 287 | * @return array |
||
| 288 | */ |
||
| 289 | 2 | protected function getAliases(Builder $query) |
|
| 290 | { |
||
| 291 | 2 | $aliases = []; |
|
| 292 | |||
| 293 | 2 | foreach ($query->getQuery()->columns as $column) { |
|
| 294 | 2 | if (preg_match("~AS (\w+)~i", $column, $matches)) { |
|
| 295 | $aliases[$query->getModel()->getTable() . '.' . $matches[1]] |
||
| 296 | = \DB::raw(str_replace($matches[0], '', $column)); |
||
| 297 | } |
||
| 298 | 2 | } |
|
| 299 | |||
| 300 | 2 | return $aliases; |
|
| 301 | } |
||
| 302 | |||
| 303 | /** |
||
| 304 | * Replace aliases in where statement with sub queries. |
||
| 305 | * |
||
| 306 | * @param Builder $query |
||
| 307 | * @param array $aliases |
||
| 308 | * |
||
| 309 | * @return void |
||
| 310 | */ |
||
| 311 | protected function replaceAliases(Builder $query, $aliases) |
||
| 312 | { |
||
| 313 | foreach ($query->getQuery()->wheres as $key => $value) { |
||
| 314 | if (in_array($value['column'], array_keys($aliases))) { |
||
| 315 | $query->getQuery()->wheres[$key]['column'] = $aliases[$value['column']]; |
||
| 316 | } |
||
| 317 | } |
||
| 318 | } |
||
| 319 | } |
||
| 320 |
It seems like the method you are trying to call exists only in some of the possible types.
Let’s take a look at an example:
Available Fixes
Add an additional type-check:
Only allow a single type to be passed if the variable comes from a parameter: