stable
Clone or download
Read-only
request #14529: Do not depend on concrete logger implementations
To respect the dependency inversion principle so we easily swap implementations of those loggers in the future, the code now depends on the abstraction everywhere it was possible to do the switch. The loggers in the tracker workflow area are kept as is because they define methods outside of the PSR-3 LoggerInterface, the implementation cannot be swapped for something else. Change-Id: I56224bd3736d7be55dd6ee1a616c6e7a915a03db
Modified Files
| Name | ||||
|---|---|---|---|---|
| M | plugins/agiledashboard/include/AgileDashboard/FormElement/Burnup.php | +2 | −4 | Go to diff View file |
| M | plugins/agiledashboard/include/AgileDashboard/FormElement/SystemEvent/SystemEvent_BURNUP_DAILY.php | +3 | −3 | Go to diff View file |
| M | plugins/agiledashboard/include/AgileDashboard/FormElement/SystemEvent/SystemEvent_BURNUP_GENERATE.php | +3 | −3 | Go to diff View file |
| M | plugins/agiledashboard/include/AgileDashboard/Kanban/RealTime/KanbanArtifactMessageSender.php | +4 | −4 | Go to diff View file |
| M | plugins/agiledashboard/include/agiledashboardPlugin.php | +1 | −4 | Go to diff View file |
| M | plugins/agiledashboard/phpunit/AgileDashboard/ExplicitBacklog/XMLImporterTest.php | +2 | −2 | Go to diff View file |
| M | plugins/archivedeleteditems/include/FileCopier.php | +5 | −3 | Go to diff View file |
| M | plugins/archivedeleteditems/include/archivedeleteditemsPlugin.php | +1 | −1 | Go to diff View file |
| M | plugins/archivedeleteditems/phpunit/FileCopierTest.php | +3 | −2 | Go to diff View file |
| M | plugins/bugzilla_reference/include/Bugzilla/Reference/RESTReferenceCreator.php | +3 | −3 | Go to diff View file |
| M | plugins/bugzilla_reference/phpunit/Bugzilla/RESTReferenceCreatorTest.php | +2 | −1 | Go to diff View file |
| M | plugins/docman/phpunit/SystemCheckTest.php | +2 | −1 | Go to diff View file |
| M | plugins/git/phpunit/Git/Gitolite/Gitolite3LogParserTest.php | +2 | −2 | Go to diff View file |
| M | plugins/ldap/include/LDAP_CleanUpManager.class.php | +1 | −3 | Go to diff View file |
| M | plugins/ldap/include/ldapPlugin.php | +1 | −4 | Go to diff View file |
| M | plugins/mediawiki/include/MediawikiInstantiater.class.php | +1 | −1 | Go to diff View file |
| M | plugins/mediawiki/phpunit/XMLMediaWikiExporterTest.php | +2 | −2 | Go to diff View file |
| M | plugins/pullrequest/include/PullRequest/REST/v1/PullRequestsResource.php | +2 | −1 | Go to diff View file |
| M | plugins/tracker/include/REST/v1/Workflow/TransitionsResource.class.php | +2 | −4 | Go to diff View file |
| M | plugins/tracker/include/Tracker/Artifact/Changeset/PostCreation/EmailNotificationTask.php | +2 | −2 | Go to diff View file |
| M | plugins/tracker/include/Tracker/FormElement/SystemEvent/SystemEvent_BURNDOWN_DAILY.php | +4 | −4 | Go to diff View file |
| M | plugins/tracker/include/Tracker/FormElement/SystemEvent/SystemEvent_BURNDOWN_GENERATE.php | +3 | −3 | Go to diff View file |
| M | plugins/tracker/include/Tracker/Migration/MigrationLogger.php | +5 | −3 | Go to diff View file |
| M | plugins/tracker/include/Tracker/XML/Importer/CopyArtifactInformationsAggregator.php | +4 | −3 | Go to diff View file |
| M | plugins/tracker/include/Workflow/WorkflowRulesManagerLoopSafeGuard.php | +3 | −2 | Go to diff View file |
| M | plugins/tracker/phpunit/Tracker/Action/CopyArtifactTest.php | +1 | −1 | Go to diff View file |
| M | plugins/tracker/phpunit/Tracker/Artifact/Changeset/PostCreation/EmailNotificationTaskTest.php | +3 | −3 | Go to diff View file |
| M | plugins/tracker/phpunit/Tracker/Artifact/Changeset/PostCreation/NotifierCustomSenderTest.php | +2 | −2 | Go to diff View file |
| M | plugins/tracker/phpunit/Tracker/Migration/MigrationLoggerTest.php | +5 | −5 | Go to diff View file |
| M | plugins/tracker/phpunit/Tracker/Semantic/Timeframe/TimeframeBuilderTest.php | +3 | −2 | Go to diff View file |
| M | plugins/tracker/phpunit/Tracker/Workflow/WorkflowBackendLoggerTest.php | +2 | −2 | Go to diff View file |
| M | plugins/tracker/tests/Tracker/XML/Importer/CopyArtifactInformationsAggregatorTest.php | +3 | −3 | Go to diff View file |
| M | src/common/Project/Admin/DescriptionFields/FieldUpdator.php | +3 | −2 | Go to diff View file |
| M | src/common/User/UserSuspensionManager.php | +4 | −4 | Go to diff View file |
| M | src/common/dao/UserSuspensionDao.php | +3 | −3 | Go to diff View file |
| M | src/common/mail/Codendi_Mail.class.php | +1 | −1 | Go to diff View file |
| M | src/common/plugin/Plugin.class.php | +3 | −5 | Go to diff View file |
| M | tests/phpunit/common/Project/Admin/DescriptionFields/FieldUpdatorTest.php | +3 | −2 | Go to diff View file |
| M | tests/phpunit/common/Project/ProjectXMLExporterTest.php | +1 | −1 | Go to diff View file |
| M | tests/phpunit/common/Project/ProjectXMLExporterWithSynchronizedUGroupsTest.php | +2 | −2 | Go to diff View file |
| M | tests/phpunit/common/Project/ProjectXMLImporterTest.php | +1 | −1 | Go to diff View file |
| M | tests/phpunit/common/User/UserSuspensionManagerTest.php | +2 | −1 | Go to diff View file |
| M | tests/phpunit/common/mail/MailFilterTest.php | +3 | −2 | Go to diff View file |
| M | tools/Configuration/Setup/DistributedSVN.php | +3 | −2 | Go to diff View file |