Skip to content

Commit

Permalink
Remove deprecated method Statement::mergeExpressions
Browse files Browse the repository at this point in the history
  • Loading branch information
natanfelles committed May 16, 2022
1 parent 56f78ac commit 7829b99
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 27 deletions.
22 changes: 0 additions & 22 deletions src/Manipulation/Statement.php
Original file line number Diff line number Diff line change
Expand Up @@ -176,26 +176,4 @@ protected function renderAssignment(
return $this->database->protectIdentifier($identifier)
. ' = ' . $this->renderValue($expression);
}

/**
* Used when a function requires at least one expression (identifier or value).
*
* @param mixed $expression
* @param array|mixed[] $expressions
*
* @deprecated Unpack arrays with the spread operator
* @codeCoverageIgnore
*
* @return array|mixed[]
*/
protected function mergeExpressions(mixed $expression, array $expressions) : array
{
\trigger_error(
'Method ' . __METHOD__ . ' is deprecated',
\E_USER_DEPRECATED
);
return $expressions
? \array_merge([$expression], $expressions)
: [$expression];
}
}
5 changes: 0 additions & 5 deletions tests/Manipulation/StatementMock.php
Original file line number Diff line number Diff line change
Expand Up @@ -50,11 +50,6 @@ public function renderAssignment(string $identifier, $expression) : string
return parent::renderAssignment($identifier, $expression);
}

public function mergeExpressions($expression, array $expressions) : array
{
return parent::mergeExpressions($expression, $expressions);
}

public function renderOptions() : ?string
{
if ( ! $this->hasOptions()) {
Expand Down

0 comments on commit 7829b99

Please sign in to comment.