Fix code style issues in EntityStatusTest and StemmerTest

Created on 7 November 2022, over 1 year ago
Updated 22 January 2023, over 1 year ago

Problem/Motivation

All method arguments with default values should be declared after the method arguments without default values. Otherwise, it makes it impossible for callers to take advantage of defaults; they must re-specify the defaulted values in order to "get to" the non-default arguments.

File: search_api/tests/src/Unit/Processor/EntityStatusTest.php

83: public function testSupportsIndex(array $datasource_ids = NULL, $expected) {

Proposed resolution

83: public function testSupportsIndex($expected, array $datasource_ids = NULL) {
πŸ› Bug report
Status

Fixed

Version

1.0

Component

Tests

Created by

πŸ‡©πŸ‡ͺGermany fnalb2

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.69.0 2024