Skip to content

Merge pull request #1153 from nextcloud/bugfix/noid/share-error #1507

Merge pull request #1153 from nextcloud/bugfix/noid/share-error

Merge pull request #1153 from nextcloud/bugfix/noid/share-error #1507

Triggered via push November 10, 2023 08:58
Status Success
Total duration 5m 57s
Artifacts

lint.yml

on: push
Matrix: lint-php
Fit to window
Zoom out
Zoom in

Annotations

19 warnings
lint-php (max): lib/Controller/NotesController.php#L325
Line exceeds 120 characters; contains 128 characters
lint-php (max): lib/Controller/NotesController.php#L326
Line exceeds 120 characters; contains 122 characters
lint-php (max): lib/AppInfo/DashboardWidget.php#L77
Line exceeds 120 characters; contains 141 characters
lint-php (max): lib/AppInfo/DashboardWidget.php#L80
Line exceeds 120 characters; contains 150 characters
lint-php (max): lib/AppInfo/Application.php#L15
Possibly zero references to use statement for classlike/namespace IShare (\OCP\Share\IShare)
lint-php (max): lib/AppInfo/Application.php#L33
Saw reference to \OCP\Share\Events\BeforeShareCreatedEvent declared at vendor/nextcloud/ocp/OCP/Share/Events/BeforeShareCreatedEvent.php:34 which is also declared at tests/stubs/ocp.php:7. This may lead to confusing errors. It may be possible to exclude the class that isn't used with exclude_file_list. In addition to normal ways to suppress issues, this issue type can be suppressed on either of the class definitions if it is impractical to exclude one file.
lint-php (max): lib/AppInfo/Application.php#L35
Saw reference to \OCP\Share\Events\BeforeShareCreatedEvent declared at vendor/nextcloud/ocp/OCP/Share/Events/BeforeShareCreatedEvent.php:34 which is also declared at tests/stubs/ocp.php:7. This may lead to confusing errors. It may be possible to exclude the class that isn't used with exclude_file_list. In addition to normal ways to suppress issues, this issue type can be suppressed on either of the class definitions if it is impractical to exclude one file.
lint-php (max): lib/AppInfo/Application.php#L46
Call to deprecated function \OCP\EventDispatcher\GenericEvent::getSubject() defined at vendor/nextcloud/ocp/OCP/EventDispatcher/GenericEvent.php:70
lint-php (max): lib/AppInfo/BeforeShareCreatedListener.php#L23
Reference to undeclared property \OCA\Notes\AppInfo\BeforeShareCreatedListener->logger (Did you mean $logger)
lint-php (max): lib/AppInfo/BeforeShareCreatedListener.php#L27
Saw reference to \OCP\Share\Events\BeforeShareCreatedEvent declared at vendor/nextcloud/ocp/OCP/Share/Events/BeforeShareCreatedEvent.php:34 which is also declared at tests/stubs/ocp.php:7. This may lead to confusing errors. It may be possible to exclude the class that isn't used with exclude_file_list. In addition to normal ways to suppress issues, this issue type can be suppressed on either of the class definitions if it is impractical to exclude one file.
lint-php (min): lib/Controller/NotesController.php#L325
Line exceeds 120 characters; contains 128 characters
lint-php (min): lib/Controller/NotesController.php#L326
Line exceeds 120 characters; contains 122 characters
lint-php (min): lib/AppInfo/DashboardWidget.php#L77
Line exceeds 120 characters; contains 141 characters
lint-php (min): lib/AppInfo/DashboardWidget.php#L80
Line exceeds 120 characters; contains 150 characters
lint-php (min): lib/AppInfo/Application.php#L15
Possibly zero references to use statement for classlike/namespace IShare (\OCP\Share\IShare)
lint-php (min): lib/AppInfo/Application.php#L46
Call to deprecated function \OCP\EventDispatcher\GenericEvent::getSubject() defined at vendor/nextcloud/ocp/OCP/EventDispatcher/GenericEvent.php:70
lint-php (min): lib/AppInfo/BeforeShareCreatedListener.php#L23
Reference to undeclared property \OCA\Notes\AppInfo\BeforeShareCreatedListener->logger (Did you mean $logger)
lint-php (min): lib/AppInfo/BeforeShareCreatedListener.php#L60
Reference to undeclared property \OCA\Notes\AppInfo\BeforeShareCreatedListener->logger
lint-js
The following actions uses node12 which is deprecated and will be forced to run on node16: xt0rted/stylelint-problem-matcher@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/