Feature Maintainers Checked PR-5.1-dev Pending

User tests: Successful: 0 Unsuccessful: 0

avatar Denitz
Denitz
28 Sep 2023

Summary of Changes

BaseApplication, CliApplication and ConsoleApplication should not implement DispatcherAwareInterface and use DispatcherAwareTrait because they extend AbstractApplication which already implements and uses this interface and trait.

Testing Instructions

Apply patch.

Actual result BEFORE applying this Pull Request

Duplicate interface inheritance and trait usage.

Expected result AFTER applying this Pull Request

Simpler code.

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

86a67a0 28 Sep 2023 avatar Denitz fix
avatar joomla-cms-bot joomla-cms-bot - change - 28 Sep 2023
Category Libraries
avatar Denitz Denitz - open - 28 Sep 2023
avatar Denitz Denitz - change - 28 Sep 2023
Status New Pending
avatar HLeithner HLeithner - change - 30 Sep 2023
Title
[5.0] Duplicate DispatcherAwareInterface implementation
[5.1] Duplicate DispatcherAwareInterface implementation
avatar HLeithner HLeithner - edited - 30 Sep 2023
avatar HLeithner
HLeithner - comment - 30 Sep 2023

This pull request has been automatically rebased to 5.1-dev.

avatar rdeutz
rdeutz - comment - 11 Jan 2024

LGTM, code review

avatar LadySolveig LadySolveig - change - 17 Jan 2024
Labels Added: Feature Maintainers Checked PR-5.1-dev
avatar bembelimen bembelimen - change - 2 Mar 2024
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2024-03-02 22:54:34
Closed_By bembelimen
avatar bembelimen bembelimen - close - 2 Mar 2024
avatar bembelimen bembelimen - merge - 2 Mar 2024
avatar bembelimen
bembelimen - comment - 2 Mar 2024

Thx

Add a Comment

Login with GitHub to post a comment