Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • fusiondirectory fusiondirectory
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 33
    • Issues 33
    • List
    • Boards
    • Service Desk
    • Milestones
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Container Registry
    • Terraform modules
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
Collapse sidebar
  • fusiondirectoryfusiondirectory
  • fusiondirectoryfusiondirectory
  • Issues
  • #5955
Closed
Open
Issue created Feb 13, 2019 by bmortier@bmortierMaintainer

Use PHP7 array syntax

PHP now allows the [] syntax for arrays instead of array().

We should make the change and use this new syntax in FD, and adapt our code review profile to detect uses of the old syntax.

Advantages:

  • less verbose for small and empty arrays
  • clear indication that we are closing an array and not a function on multilines arrays
  • symmetry with new list syntax (in PHP 7.1)
Edited Feb 13, 2019 by bmortier
Assignee
Assign to
Time tracking