Skip to content
Snippets Groups Projects

Issue #2924782: Missing @var annotation for arrays that provide default values

Closed spokje requested to merge issue/drupal-2924782:2924782-missing-var-annotation into 9.4.x

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • quietone
  • spokje added 294 commits

    added 294 commits

    Compare with previous version

  • spokje added 1 commit

    added 1 commit

    Compare with previous version

  • spokje added 1 commit

    added 1 commit

    • 6aba761d - Resolving threads opened by quietone

    Compare with previous version

  • spokje changed target branch from 9.3.x to 9.4.x

    changed target branch from 9.3.x to 9.4.x

  • spokje added 452 commits

    added 452 commits

    Compare with previous version

  • spokje added 2 commits

    added 2 commits

    • 567f483a - 1 commit from branch project:9.4.x
    • c0070839 - Merge branch '9.4.x' into 2924782-missing-var-annotation

    Compare with previous version

  • spokje added 12 commits

    added 12 commits

    Compare with previous version

  • quietone
  • quietone
    quietone @quietone started a thread on the diff
  • 65 65 * transliterates to more than one ASCII character require special
    66 66 * treatment: we want to remove their accent and use the un-
    67 67 * transliterated base character.
    68 *
    69 * @var string[]
    • The array it two dimensional, shouldn't this be string[][]? Or do I have that wrong?

    • AFAIK: string[][] is an array like this:

      ['string1' => ['string2' => 'string3']]

      Grepping Core for string[][] seem to back this idea up, however I'm not a 100% sure.

    • Please register or sign in to reply
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading