faithgen /
laravel-sdk
| 1 | <?php |
||||
| 2 | |||||
| 3 | namespace FaithGen\SDK\Traits; |
||||
| 4 | |||||
| 5 | trait ExcludesColumns |
||||
| 6 | { |
||||
| 7 | /** |
||||
| 8 | * Excludes some columns from a query. |
||||
| 9 | * |
||||
| 10 | * @param $query |
||||
| 11 | * @param array $columns |
||||
| 12 | * @return mixed |
||||
| 13 | */ |
||||
| 14 | public function scopeExclude($query, array $columns = []) |
||||
| 15 | { |
||||
| 16 | $tableColumns = $this->getConnection()->getSchemaBuilder()->getColumnListing($this->getTable()); |
||||
|
0 ignored issues
–
show
Bug
introduced
by
Loading history...
It seems like
getTable() must be provided by classes using this trait. How about adding it as abstract method to this trait?
(
Ignorable by Annotation
)
If this is a false-positive, you can also ignore this issue in your code via the
Loading history...
|
|||||
| 17 | |||||
| 18 | $requiredColumns = collect($tableColumns)->filter(fn ($column) => ! in_array($column, $columns))->toArray(); |
||||
| 19 | |||||
| 20 | return $query->select($requiredColumns); |
||||
| 21 | } |
||||
| 22 | } |
||||
| 23 |