Feeling Lucky
?
?
[#34802] - [3.x] Fix short array syntax
- Fixed in Code Base
- 18 Jul 2021
- Medium
- Build: staging
- # 34802
- Diff
- zero-24:zero-24-patch-1
User tests: Successful: Unsuccessful:
Summary of Changes
Make sure we dont use the short array syntax within Joomla 3.x
Testing Instructions
Make sure you can install extensions from URL in php 5.3
Actual result BEFORE applying this Pull Request
Parse error: syntax error, unexpected '[' in administrator\components\com_installer\models\install.php on line 391
Expected result AFTER applying this Pull Request
Works
Documentation Changes Required
none
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration com_installer |
richard67
- comment
- 16 Jul 2021
I have tested this item
Code review.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34802.
| Status | Pending | ⇒ | Ready to Commit |
| Labels |
Added:
?
|
||
richard67
- comment
- 16 Jul 2021
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34802.
zero-24
- comment
- 18 Jul 2021
Merging for now thanks!
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-07-18 06:38:43 |
| Closed_By | ⇒ | zero-24 | |
| Labels |
Added:
?
|
||
sakiss
- comment
- 19 Jul 2021
Btw. Is this some type of code style requirement?
alikon
- comment
- 19 Jul 2021
no j3 minimum requirement for php is 5.3 where short array syntax is illegal
I have tested this item✅ successfully on c3390b9
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34802.