[5.2] Improve Multilanguage::getSiteHomePages()

[Denitz]

Summary of Changes

Joomla\CMS\Language\Multilanguage::getSiteHomePages() uses an extra database query per each page load to discover the language-specific site homepages.

We can easily eliminate this extra database query and use the native SiteMenu::getItems().

Testing Instructions

Test a website with enabled multilanguage functionality and Joomla system caching enabled.

Actual result BEFORE applying this Pull Request

See an extra unoptimized SQL query per each page load:

SELECT `language`,`id` FROM `jos_menu` WHERE `home` = '1' AND `published` = 1 AND `client_id` = 0

Expected result AFTER applying this Pull Request

No extra SQL query, the homepages are discovered from the cached menu items.

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

[HLeithner]

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

[crommie]

I have tested this item ✅ successfully on 9ea35c7

Extra query has gone.

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/41507.}

[viocassel]

I have tested this item ✅ successfully on 9ea35c7

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/41507.}

[Quy]

RTC

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/41507.}

[HLeithner]

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

[rdeutz]

This was discussed at the maintainers meeting today. Please don't remove the parameter but make a depeciated message for the parameter so that we can remove it in one of the next versions. Beside this the change is approved. Thanks.

[Denitz]

@rdeutz Done. Note that I can't add @deprecated attribute because it deprecates the whole method.