回答編集履歴
1
内容の修正
test
CHANGED
@@ -6,7 +6,7 @@
|
|
6
6
|
|
7
7
|
いいと考えるか、ダメと考えるかで問われれば**ダメだと考えます**。
|
8
8
|
|
9
|
-
理由としてはいわゆるベースコントローラーはLaravelのパッケージをアップデートした際に上書きされますよね?
|
9
|
+
~~理由としてはいわゆるベースコントローラーはLaravelのパッケージをアップデートした際に上書きされますよね?~~
|
10
10
|
|
11
11
|
どうしても全コントローラーに共通の処理を組み込みたいのであれば、
|
12
12
|
|
@@ -16,6 +16,34 @@
|
|
16
16
|
|
17
17
|
|
18
18
|
|
19
|
+
■追記
|
20
|
+
|
21
|
+
コメントで指摘されたので、修正します。ベースコントローラーについて勘違いしていました。
|
22
|
+
|
23
|
+
よく読まずに回答して申し訳ございません。
|
24
|
+
|
25
|
+
App\Http\Controllers\Controller.php内で質問者が指すファイルとは別のベースコントローラが存在します。
|
26
|
+
|
27
|
+
```php
|
28
|
+
|
29
|
+
use Illuminate\Routing\Controller as BaseController;
|
30
|
+
|
31
|
+
```
|
32
|
+
|
33
|
+
こちらと早とちりしていました。
|
34
|
+
|
35
|
+
|
36
|
+
|
37
|
+
App\Http\Controllers\Controller.phpの修正は可能
|
38
|
+
|
39
|
+
Illuminate\Routing\Controller as BaseControllerの修正は避けるべき
|
40
|
+
|
41
|
+
|
42
|
+
|
43
|
+
と修正させていただきます。
|
44
|
+
|
45
|
+
|
46
|
+
|
19
47
|
ちなみにLaravelに用意されているmiddlewareの機能を考慮した上でのご質問でしょうか。
|
20
48
|
|
21
49
|
無理にベースコントローラーを改修することを考えるより、middlewareで実現できないか考えるほうが望ましいのではないでしょうか。
|