[#44739] - [5.2] Fix missing brackets
- Fixed in Code Base
- 17 Jan 2025
- Medium
- Build: 5.2-dev
- # 44739
- Diff
- janschoenherr:patch-22
User tests: Successful: Unsuccessful:
Fixed missing brackets
Testing Instructions
Nothing
Actual result BEFORE applying this Pull Request
Probably broken version string
Expected result AFTER applying this Pull Request
Correct version string
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
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Libraries |
I also tested it, the patch applied the missing brackets.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44739.
I have tested this item ✅ successfully on 66ac320
I also tested it, the patch applied the missing brackets.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44739.
I have tested this item ✅ successfully on 66ac320
tested, it works.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44739.
| Status | Pending | ⇒ | Ready to Commit |
| Labels |
Added:
bug
PR-5.2-dev
|
||
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44739.
| Labels |
Added:
RTC
|
||
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2025-01-17 20:00:55 |
| Closed_By | ⇒ | Hackwar |
Thank you!
@janschoenherr What about line 835?
@janschoenherr What about line 835?
Thanks, I did another PR: #44742
I have tested this item ✅ successfully on 66ac320
appears the patch has applied the missing brackets.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44739.