Closed (fixed)
Project:
Quiz
Version:
7.x-5.x-dev
Component:
Code - Multichoice
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
31 Aug 2018 at 17:19 UTC
Updated:
12 Dec 2018 at 06:34 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
djdevinPlease upgrade to PHP 5.4+
Sorry this is not a listed requirement, I will add it.
Comment #3
vensiresWe may be in the D8 era, but according to D7's PHP requirements, the short-syntax of the arrays should not be enforced. I believe it should be set back to
array()Comment #4
djdevinDoesn't that page say that PHP 5.4 is the recommended minimum?
I noticed we are missing an info line for the requirement, but we can add it.
Comment #5
vensiresRecommended yes. Not unsupported. I wouldn't like to be the one to say no to upgrades, but the array() syntax isn't deprecated in PHP 7.0+ and that's why I stand against the [] syntax in D7.
Comment #6
joelpittetWe are upgrading to 7 but it's a slow going process at educational institutions... pity us!
Comment #7
vensiresComment #8
djdevinComment #10
djdevinComment #11
djdevinSince Quiz did not advertise that it needed 5.4 I have committed this.
However, in the future there may be a release that will be marked for a minimum version of PHP.
Comment #12
joelpittetThank you for pitying us!
Comment #13
steinmb commentedThere might be more modules that drop older PHP 5 versions after core move to minimum 5.6.x (https://dri.es/the-end-of-php-5)
Comment #14
vensiresChanged the title of the issue for better understanding of why this is not a blocker for PHP 7.2 support.
@steinmb Dropping older PHP 5 version support doesn't mean we should always use [] instead of array() since it is not even declared as deprecated yet.