Conditions | 1 |
Paths | 1 |
Total Lines | 101 |
Code Lines | 75 |
Lines | 0 |
Ratio | 0 % |
Changes | 2 | ||
Bugs | 1 | Features | 0 |
Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.
For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.
Commonly applied refactorings include:
If many parameters/temporary variables are present:
1 | <?php |
||
13 | public function run() |
||
14 | { |
||
15 | DBM::Permission()::firstOrCreate( |
||
16 | ['slug' => 'browse', 'prefix' => 'permission'], |
||
17 | ['name' => 'Browse', 'slug' => 'browse', 'prefix' => 'permission'] |
||
18 | ); |
||
19 | DBM::Permission()::firstOrCreate( |
||
20 | ['slug' => 'create', 'prefix' => 'permission'], |
||
21 | ['name' => 'Create', 'slug' => 'create', 'prefix' => 'permission'] |
||
22 | ); |
||
23 | DBM::Permission()::firstOrCreate( |
||
24 | ['slug' => 'update', 'prefix' => 'permission'], |
||
25 | ['name' => 'Update', 'slug' => 'update', 'prefix' => 'permission'] |
||
26 | ); |
||
27 | DBM::Permission()::firstOrCreate( |
||
28 | ['slug' => 'delete', 'prefix' => 'permission'], |
||
29 | ['name' => 'Delete', 'slug' => 'delete', 'prefix' => 'permission'] |
||
30 | ); |
||
31 | DBM::Permission()::firstOrCreate( |
||
32 | ['slug' => 'browse', 'prefix' => 'database'], |
||
33 | ['name' => 'Browse', 'slug' => 'browse', 'prefix' => 'database'] |
||
34 | ); |
||
35 | DBM::Permission()::firstOrCreate( |
||
36 | ['slug' => 'create', 'prefix' => 'database'], |
||
37 | ['name' => 'Create', 'slug' => 'create', 'prefix' => 'database'] |
||
38 | ); |
||
39 | DBM::Permission()::firstOrCreate( |
||
40 | ['slug' => 'update', 'prefix' => 'database'], |
||
41 | ['name' => 'Update', 'slug' => 'update', 'prefix' => 'database'] |
||
42 | ); |
||
43 | DBM::Permission()::firstOrCreate( |
||
44 | ['slug' => 'delete', 'prefix' => 'database'], |
||
45 | ['name' => 'Delete', 'slug' => 'delete', 'prefix' => 'database'] |
||
46 | ); |
||
47 | DBM::Permission()::firstOrCreate( |
||
48 | ['slug' => 'browse', 'prefix' => 'crud'], |
||
49 | ['name' => 'Browse', 'slug' => 'browse', 'prefix' => 'crud'] |
||
50 | ); |
||
51 | DBM::Permission()::firstOrCreate( |
||
52 | ['slug' => 'create', 'prefix' => 'crud'], |
||
53 | ['name' => 'Create', 'slug' => 'create', 'prefix' => 'crud'] |
||
54 | ); |
||
55 | DBM::Permission()::firstOrCreate( |
||
56 | ['slug' => 'update', 'prefix' => 'crud'], |
||
57 | ['name' => 'Update', 'slug' => 'update', 'prefix' => 'crud'] |
||
58 | ); |
||
59 | DBM::Permission()::firstOrCreate( |
||
60 | ['slug' => 'delete', 'prefix' => 'crud'], |
||
61 | ['name' => 'Delete', 'slug' => 'delete', 'prefix' => 'crud'] |
||
62 | ); |
||
63 | DBM::Permission()::firstOrCreate( |
||
64 | ['slug' => 'browse', 'prefix' => 'relationship'], |
||
65 | ['name' => 'Browse', 'slug' => 'browse', 'prefix' => 'relationship'] |
||
66 | ); |
||
67 | DBM::Permission()::firstOrCreate( |
||
68 | ['slug' => 'create', 'prefix' => 'relationship'], |
||
69 | ['name' => 'Create', 'slug' => 'create', 'prefix' => 'relationship'] |
||
70 | ); |
||
71 | DBM::Permission()::firstOrCreate( |
||
72 | ['slug' => 'update', 'prefix' => 'relationship'], |
||
73 | ['name' => 'Update', 'slug' => 'update', 'prefix' => 'relationship'] |
||
74 | ); |
||
75 | DBM::Permission()::firstOrCreate( |
||
76 | ['slug' => 'delete', 'prefix' => 'relationship'], |
||
77 | ['name' => 'Delete', 'slug' => 'delete', 'prefix' => 'relationship'] |
||
78 | ); |
||
79 | DBM::Permission()::firstOrCreate( |
||
80 | ['slug' => 'browse', 'prefix' => 'record'], |
||
81 | ['name' => 'Browse', 'slug' => 'browse', 'prefix' => 'record'] |
||
82 | ); |
||
83 | DBM::Permission()::firstOrCreate( |
||
84 | ['slug' => 'create', 'prefix' => 'record'], |
||
85 | ['name' => 'Create', 'slug' => 'create', 'prefix' => 'record'] |
||
86 | ); |
||
87 | DBM::Permission()::firstOrCreate( |
||
88 | ['slug' => 'update', 'prefix' => 'record'], |
||
89 | ['name' => 'Update', 'slug' => 'update', 'prefix' => 'record'] |
||
90 | ); |
||
91 | DBM::Permission()::firstOrCreate( |
||
92 | ['slug' => 'delete', 'prefix' => 'record'], |
||
93 | ['name' => 'Delete', 'slug' => 'delete', 'prefix' => 'record'] |
||
94 | ); |
||
95 | DBM::Permission()::firstOrCreate( |
||
96 | ['slug' => 'browse', 'prefix' => 'backup'], |
||
97 | ['name' => 'Browse', 'slug' => 'browse', 'prefix' => 'backup'] |
||
98 | ); |
||
99 | DBM::Permission()::firstOrCreate( |
||
100 | ['slug' => 'create', 'prefix' => 'backup'], |
||
101 | ['name' => 'Create', 'slug' => 'create', 'prefix' => 'backup'] |
||
102 | ); |
||
103 | DBM::Permission()::firstOrCreate( |
||
104 | ['slug' => 'restore', 'prefix' => 'backup'], |
||
105 | ['name' => 'Restore', 'slug' => 'restore', 'prefix' => 'backup'] |
||
106 | ); |
||
107 | DBM::Permission()::firstOrCreate( |
||
108 | ['slug' => 'delete', 'prefix' => 'backup'], |
||
109 | ['name' => 'Delete', 'slug' => 'delete', 'prefix' => 'backup'] |
||
110 | ); |
||
111 | DBM::Permission()::firstOrCreate( |
||
112 | ['slug' => 'download', 'prefix' => 'backup'], |
||
113 | ['name' => 'Download', 'slug' => 'download', 'prefix' => 'backup'] |
||
114 | ); |
||
117 |