Skip to content
Snippets Groups Projects

Issue #3458945: Add symfony/polyfill-php84 and make use of new array functions

Open Issue #3458945: Add symfony/polyfill-php84 and make use of new array functions

Closes #3458945

Merge request reports

Members who can merge are allowed to add commits.
Code Quality is loading
Test summary results are being parsed
Ready to merge by members who can write to the target branch.
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
136 123 if (!is_array($array)) {
137 124 return FALSE;
138 125 }
139 $i = 0;
140
141 foreach (array_keys($array) as $key) {
142 if ($i !== $key) {
143 return FALSE;
144 }
145 $i++;
146 }
147 return TRUE;
126 return array_is_list($array);
  • 199 foreach ($args as $key) {
    200 if (!array_key_exists($key, $member)) {
    201 return FALSE;
    202 }
    203 }
    204 }
    205 return TRUE;
    206 }
    207 return FALSE;
    176 return static::assertAll(
    177 fn ($member) => array_all(
    178 $args,
    179 fn ($key) => array_key_exists($key, $member),
    180 ),
    181 $traversable,
    182 );
  • Have not yet attempted to refactor assertAllMatch, assertAllRegularExpressionMatch or assertAllObjects.

  • Michael Strelan added 3 commits

    added 3 commits

    Compare with previous version

  • Michael Strelan added 131 commits

    added 131 commits

    Compare with previous version

  • added 1 commit

    • c5a9542d - Add dependency to Assertion composer

    Compare with previous version

  • Michael Strelan added 15 commits

    added 15 commits

    Compare with previous version

  • Andrey Postnikov added 500 commits

    added 500 commits

    Compare with previous version

  • added 1 commit

    • 23d7c657 - polyfills are top-level functions

    Compare with previous version

  • added 1 commit

    • 65d8debb - use fresh autoloader calling drush CR

    Compare with previous version

  • 534 534 - vendor/bin/drush config:inspect --statistics > HEAD.json
    535 535 # Return to the MR commit being tested, conditionally install updates, always rebuild the container.
    536 536 - git checkout -f $CI_COMMIT_SHA
    537 # Composer-install Drush & the Config Inspector module again to prevent autoloading issues.
    538 - composer require drush/drush drupal/config_inspector || exit 100
    Please register or sign in to reply
    Loading