[#19239] - Improve code to build pathway for ContentViewCategory
- Closed
- 3 Oct 2018
- Medium
- Build: staging
- # 19239
- Diff
- joomdonation:patch-15
User tests: Successful: Unsuccessful:
Pull Request for Issue # .
Summary of Changes
Our current code to build pathway to category looks quite complicated and not easy to understand (and by reading the code, I think it can be wrong in some case - for example, id is passed in active menu item but that menu item is not linked to categories or category view of com_content)
This PR improves that code, make it easier to understand and more accurate. The logic is that we add the path from current to category to the category which is setup in active menu item (or to root category if no category found from active menu item)
Testing Instructions
-
Prefer to have one developer to review code
-
For human testing, apply patch, check and see that pathway is still generated correctly as before (it usually has the format Home -> Menu item -> [Category] -> [Sub-Category] -> Article
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Front End com_content |
| Labels |
Added:
?
|
||
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2018-10-03 01:31:21 |
| Closed_By | ⇒ | joomdonation |