-
-
Notifications
You must be signed in to change notification settings - Fork 59
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore(deps): Update nextcloud/coding-standard and nextcloud/ocp #2510
Conversation
46297d5
to
913dd91
Compare
Signed-off-by: Louis Chemineau <louis@chmn.me>
Signed-off-by: Louis Chemineau <louis@chmn.me>
Signed-off-by: Louis Chemineau <louis@chmn.me>
Signed-off-by: Louis Chemineau <louis@chmn.me>
913dd91
to
4d2299a
Compare
Signed-off-by: Louis Chemineau <louis@chmn.me>
0c91e96
to
309fcb9
Compare
Signed-off-by: Louis Chemineau <louis@chmn.me>
309fcb9
to
561d8d1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would avoid putting Event as template parameter for IEventListener but apart from this all good.
use OCP\Group\Events\GroupDeletedEvent; | ||
use OCP\Group\Events\UserRemovedEvent; | ||
use OCP\Share\Events\ShareDeletedEvent; | ||
use OCP\User\Events\UserDeletedEvent; | ||
use Psr\Log\LoggerInterface; | ||
|
||
/** | ||
* @template-implements IEventListener<Event|NodeDeletedEvent|GroupDeletedEvent|ShareDeletedEvent|UserDeletedEvent> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* @template-implements IEventListener<Event|NodeDeletedEvent|GroupDeletedEvent|ShareDeletedEvent|UserDeletedEvent> | |
* @template-implements IEventListener<NodeDeletedEvent|GroupDeletedEvent|ShareDeletedEvent|UserDeletedEvent> |
is Event
really needed in there?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No idea, it works with it, and does not hurt
Fix #2402
Fix #2508
Fix #2502
Fix #2499
Fix #2491
Fix #2487
Fix #2483
Fix #2471
Fix #2463
Fix #2454
Fix #2442
Fix #2429
Fix #2422
Fix #2416