search

laminas-api-tools / api-tools-configuration

Laminas module providing a REST resource for manipulating configuration
https://api-tools.getlaminas.org/documentation
BSD 3-Clause "New" or "Revised" License
4 stars 8 forks source link

Enable short array syntax by default #4

Open weierophinney opened 4 years ago

weierophinney commented 4 years ago

As we support php >= 5.6, we can enable short array syntax in configuration by default.

Originally posted by @snapshotpl at https://github.com/zfcampus/zf-configuration/pull/18

weierophinney commented 4 years ago

Except that it breaks existing applications. Not in a functional way, but in a "why is my configuration completely rewritten on this change?" way.

What will happen is users will have long array syntax in configuration files managed by Apigiliy. When they update to a version of zf-configuration that enables short array by default, and then use Apigility to manipulate their API, these files will switch to short array notation — and something that might have been a single line of changes previously now affects several dozen lines in the file.

As such, until we do a new major version of zf-configuration, we cannot enable short array syntax by default.

We can, however, enable it by default in the skeleton in a new version, and we're planning that for the 1.4 release.

Originally posted by @weierophinney at https://github.com/zfcampus/zf-configuration/pull/18#issuecomment-239625225