Feeling Lucky
?
[#10880] - There's no reason to instantiate TagsHelperRoute here
- Fixed in Code Base
- 20 Jun 2016
- Medium
- Build: staging
- # 10880
- Diff
- mbabker:Why-Is-This-Being-Instantiated-Needlessly
- Success continuous-integration/travis-ci/pr The Travis CI build passed Details
User tests: Successful: Unsuccessful:
Summary of Changes
TagsHelperRoute is accessed here through its static methods. There's no need to not only instantiate it, but instantiate a new instance on every loop.
Testing Instructions
mod_tags_popular still works. Assuming it ever did to begin with.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Labels |
Added:
?
|
||
wilsonge
- | Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-06-20 20:16:33 |
| Closed_By | ⇒ | wilsonge |
| Milestone |
Added: |
||