-
VladimirAus β
committed f54041d1 on 7.x-1.x authored by
poker10 β
Issue #3254703 by andregp, poker10, frank.schram, joelpittet,...
-
VladimirAus β
committed f54041d1 on 7.x-1.x authored by
poker10 β
- Status changed to Fixed
over 1 year ago 6:09am 27 February 2023 - π¦πΊAustralia VladimirAus Brisbane, Australia
Thanks all! Committed! π»
- Status changed to RTBC
over 1 year ago 4:43pm 28 February 2023 - π¬π§United Kingdom MustangGB Coventry, United Kingdom
Re-opening this to ask if the commit intentionally changed arrays to use the short syntax.
Drupal 7 supports PHP 5.2.5 and short array syntax wasn't added until PHP 5.4, so as a general rule no-one else is doing this in D7.
- Status changed to Fixed
over 1 year ago 9:35am 2 March 2023 - π¦πΊAustralia VladimirAus Brisbane, Australia
Yes, it was intentional.
I'm happy to support Drupal 7 version this year, but I'm not supporting outdated and not secure versions of php. - π¨π¦Canada joelpittet Vancouver
Thanks @VladimirAus for getting this in.
To add to @MustangGB's point usually we are explicit about our support (if we use short array syntax) by adding a php directive in the info file.
https://www.drupal.org/docs/7/creating-custom-modules/writing-module-inf... β
Then likely @MustangGB wouldn't even needed to ask the question.
Hot tip, take it or leave it :) If it were composer then that new release wouldn't even update if the platform/PHP version requirements in the composer.json weren't met, which would likely be fine for the people who are happy to live in the past.
Automatically closed - issue fixed for 2 weeks with no activity.