1 | <?php |
||||
2 | |||||
3 | namespace OwenIt\Auditing; |
||||
4 | |||||
5 | use Illuminate\Support\Facades\Config; |
||||
6 | use OwenIt\Auditing\Contracts\Auditable; |
||||
0 ignored issues
–
show
|
|||||
7 | use OwenIt\Auditing\Events\DispatchAudit; |
||||
8 | use OwenIt\Auditing\Events\DispatchingAudit; |
||||
9 | use OwenIt\Auditing\Facades\Auditor; |
||||
0 ignored issues
–
show
This use statement conflicts with another class in this namespace,
OwenIt\Auditing\Auditor . Consider defining an alias.
Let?s assume that you have a directory layout like this: .
|-- OtherDir
| |-- Bar.php
| `-- Foo.php
`-- SomeDir
`-- Foo.php
and let?s assume the following content of // Bar.php
namespace OtherDir;
use SomeDir\Foo; // This now conflicts the class OtherDir\Foo
If both files PHP Fatal error: Cannot use SomeDir\Foo as Foo because the name is already in use in OtherDir/Foo.php
However, as // Bar.php
namespace OtherDir;
use SomeDir\Foo as SomeDirFoo; // There is no conflict anymore.
![]() |
|||||
10 | |||||
11 | class AuditableObserver |
||||
12 | { |
||||
13 | /** |
||||
14 | * Is the model being restored? |
||||
15 | * |
||||
16 | * @var bool |
||||
17 | */ |
||||
18 | public static $restoring = false; |
||||
19 | |||||
20 | /** |
||||
21 | * Handle the retrieved event. |
||||
22 | * |
||||
23 | * @return void |
||||
24 | */ |
||||
25 | public function retrieved(Auditable $model) |
||||
26 | { |
||||
27 | 5 | $this->dispatchAudit($model->setAuditEvent('retrieved')); |
|||
28 | } |
||||
29 | 5 | ||||
30 | /** |
||||
31 | * Handle the created event. |
||||
32 | * |
||||
33 | * @return void |
||||
34 | */ |
||||
35 | public function created(Auditable $model) |
||||
36 | { |
||||
37 | $this->dispatchAudit($model->setAuditEvent('created')); |
||||
38 | } |
||||
39 | 15 | ||||
40 | /** |
||||
41 | 15 | * Handle the updated event. |
|||
42 | * |
||||
43 | * @return void |
||||
44 | */ |
||||
45 | public function updated(Auditable $model) |
||||
46 | { |
||||
47 | // Ignore the updated event when restoring |
||||
48 | if (!static::$restoring) { |
||||
49 | $this->dispatchAudit($model->setAuditEvent('updated')); |
||||
50 | } |
||||
51 | 4 | } |
|||
52 | |||||
53 | /** |
||||
54 | 4 | * Handle the deleted event. |
|||
55 | 3 | * |
|||
56 | * @return void |
||||
57 | */ |
||||
58 | public function deleted(Auditable $model) |
||||
59 | { |
||||
60 | $this->dispatchAudit($model->setAuditEvent('deleted')); |
||||
61 | } |
||||
62 | |||||
63 | /** |
||||
64 | * Handle the restoring event. |
||||
65 | * |
||||
66 | 2 | * @return void |
|||
67 | */ |
||||
68 | 2 | public function restoring(Auditable $model) |
|||
0 ignored issues
–
show
The parameter
$model is not used and could be removed.
(
Ignorable by Annotation
)
If this is a false-positive, you can also ignore this issue in your code via the
This check looks for parameters that have been defined for a function or method, but which are not used in the method body. ![]() |
|||||
69 | { |
||||
70 | // When restoring a model, an updated event is also fired. |
||||
71 | // By keeping track of the main event that took place, |
||||
72 | // we avoid creating a second audit with wrong values |
||||
73 | static::$restoring = true; |
||||
74 | } |
||||
75 | |||||
76 | /** |
||||
77 | * Handle the restored event. |
||||
78 | 1 | * |
|||
79 | * @return void |
||||
80 | */ |
||||
81 | public function restored(Auditable $model) |
||||
82 | { |
||||
83 | 1 | $this->dispatchAudit($model->setAuditEvent('restored')); |
|||
84 | |||||
85 | // Once the model is restored, we need to put everything back |
||||
86 | // as before, in case a legitimate update event is fired |
||||
87 | static::$restoring = false; |
||||
88 | } |
||||
89 | |||||
90 | protected function dispatchAudit(Auditable $model): void |
||||
91 | { |
||||
92 | if (!$model->readyForAuditing()) { |
||||
93 | 1 | return; |
|||
94 | } |
||||
95 | 1 | ||||
96 | // @phpstan-ignore method.notFound |
||||
97 | $model->preloadResolverData(); |
||||
98 | if (!Config::get('audit.queue.enable', false)) { |
||||
99 | 1 | Auditor::execute($model); |
|||
100 | return; |
||||
101 | } |
||||
102 | 15 | ||||
103 | if (!$this->fireDispatchingAuditEvent($model)) { |
||||
104 | 15 | return; |
|||
105 | 8 | } |
|||
106 | |||||
107 | // Unload the relations to prevent large amounts of unnecessary data from being serialized. |
||||
108 | 15 | $model->withoutRelations(); |
|||
109 | 15 | app()->make('events')->dispatch(new DispatchAudit($model)); |
|||
110 | } |
||||
111 | |||||
112 | /** |
||||
113 | 15 | * Fire the Auditing event. |
|||
114 | */ |
||||
115 | protected function fireDispatchingAuditEvent(Auditable $model): bool |
||||
116 | { |
||||
117 | return app()->make('events') |
||||
118 | 15 | ->until(new DispatchingAudit($model)) !== false; |
|||
119 | } |
||||
120 | } |
||||
121 |
Let?s assume that you have a directory layout like this:
and let?s assume the following content of
Bar.php
:If both files
OtherDir/Foo.php
andSomeDir/Foo.php
are loaded in the same runtime, you will see a PHP error such as the following:PHP Fatal error: Cannot use SomeDir\Foo as Foo because the name is already in use in OtherDir/Foo.php
However, as
OtherDir/Foo.php
does not necessarily have to be loaded and the error is only triggered if it is loaded beforeOtherDir/Bar.php
, this problem might go unnoticed for a while. In order to prevent this error from surfacing, you must import the namespace with a different alias: