Scrutinizer GitHub App not installed

We could not synchronize checks via GitHub's checks API since Scrutinizer's GitHub App is not installed for this repository.

Install GitHub App

Test Setup Failed
Pull Request — master (#3332)
by Cristian
31:35 queued 15:14
created

Read::disableDetailsRow()   A

Complexity

Conditions 1
Paths 1

Size

Total Lines 3
Code Lines 1

Duplication

Lines 0
Ratio 0 %

Importance

Changes 0
Metric Value
cc 1
eloc 1
nc 1
nop 0
dl 0
loc 3
rs 10
c 0
b 0
f 0
1
<?php
2
3
namespace Backpack\CRUD\app\Library\CrudPanel\Traits;
4
5
/**
6
 * Properties and methods used by the List operation.
7
 */
8
trait Read
9
{
10
    /**
11
     * Find and retrieve the id of the current entry.
12
     *
13
     * @return int|bool The id in the db or false.
14
     */
15
    public function getCurrentEntryId()
16
    {
17
        if ($this->entry) {
18
            return $this->entry->getKey();
19
        }
20
21
        $params = \Route::current()->parameters();
22
23
        return  // use the entity name to get the current entry
24
                // this makes sure the ID is corrent even for nested resources
25
                $this->getRequest()->input($this->entity_name) ??
0 ignored issues
show
Bug introduced by
It seems like getRequest() 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 ignore-call  annotation

25
                $this->/** @scrutinizer ignore-call */ 
26
                       getRequest()->input($this->entity_name) ??
Loading history...
26
                // otherwise use the next to last parameter
27
                array_values($params)[count($params) - 1] ??
28
                // otherwise return false
29
                false;
30
    }
31
32
    /**
33
     * Find and retrieve the current entry.
34
     *
35
     * @return \Illuminate\Database\Eloquent\Model|bool The row in the db or false.
36
     */
37
    public function getCurrentEntry()
38
    {
39
        $id = $this->getCurrentEntryId();
40
41
        if (! $id) {
42
            return false;
43
        }
44
45
        return $this->getEntry($id);
46
    }
47
48
    /**
49
     * Find and retrieve an entry in the database or fail.
50
     *
51
     * @param int The id of the row in the db to fetch.
0 ignored issues
show
Bug introduced by
The type Backpack\CRUD\app\Library\CrudPanel\Traits\The was not found. Maybe you did not declare it correctly or list all dependencies?

The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. excluded_paths: ["lib/*"], you can move it to the dependency path list as follows:

filter:
    dependency_paths: ["lib/*"]

For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths

Loading history...
52
     *
53
     * @return \Illuminate\Database\Eloquent\Model The row in the db.
54
     */
55
    public function getEntry($id)
56
    {
57
        if (! $this->entry) {
58
            $this->entry = $this->model->findOrFail($id);
0 ignored issues
show
Bug Best Practice introduced by
The property entry does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.
Loading history...
59
            $this->entry = $this->entry->withFakes();
60
        }
61
62
        return $this->entry;
63
    }
64
65
    /**
66
     * Find and retrieve an entry in the database or fail.
67
     *
68
     * @param int The id of the row in the db to fetch.
69
     *
70
     * @return \Illuminate\Database\Eloquent\Model The row in the db.
71
     */
72
    public function getEntryWithoutFakes($id)
73
    {
74
        return $this->model->findOrFail($id);
75
    }
76
77
    /**
78
     * Make the query JOIN all relationships used in the columns, too,
79
     * so there will be less database queries overall.
80
     */
81
    public function autoEagerLoadRelationshipColumns()
82
    {
83
        $relationships = $this->getColumnsRelationships();
0 ignored issues
show
Bug introduced by
It seems like getColumnsRelationships() 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 ignore-call  annotation

83
        /** @scrutinizer ignore-call */ 
84
        $relationships = $this->getColumnsRelationships();
Loading history...
84
85
        if (count($relationships)) {
86
            $this->with($relationships);
0 ignored issues
show
Bug introduced by
It seems like with() 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 ignore-call  annotation

86
            $this->/** @scrutinizer ignore-call */ 
87
                   with($relationships);
Loading history...
87
        }
88
    }
89
90
    /**
91
     * Get all entries from the database.
92
     *
93
     * @return array|\Illuminate\Database\Eloquent\Collection
94
     */
95
    public function getEntries()
96
    {
97
        $this->autoEagerLoadRelationshipColumns();
98
99
        $entries = $this->query->get();
100
101
        // add the fake columns for each entry
102
        foreach ($entries as $key => $entry) {
103
            $entry->addFakes($this->getFakeColumnsAsArray());
0 ignored issues
show
Bug introduced by
It seems like getFakeColumnsAsArray() 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 ignore-call  annotation

103
            $entry->addFakes($this->/** @scrutinizer ignore-call */ getFakeColumnsAsArray());
Loading history...
104
        }
105
106
        return $entries;
107
    }
108
109
    /**
110
     * Enable the DETAILS ROW functionality:.
111
     *
112
     * In the table view, show a plus sign next to each entry.
113
     * When clicking that plus sign, an AJAX call will bring whatever content you want from the EntityCrudController::showDetailsRow($id) and show it to the user.
114
     */
115
    public function enableDetailsRow()
116
    {
117
        $this->setOperationSetting('detailsRow', true);
0 ignored issues
show
Bug introduced by
It seems like setOperationSetting() 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 ignore-call  annotation

117
        $this->/** @scrutinizer ignore-call */ 
118
               setOperationSetting('detailsRow', true);
Loading history...
118
    }
119
120
    /**
121
     * Disable the DETAILS ROW functionality:.
122
     */
123
    public function disableDetailsRow()
124
    {
125
        $this->setOperationSetting('detailsRow', false);
126
    }
127
128
    /**
129
     * Add two more columns at the beginning of the ListEntrie table:
130
     * - one shows the checkboxes needed for bulk actions
131
     * - one is blank, in order for evenual detailsRow or expand buttons
132
     * to be in a separate column.
133
     */
134
    public function enableBulkActions()
135
    {
136
        if ($this->getOperationSetting('bulkActions') == true) {
0 ignored issues
show
Bug introduced by
It seems like getOperationSetting() 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 ignore-call  annotation

136
        if ($this->/** @scrutinizer ignore-call */ getOperationSetting('bulkActions') == true) {
Loading history...
137
            return;
138
        }
139
140
        $this->setOperationSetting('bulkActions', true);
141
142
        $this->addColumn([
0 ignored issues
show
Bug introduced by
It seems like addColumn() 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 ignore-call  annotation

142
        $this->/** @scrutinizer ignore-call */ 
143
               addColumn([
Loading history...
143
            'type'            => 'checkbox',
144
            'name'            => 'bulk_actions',
145
            'label'           => ' <input type="checkbox" class="crud_bulk_actions_main_checkbox" style="width: 16px; height: 16px;" />',
146
            'priority'        => 0,
147
            'searchLogic'     => false,
148
            'orderable'       => false,
149
            'visibleInTable'  => true,
150
            'visibleInModal'  => false,
151
            'visibleInExport' => false,
152
            'visibleInShow'   => false,
153
            'hasActions'      => true,
154
        ])->makeFirstColumn();
155
156
        $this->addColumn([
157
            'type'            => 'custom_html',
158
            'name'            => 'blank_first_column',
159
            'label'           => ' ',
160
            'priority'        => 0,
161
            'searchLogic'     => false,
162
            'orderable'       => false,
163
            'visibleInTabel'  => true,
164
            'visibleInModal'  => false,
165
            'visibleInExport' => false,
166
            'visibleInShow'   => false,
167
            'hasActions'      => true,
168
        ])->makeFirstColumn();
169
    }
170
171
    /**
172
     * Remove the two columns needed for bulk actions.
173
     */
174
    public function disableBulkActions()
175
    {
176
        $this->setOperationSetting('bulkActions', false);
177
178
        $this->removeColumn('bulk_actions');
0 ignored issues
show
Bug introduced by
It seems like removeColumn() 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 ignore-call  annotation

178
        $this->/** @scrutinizer ignore-call */ 
179
               removeColumn('bulk_actions');
Loading history...
179
        $this->removeColumn('blank_first_column');
180
    }
181
182
    /**
183
     * Set the number of rows that should be show on the list view.
184
     */
185
    public function setDefaultPageLength($value)
186
    {
187
        if ($value !== 0) {
188
            $this->setOperationSetting('defaultPageLength', $value);
189
        } else {
190
            abort(500, 'You should not use 0 as a key in paginator. If you are looking for "ALL" option, use -1 instead.');
191
        }
192
    }
193
194
    /**
195
     * Get the number of rows that should be show on the list view.
196
     *
197
     * @return int
198
     */
199
    public function getDefaultPageLength()
200
    {
201
        return $this->getOperationSetting('defaultPageLength') ?? config('backpack.crud.operations.list.defaultPageLength') ?? 25;
202
    }
203
204
    /**
205
     * If a custom page length was specified as default, make sure it
206
     * also show up in the page length menu.
207
     */
208
    public function addCustomPageLengthToPageLengthMenu()
209
    {
210
        $values = $this->getOperationSetting('pageLengthMenu')[0];
211
        $labels = $this->getOperationSetting('pageLengthMenu')[1];
212
        // this is a condition that should be always true.
213
        if (is_array($values) && is_array($labels)) {
214
            $position = array_search($this->getDefaultPageLength(), $values);
215
            // if position is not false we already have that value in the pagination array
216
            // we are just going to make it first element in array
217
            if ($position !== false) {
218
                array_unshift($values, $this->getDefaultPageLength());
219
                array_unshift($labels, $labels[$position]);
220
            } else {
221
                // if it's not in array we add it as the first element
222
                array_unshift($values, $this->getDefaultPageLength());
223
                array_unshift($labels, $this->getDefaultPageLength());
224
            }
225
            //now make it unique.
226
            $values = array_unique($values);
227
            $labels = array_unique($labels);
228
        }
229
230
        $this->setOperationSetting('pageLengthMenu', [$values, $labels]);
231
    }
232
233
    /**
234
     * Specify array of available page lengths on the list view.
235
     *
236
     * @param array|int $menu
237
     *
238
     * https://backpackforlaravel.com/docs/4.1/crud-cheat-sheet#page-length
239
     */
240
    public function setPageLengthMenu($menu)
241
    {
242
        if (is_array($menu)) {
243
            // start checking $menu integrity
244
            if (count($menu) !== count($menu, COUNT_RECURSIVE)) {
245
                // developer defined as setPageLengthMenu([[50, 100, 300]]) or setPageLengthMenu([[50, 100, 300],['f','h','t']])
246
                // we will apply the same labels as the values to the menu if developer didn't
247
                if (! in_array(0, $menu[0])) {
248
                    if (! isset($menu[1]) || ! is_array($menu[1])) {
249
                        $menu[1] = $menu[0];
250
                    }
251
                } else {
252
                    abort(500, 'You should not use 0 as a key in paginator. If you are looking for "ALL" option, use -1 instead.');
253
                }
254
            } else {
255
                // developer defined setPageLengthMenu([10 => 'f', 100 => 'h', 300 => 't']) OR setPageLengthMenu([50, 100, 300])
256
                $menu = $this->buildPageLengthMenuFromArray($menu);
257
            }
258
        } else {
259
            // developer added only a single value setPageLengthMenu(10)
260
            if ($menu !== 0) {
261
                $menu = [[$menu], [$menu]];
262
            } else {
263
                abort(500, 'You should not use 0 as a key in paginator. If you are looking for "ALL" option, use -1 instead.');
264
            }
265
        }
266
267
        $this->setOperationSetting('pageLengthMenu', $menu);
268
    }
269
270
    /**
271
     * Builds the menu from the given array. It works out with two different types of arrays:
272
     *  [1, 2, 3] AND [1 => 'one', 2 => 'two', 3 => 'three'].
273
     *
274
     * @param array $menu
275
     * @return array
276
     */
277
    private function buildPageLengthMenuFromArray($menu)
278
    {
279
        // check if the values of the array are strings, in case developer defined:
280
        // setPageLengthMenu([0 => 'f', 100 => 'h', 300 => 't'])
281
        if (count(array_filter(array_values($menu), 'is_string')) > 0) {
282
            $values = array_keys($menu);
283
            $labels = array_values($menu);
284
285
            if (! in_array(0, $values)) {
286
                return [$values, $labels];
287
            }
288
289
            abort(500, 'You should not use 0 as a key in paginator. If you are looking for "ALL" option, use -1 instead.');
290
        } else {
291
            // developer defined length as setPageLengthMenu([50, 100, 300])
292
            // we will use the same values as labels
293
            if (! in_array(0, $menu)) {
294
                return [$menu, $menu];
295
            }
296
297
            abort(500, 'You should not use 0 as a key in paginator. If you are looking for "ALL" option, use -1 instead.');
298
        }
299
    }
300
301
    /**
302
     * Get page length menu for the list view.
303
     *
304
     * @return array
305
     */
306
    public function getPageLengthMenu()
307
    {
308
        // if we have a 2D array, update all the values in the right hand array to their translated values
309
        if (isset($this->getOperationSetting('pageLengthMenu')[1]) && is_array($this->getOperationSetting('pageLengthMenu')[1])) {
310
            $aux = $this->getOperationSetting('pageLengthMenu');
311
            foreach ($this->getOperationSetting('pageLengthMenu')[1] as $key => $val) {
312
                $aux[1][$key] = trans($val);
313
            }
314
            $this->setOperationSetting('pageLengthMenu', $aux);
315
        }
316
        $this->addCustomPageLengthToPageLengthMenu();
317
318
        return $this->getOperationSetting('pageLengthMenu');
319
    }
320
321
    /*
322
    |--------------------------------------------------------------------------
323
    |                                EXPORT BUTTONS
324
    |--------------------------------------------------------------------------
325
    */
326
327
    /**
328
     * Tell the list view to show the DataTables export buttons.
329
     */
330
    public function enableExportButtons()
331
    {
332
        $this->setOperationSetting('exportButtons', true);
333
    }
334
335
    /**
336
     * Check if export buttons are enabled for the table view.
337
     *
338
     * @return bool
339
     */
340
    public function exportButtons()
341
    {
342
        return $this->getOperationSetting('exportButtons') ?? false;
343
    }
344
}
345