PR-4.4-dev Pending

User tests: Successful: Unsuccessful:

avatar HLeithner
HLeithner
4 May 2023

Summary of Changes

Deprecate HtmlDocument::countMenuChildren() method and rewrite the method to use the menu already loaded menu to count childrens.

This function is not used in the CMS and is conceptional wrong in the HtmlDocument, if it needs to exists it should be part of the menu class. But even this makes not much sense because you simple need to count() the children auf the menu item.

Testing Instructions

Use the countMenuChildren method in the template with $this->countMenuChildren()

Actual result BEFORE applying this Pull Request

Should work.

Expected result AFTER applying this Pull Request

Should be the same value as before.

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: joomla/Manual#117

  • No documentation changes for manual.joomla.org needed

avatar joomla-cms-bot joomla-cms-bot - change - 4 May 2023
Category Libraries
avatar HLeithner HLeithner - change - 4 May 2023
Status New Pending
avatar HLeithner HLeithner - open - 4 May 2023
avatar laoneo laoneo - change - 8 May 2023
Labels Added: PR-4.4-dev
avatar laoneo laoneo - change - 8 May 2023
The description was changed
avatar laoneo laoneo - edited - 8 May 2023
avatar laoneo laoneo - change - 12 May 2023
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2023-05-12 05:41:00
Closed_By laoneo
avatar laoneo laoneo - close - 12 May 2023
avatar laoneo laoneo - merge - 12 May 2023
avatar laoneo
laoneo - comment - 12 May 2023

Thanks!

Add a Comment

Login with GitHub to post a comment