use older php array syntax in index.php#1661
Conversation
|
@georgehrke, thanks for your PR! By analyzing the history of the files in this pull request, we identified @DeepDiver1975, @owncloud-bot and @LukasReschke to be potential reviewers. |
|
Why? we don't support it. |
|
Yes, we don't support it. |
|
I don't really care. Either way is fine for me. We could also add the exact parse error in our docs, so it has a hint on google... |
|
mmm fine by me. But it will fail at some point in the future when somebody modifies base.php again. |
I don't think so. As long as this check kicks on, base.php won't even be included. Line 30 in a48aed2 My point is, that the user won't even see: when running on PHP5.3 or earlier, because php isn't even able to parse the index.php |
|
LGTM |
We should use the old array syntax in the index.php, because the shorthand version was introduced with PHP 5.4.
Users with an older PHP will only see an error message about the Syntax error in the index.php
fixes #1659
please review @MorrisJobke @nickvergessen @fancycode