stable
Clone or download
Read-only
chore: request #37591 slevomat/coding-standard: 8.6.0 -> 8.15.0
Most of changes are because the sniff detecting useless annotations got better. Main changes: https://github.com/slevomat/coding-standard/releases/tag/8.15.0 https://github.com/slevomat/coding-standard/releases/tag/8.14.0 https://github.com/slevomat/coding-standard/releases/tag/8.13.0 https://github.com/slevomat/coding-standard/releases/tag/8.12.0 https://github.com/slevomat/coding-standard/releases/tag/8.11.0 https://github.com/slevomat/coding-standard/releases/tag/8.10.0 https://github.com/slevomat/coding-standard/releases/tag/8.9.0 https://github.com/slevomat/coding-standard/releases/tag/8.8.0 https://github.com/slevomat/coding-standard/releases/tag/8.7.0 Change-Id: I2c93bc0377ed4f26d86a4fbb685a388aab785e87
Modified Files
Name | ||||
---|---|---|---|---|
M | plugins/agiledashboard/include/AgileDashboard/AgileDashboardLegacyController.php | +0 | −1 | Go to diff View file |
M | plugins/agiledashboard/include/AgileDashboard/Milestone/Backlog/ProvidedAddedIdIsNotInPartOfTopBacklogException.php | +0 | −3 | Go to diff View file |
M | plugins/agiledashboard/include/AgileDashboard/REST/v1/CardRepresentation.class.php | +0 | −1 | Go to diff View file |
M | plugins/agiledashboard/include/AgileDashboard/REST/v1/Milestone/MilestoneElementMover.php | +0 | −3 | Go to diff View file |
M | plugins/agiledashboard/include/AgileDashboard/REST/v1/PatchAddBacklogItemsValidator.class.php | +0 | −1 | Go to diff View file |
M | plugins/agiledashboard/include/AgileDashboard/XMLImporter.class.php | +0 | −1 | Go to diff View file |
M | plugins/agiledashboard/include/Planning/ArtifactMilestone.class.php | +0 | −1 | Go to diff View file |
M | plugins/agiledashboard/include/Planning/PlanningFactory.class.php | +0 | −1 | Go to diff View file |
M | plugins/baseline/include/ServiceController.php | +0 | −2 | Go to diff View file |
M | plugins/botmattermost/include/BotMattermost/Router.php | +0 | −1 | Go to diff View file |
M | plugins/cardwall/include/Cardwall/CardwallLegacyController.php | +0 | −1 | Go to diff View file |
M | plugins/cardwall/include/Semantic/CardFieldXmlExporter.php | +0 | −3 | Go to diff View file |
M | plugins/cardwall/include/Semantic/CardFieldsPresenterBuilder.php | +0 | −2 | Go to diff View file |
M | plugins/cardwall/include/Semantic/FieldUsedInSemanticObjectChecker.php | +0 | −1 | Go to diff View file |
M | plugins/cardwall/include/SwimlineSoloNoMatchingColumns.class.php | +0 | −4 | Go to diff View file |
M | plugins/cardwall/include/SwimlineTrackerRenderer.class.php | +0 | −3 | Go to diff View file |
M | plugins/crosstracker/include/CrossTracker/CrossTrackerReportDao.php | +0 | −1 | Go to diff View file |
M | plugins/crosstracker/include/CrossTracker/Report/CSV/CSVExportController.php | +0 | −1 | Go to diff View file |
M | plugins/docman/include/DocmanSettingsSiteAdmin/FilesUploadLimits/DocmanFilesUploadLimitsAdminController.php | +0 | −1 | Go to diff View file |
M | plugins/docman/include/DocmanSettingsSiteAdmin/FilesUploadLimits/DocmanFilesUploadLimitsAdminSaveController.php | +0 | −1 | Go to diff View file |
M | plugins/docman/include/PermissionsPerGroup/DocmanGlobalAdminPermissionCollection.php | +0 | −3 | Go to diff View file |
M | plugins/docman/include/REST/v1/DocumentBeforeVersionCreationValidatorVisitor.php | +0 | −1 | Go to diff View file |
M | plugins/docman/include/REST/v1/Metadata/MetadataToCreate.php | +0 | −3 | Go to diff View file |
M | plugins/docman/tests/rest/Docman/DocmanItemsTest.php | +0 | −1 | Go to diff View file |
M | plugins/document/include/Config/Project/SearchView.php | +0 | −1 | Go to diff View file |
M | plugins/document/include/Config/Project/UpdateSearchView.php | +0 | −1 | Go to diff View file |
M | plugins/document/include/Tree/DocumentTreeController.php | +0 | −1 | Go to diff View file |
M | plugins/document/include/Tree/DocumentTreeProjectExtractor.php | +0 | −1 | Go to diff View file |
M | plugins/enalean_licensemanager/include/enalean_licensemanagerPlugin.php | +0 | −1 | Go to diff View file |
M | plugins/frs/include/FRS/REST/v1/PackageRepresentationBuilder.php | +0 | −3 | Go to diff View file |
M | plugins/git/include/Git/Driver/Gerrit/Template/TemplateFactory.class.php | +0 | −1 | Go to diff View file |
M | plugins/git/include/Git/Driver/Gerrit/UmbrellaProjectManager.class.php | +0 | −6 | Go to diff View file |
M | plugins/git/include/Git/Driver/Gerrit/UserAccountManager.class.php | +0 | −10 | Go to diff View file |
M | plugins/git/include/Git/GitPluginDefaultController.php | +0 | −1 | Go to diff View file |
M | plugins/git/include/Git/GitRepositoryBrowserController.php | +0 | −2 | Go to diff View file |
M | plugins/git/include/Git/Gitolite/SSHKey/Provider/GitoliteAdmin.php | +0 | −1 | Go to diff View file |
M | plugins/git/include/Git/HTTP/HTTPController.php | +0 | −2 | Go to diff View file |
M | plugins/git/include/Git/PermissionsPerGroup/CollectionOfUGroupsRepresentationFormatter.php | +0 | −1 | Go to diff View file |
M | plugins/git/include/Git/PermissionsPerGroup/CollectionOfUgroupsFormatter.php | +0 | −1 | Go to diff View file |
M | plugins/git/include/Git/RemoteServer/GerritServerFactory.class.php | +0 | −1 | Go to diff View file |
M | plugins/git/include/Git/RepositoryList/DaoByRepositoryPathSorter.php | +0 | −1 | Go to diff View file |
M | plugins/git/include/Git/RepositoryList/GitRepositoryListController.php | +0 | −2 | Go to diff View file |
M | plugins/git/include/Git/UserAccountManager.class.php | +0 | −2 | Go to diff View file |
M | plugins/git/include/GitDao.class.php | +0 | −3 | Go to diff View file |
M | plugins/git/include/GitLegacyURLRedirectController.php | +0 | −1 | Go to diff View file |
M | plugins/git/include/GitPHP/git/FileDiff.php | +0 | −1 | Go to diff View file |
M | plugins/git/include/GitPHP/git/TreeDiff.php | +0 | −1 | Go to diff View file |
M | plugins/git/include/gitPlugin.php | +0 | −2 | Go to diff View file |
M | plugins/gitlfs/include/Admin/IndexController.php | +0 | −1 | Go to diff View file |
M | plugins/gitlfs/include/Admin/IndexPostController.php | +0 | −1 | Go to diff View file |
M | plugins/gitlfs/include/Download/FileDownloaderController.php | +0 | −1 | Go to diff View file |
M | plugins/ldap/include/BindMembersConfirmController.php | +0 | −1 | Go to diff View file |
M | plugins/ldap/include/BindUgroupConfirmController.php | +0 | −1 | Go to diff View file |
M | plugins/ldap/include/GroupAutocompleteController.php | +0 | −1 | Go to diff View file |
M | plugins/ldap/include/GroupSyncAdminEmailNotificationsManager.class.php | +0 | −1 | Go to diff View file |
M | plugins/ldap/include/WelcomeDisplayController.php | +0 | −1 | Go to diff View file |
M | plugins/ldap/include/WelcomeUpdateController.php | +0 | −1 | Go to diff View file |
M | plugins/mediawiki/include/MediawikiUserGroupsMapper.class.php | +0 | −4 | Go to diff View file |
M | plugins/openidconnectclient/include/OpenIDConnectClient/LoginController.php | +0 | −1 | Go to diff View file |
M | plugins/openidconnectclient/include/OpenIDConnectClient/Provider/ProviderManager.php | +0 | −2 | Go to diff View file |
M | plugins/svn/include/SVN/Notifications/NotificationListBuilder.php | +0 | −3 | Go to diff View file |
M | plugins/svn/include/SVN/PermissionsPerGroup/PermissionPerGroupSVNGlobalAdminPermissionCollection.php | +0 | −3 | Go to diff View file |
M | plugins/svn/include/SVN/RedirectOldViewVCUrls.php | +0 | −1 | Go to diff View file |
M | plugins/svn/include/SVN/Repository/HookConfigSanitizer.php | +0 | −1 | Go to diff View file |
M | plugins/testmanagement/include/TestManagement/ArtifactFactory.php | +0 | −1 | Go to diff View file |
M | plugins/testmanagement/include/TestManagement/Breadcrumbs/AdmininistrationBreadcrumbs.php | +0 | −3 | Go to diff View file |
M | plugins/testmanagement/include/TestManagement/Breadcrumbs/NoCrumb.php | +0 | −3 | Go to diff View file |
M | plugins/testmanagement/include/TestManagement/REST/v1/ArtifactNodeBuilder.php | +0 | −4 | Go to diff View file |
M | plugins/testmanagement/include/TestManagement/REST/v1/DefinitionSelector.php | +0 | −2 | Go to diff View file |
M | plugins/testmanagement/include/TestManagement/REST/v1/ExecutionRepresentationBuilder.php | +0 | −2 | Go to diff View file |
M | plugins/testmanagement/include/TestManagement/REST/v1/ExecutionStatusUpdater.php | +0 | −1 | Go to diff View file |
M | plugins/testmanagement/include/TestManagement/REST/v1/TestStatusAccordingToStepsStatusChangesBuilder.php | +0 | −4 | Go to diff View file |
M | plugins/testmanagement/include/TestManagement/Step/Definition/Field/StepDefinition.php | +0 | −1 | Go to diff View file |
M | plugins/timetracking/include/Timetracking/REST/v1/ProjectResource.php | +0 | −1 | Go to diff View file |
M | plugins/timetracking/include/Timetracking/REST/v1/TimetrackingQueryChecker.php | +0 | −1 | Go to diff View file |
M | plugins/timetracking/include/Timetracking/REST/v1/TimetrackingReportResource.php | +0 | −1 | Go to diff View file |
M | plugins/tracker/include/REST/Artifact/ArtifactCreator.php | +0 | −1 | Go to diff View file |
M | plugins/tracker/include/REST/Artifact/ArtifactRepresentationBuilder.php | +0 | −1 | Go to diff View file |
M | plugins/tracker/include/REST/v1/ArtifactsResource.class.php | +0 | −3 | Go to diff View file |
M | plugins/tracker/include/REST/v1/GetTrackersQueryChecker.php | +0 | −1 | Go to diff View file |
M | plugins/tracker/include/REST/v1/TrackersResource.class.php | +0 | −2 | Go to diff View file |
M | plugins/tracker/include/Tracker/Admin/GlobalAdmin/ArtifactLinks/ArtifactLinksController.php | +0 | −3 | Go to diff View file |
M | plugins/tracker/include/Tracker/Artifact/InvertDisplayChangesController.php | +0 | −1 | Go to diff View file |
M | plugins/tracker/include/Tracker/Artifact/Tracker_ArtifactFactory.class.php | +0 | −1 | Go to diff View file |
M | plugins/tracker/include/Tracker/Artifact/XMLImport.class.php | +0 | −1 | Go to diff View file |
M | plugins/tracker/include/Tracker/Artifact/dao/Tracker_ArtifactDao.class.php | +0 | −1 | Go to diff View file |
M | plugins/tracker/include/Tracker/Creation/TrackerCreationController.php | +0 | −1 | Go to diff View file |
M | plugins/tracker/include/Tracker/FormElement/ArtifactLinkValidator.php | +0 | −3 | Go to diff View file |
M | plugins/tracker/include/Tracker/FormElement/Event/MessageFetcherAdditionalWarnings.php | +0 | −3 | Go to diff View file |
M | plugins/tracker/include/Tracker/FormElement/Field/ArtifactLink/ArtifactLinkValueSaver.php | +1 | −1 | Go to diff View file |
M | plugins/tracker/include/Tracker/FormElement/Field/ArtifactLink/SubmittedValueConvertor.php | +0 | −1 | Go to diff View file |
M | plugins/tracker/include/Tracker/FormElement/PermissionsOnArtifactUGroupRetriever.php | +0 | −1 | Go to diff View file |
M | plugins/tracker/include/Tracker/FormElement/PermissionsOnArtifactValidator.php | +0 | −2 | Go to diff View file |
M | plugins/tracker/include/Tracker/FormElement/Tracker_FormElement.php | +0 | −1 | Go to diff View file |
M | plugins/tracker/include/Tracker/FormElement/Tracker_FormElement_Field.php | +0 | −1 | Go to diff View file |
M | plugins/tracker/include/Tracker/FormElement/Tracker_FormElement_Field_ArtifactLink.php | +0 | −2 | Go to diff View file |
M | plugins/tracker/include/Tracker/FormElement/Tracker_FormElement_Field_Date.php | +0 | −1 | Go to diff View file |
M | plugins/tracker/include/Tracker/FormElement/Tracker_FormElement_Field_List.php | +0 | −4 | Go to diff View file |
M | plugins/tracker/include/Tracker/FormElement/Tracker_FormElement_Field_PermissionsOnArtifact.php | +0 | −2 | Go to diff View file |
M | plugins/tracker/include/Tracker/FormElement/Tracker_FormElement_Field_SubmittedBy.php | +0 | −1 | Go to diff View file |
M | plugins/tracker/include/Tracker/Hierarchy/HierarchyFactory.class.php | +0 | −1 | Go to diff View file |
M | plugins/tracker/include/Tracker/Permission/Fields/ByField/ByFieldController.php | +0 | −1 | Go to diff View file |
M | plugins/tracker/include/Tracker/Permission/Fields/ByGroup/ByGroupController.php | +0 | −1 | Go to diff View file |
M | plugins/tracker/include/Tracker/Permission/Fields/PermissionsOnFieldsUpdateController.php | +0 | −1 | Go to diff View file |
M | plugins/tracker/include/Tracker/Report/Event/TrackerReportProcessAdditionalQuery.php | +0 | −9 | Go to diff View file |
M | plugins/tracker/include/Tracker/Report/Renderer/Table/Sort/SortWithIntegrityChecked.php | +0 | −2 | Go to diff View file |
M | plugins/tracker/include/Tracker/Report/TrackerReportExtractor.php | +0 | −1 | Go to diff View file |
M | plugins/tracker/include/Tracker/Rule/Tracker_RuleFactory.class.php | +0 | −1 | Go to diff View file |
M | plugins/tracker/include/Tracker/Webhook/ArtifactPayload.php | +0 | −3 | Go to diff View file |
M | plugins/tracker/include/TrackerXmlImport.class.php | +0 | −4 | Go to diff View file |
M | plugins/tracker/include/Workflow/Event/GetWorkflowExternalPostActionsValuesForUpdate.php | +0 | −3 | Go to diff View file |
M | plugins/tracker/include/Workflow/PostAction/Update/FrozenFieldsValue.php | +0 | −3 | Go to diff View file |
M | plugins/tracker/include/Workflow/PostAction/Update/HiddenFieldsetsValue.php | +0 | −3 | Go to diff View file |
M | plugins/tracker/include/Workflow/Trigger/RulesManager.class.php | +0 | −2 | Go to diff View file |
M | plugins/tracker/tests/unit/Tracker/Creation/JiraImporter/PendingJiraImportCleanerTest.php | +0 | −3 | Go to diff View file |
M | plugins/tracker/tests/unit/Tracker/Permission/PermissionSerializerTest.php | +0 | −1 | Go to diff View file |
M | plugins/webdav/include/FS/WebDAVFRSPackage.class.php | +0 | −1 | Go to diff View file |
M | src/common/DB/Compat/Legacy2018/CompatPDODataAccess.php | +0 | −1 | Go to diff View file |
M | src/common/DB/Compat/Legacy2018/LegacyDataAccessInterface.php | +0 | −1 | Go to diff View file |
M | src/common/Dashboard/User/UserDashboardController.php | +0 | −1 | Go to diff View file |
M | src/common/Dashboard/Widget/WidgetDashboardController.php | +0 | −4 | Go to diff View file |
M | src/common/FRS/LicenseAgreement/Admin/EditLicenseAgreementController.php | +0 | −1 | Go to diff View file |
M | src/common/FRS/LicenseAgreement/Admin/ListLicenseAgreementsController.php | +0 | −1 | Go to diff View file |
M | src/common/FRS/PermissionsPerGroup/FrsGlobalAdminPermissionCollection.php | +0 | −3 | Go to diff View file |
M | src/common/Instrument/Prometheus/Prometheus.php | +0 | −7 | Go to diff View file |
M | src/common/Jenkins/Client.class.php | +0 | −1 | Go to diff View file |
M | src/common/Layout/BaseLayout.php | +0 | −3 | Go to diff View file |
M | src/common/Layout/SiteHomepageController.php | +0 | −1 | Go to diff View file |
M | src/common/Mail/MailNotificationBuilder.php | +0 | −2 | Go to diff View file |
M | src/common/News/PermissionsPerGroup.php | +0 | −1 | Go to diff View file |
M | src/common/Plugin/PluginLegacyController.php | +0 | −1 | Go to diff View file |
M | src/common/Project/Admin/PermissionPerGroup/PermissionPerGroupCollection.php | +0 | −3 | Go to diff View file |
M | src/common/Project/Admin/ProjectUGroup/MemberRemovalController.php | +0 | −1 | Go to diff View file |
M | src/common/Project/Home.php | +0 | −1 | Go to diff View file |
M | src/common/Project/ProjectCreationData.php | +0 | −4 | Go to diff View file |
M | src/common/Project/ProjectManager.class.php | +0 | −1 | Go to diff View file |
M | src/common/Project/REST/v1/GetProjectsQueryChecker.php | +0 | −2 | Go to diff View file |
M | src/common/Request/DispatchTemporaryRedirect.php | +0 | −1 | Go to diff View file |
M | src/common/Request/DispatchableWithProject.php | +0 | −1 | Go to diff View file |
M | src/common/Request/DispatchableWithRequest.php | +0 | −1 | Go to diff View file |
M | src/common/Request/FrontRouter.php | +0 | −1 | Go to diff View file |
M | src/common/Request/LegacyRoutesController.php | +0 | −1 | Go to diff View file |
M | src/common/Search/Presenters/SearchPanePresenter.class.php | +0 | −1 | Go to diff View file |
M | src/common/TreeNode/TreeNode.class.php | +0 | −1 | Go to diff View file |
M | src/common/User/User_SSHKeyValidator.class.php | +0 | −1 | Go to diff View file |
M | src/common/admin/PendingElements/PendingDocumentsRetriever.php | +0 | −3 | Go to diff View file |
M | src/common/admin/ProjectCreation/ProjectCategoriesDisplayController.php | +0 | −1 | Go to diff View file |
M | src/common/admin/ProjectCreation/ProjectFields/ProjectFieldsDisplayController.php | +0 | −1 | Go to diff View file |
M | src/common/admin/ProjectCreation/ProjectFields/ProjectFieldsUpdateController.php | +0 | −1 | Go to diff View file |
M | src/common/admin/ProjectCreation/WebhooksDisplayController.php | +0 | −1 | Go to diff View file |
M | src/common/admin/ProjectCreation/WebhooksUpdateController.php | +0 | −1 | Go to diff View file |
M | src/common/admin/ProjectCreationModerationDisplayController.php | +0 | −1 | Go to diff View file |
M | src/common/admin/ProjectCreationModerationUpdateController.php | +0 | −1 | Go to diff View file |
M | src/common/admin/ProjectTemplatesController.php | +0 | −1 | Go to diff View file |
M | src/common/dao/UserDao.class.php | +0 | −1 | Go to diff View file |
M | src/common/permission/PermissionsManager.class.php | +0 | −6 | Go to diff View file |
M | src/common/tracker/ArtifactStaticMultiListFieldXMLExporter.class.php | +0 | −2 | Go to diff View file |
M | src/common/trove/TroveCatListController.php | +0 | −1 | Go to diff View file |
M | src/common/wiki/phpwiki/lib/BlockParser.php | +1 | −1 | Go to diff View file |
M | src/common/wiki/phpwiki/lib/HtmlElement.php | +1 | −1 | Go to diff View file |
M | src/common/wiki/phpwiki/lib/IniConfig.php | +1 | −1 | Go to diff View file |
M | src/common/wiki/phpwiki/lib/TextSearchQuery.php | +2 | −2 | Go to diff View file |
M | src/common/wiki/phpwiki/lib/WikiDB/backend/PearDB.php | +1 | −1 | Go to diff View file |
M | src/common/wiki/phpwiki/lib/WikiUserNew.php | +2 | −2 | Go to diff View file |
M | src/common/wiki/phpwiki/lib/XmlElement.php | +1 | −1 | Go to diff View file |
M | src/common/wiki/phpwiki/lib/XmlParserPHPWiki.php | +1 | −1 | Go to diff View file |
M | src/common/wiki/phpwiki/lib/config.php | +1 | −1 | Go to diff View file |
M | src/common/wiki/phpwiki/lib/plugin/AddComment.php | +1 | −1 | Go to diff View file |
M | src/common/wiki/phpwiki/lib/plugin/BlogArchives.php | +1 | −1 | Go to diff View file |
M | src/common/wiki/phpwiki/lib/plugin/BlogJournal.php | +1 | −1 | Go to diff View file |
M | src/common/wiki/phpwiki/lib/plugin/CreateBib.php | +1 | −1 | Go to diff View file |
M | src/common/wiki/phpwiki/lib/plugin/CreatePage.php | +1 | −1 | Go to diff View file |
M | src/common/wiki/phpwiki/lib/plugin/CreateToc.php | +1 | −1 | Go to diff View file |
M | src/common/wiki/phpwiki/lib/plugin/FileInfo.php | +1 | −1 | Go to diff View file |
M | src/common/wiki/phpwiki/lib/plugin/GoTo.php | +1 | −1 | Go to diff View file |
M | src/common/wiki/phpwiki/lib/plugin/JabberPresence.php | +1 | −1 | Go to diff View file |
M | src/common/wiki/phpwiki/lib/plugin/ListPages.php | +1 | −1 | Go to diff View file |
M | src/common/wiki/phpwiki/lib/plugin/PhotoAlbum.php | +1 | −1 | Go to diff View file |
M | src/common/wiki/phpwiki/lib/plugin/RdfDefinition.php | +1 | −1 | Go to diff View file |
M | src/common/wiki/phpwiki/lib/plugin/RecentComments.php | +1 | −1 | Go to diff View file |
M | src/common/wiki/phpwiki/lib/plugin/RssFeed.php | +1 | −1 | Go to diff View file |
M | src/common/wiki/phpwiki/lib/plugin/SyntaxHighlighter.php | +1 | −1 | Go to diff View file |
M | src/common/wiki/phpwiki/lib/plugin/TranslateText.php | +1 | −1 | Go to diff View file |
M | src/common/wiki/phpwiki/lib/plugin/WikiAdminChown.php | +1 | −1 | Go to diff View file |
M | src/common/wiki/phpwiki/lib/plugin/WikiAdminMarkup.php | +1 | −1 | Go to diff View file |
M | src/common/wiki/phpwiki/lib/plugin/WikiAdminRename.php | +1 | −1 | Go to diff View file |
M | src/common/wiki/phpwiki/lib/plugin/WikiBlog.php | +1 | −1 | Go to diff View file |
M | src/common/wiki/phpwiki/lib/plugin/WikiForum.php | +1 | −1 | Go to diff View file |
M | src/common/wiki/phpwiki/lib/plugin/_GroupInfo.php | +1 | −1 | Go to diff View file |
M | src/common/wiki/phpwiki/lib/plugin/_WikiTranslation.php | +1 | −1 | Go to diff View file |
M | src/common/wiki/phpwiki/lib/upgrade.php | +1 | −1 | Go to diff View file |
M | src/composer.lock | +39 | −34 | Go to diff View file |
M | src/core/RSS/News/LatestNewsController.php | +0 | −1 | Go to diff View file |
M | src/core/RSS/Project/LatestProjectController.php | +0 | −1 | Go to diff View file |
M | tests/rest/tests/ProjectServicesTest.php | +0 | −3 | Go to diff View file |