[#14841] - Wrong component checked in the router
- Fixed in Code Base
- 21 Mar 2017
- Medium
- Build: staging
- # 14841
- Diff
- brianteeman:router
User tests: Successful: Unsuccessful:
New router for com_newsfeeds and com_contact need new router enabled also in com_content (see #14833)
Simple copy paste error
Shouldnt there be tests for this as well?
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Front End com_contact com_newsfeeds com_users |
Also newsfeed and contacts can be removed as they are duplicate of first line in function
Sooner or later we need to figure out the roadblock that causes us to not write unit tests for code outside the libraries and start writing the tests. Once upon a time system tests were relied for that stuff but there hasn't been a system test suite hooked into core for years. @yvesh here's an indirect task for your team
| Labels |
Added:
?
?
|
||
| Category | Front End com_contact com_newsfeeds com_users | ⇒ | Front End com_contact com_content com_newsfeeds com_users |
duplicates removed from three places
| Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-03-21 19:17:19 |
| Closed_By | ⇒ | wilsonge |
Merged on review. Now let's see if anything falls out because we never tested the URLs correctly :)
There's tests for the libraries not the component implementation