Feeling Lucky
?
[#33981] - [4] s/setMvcFactory/setMVCFactory
- Fixed in Code Base
- 19 May 2021
- Medium
- Build: 4.0-dev
- # 33981
- Diff
- PhilETaylor:patch-162
User tests: Successful: Unsuccessful:
Change case of method. Doesn't change the way its called, but is "more correct' that the 33+ instances that call this method, call it with the correct case
Note that getMVCFactory has MVC in all caps, and so rather than change33+ cases to be setMvcFactory it seems the right code change is to change as this PR is.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Libraries |
| Title |
|
||||||
| Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-05-19 21:21:35 |
| Closed_By | ⇒ | wilsonge | |
| Labels |
Added:
?
|
||
wilsonge
- comment
- 19 May 2021
Plus I agree with the fix - also consistent with the namespace declaration
Thanks!