Skip to content

Duplicated code#38669

Merged
wilsonge merged 1 commit intojoomla:4.2-devfrom
brianteeman:menu_item
Sep 6, 2022
Merged

Duplicated code#38669
wilsonge merged 1 commit intojoomla:4.2-devfrom
brianteeman:menu_item

Conversation

@brianteeman
Copy link
Copy Markdown
Contributor

This PR removes a block of duplicated code. They two blocks are slightly different as the home is in a different place. But as the first block is overridden by the second block I have deleted the first block.

There is no visible changes as a result of this PR. Either test by code review or by checking that you can still create and edit menu items exactly the same as before this PR

(any comments about codestyle will be ignored as out of scope for this pr)

This PR removes a block of duplicated code. They two blocks are slightly different as the home is in a different place. But as the first block is overridden by the second block I have deleted the first block.

There is no visible changes as a result of this PR. Either test by code review or by checking that you can still create and edit menu items exactly the same as before this PR
@sandewt
Copy link
Copy Markdown
Contributor

sandewt commented Sep 2, 2022

I have tested this item ✅ successfully on 4a48516

@toivo
Copy link
Copy Markdown
Contributor

toivo commented Sep 5, 2022

I have tested this item ✅ successfully on 4a48516

Tested successfully in Joomla 4.2.3-dev of 5 September in Wampserver 3.2.9 and PHP 8.1.9.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/38669.

@alikon
Copy link
Copy Markdown
Contributor

alikon commented Sep 5, 2022

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/38669.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Sep 5, 2022
@Quy Quy added the PR-4.2-dev label Sep 5, 2022
@wilsonge wilsonge merged commit c2330b3 into joomla:4.2-dev Sep 6, 2022
@wilsonge
Copy link
Copy Markdown
Contributor

wilsonge commented Sep 6, 2022

Thanks!

@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Sep 6, 2022
@wilsonge wilsonge added this to the Joomla! 4.2.3 milestone Sep 6, 2022
@brianteeman
Copy link
Copy Markdown
Contributor Author

thanks

@brianteeman brianteeman deleted the menu_item branch September 6, 2022 10:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants