Skip to content

Commit dadcec8

Browse files
Ocramiusmorozov
authored andcommitted
Merge pull request #3498 from morozov/get-other-expr
Reworked AbstractPlatform::get*Expression() methods
2 parents d340da9 + 427a28b commit dadcec8

File tree

12 files changed

+554
-364
lines changed

12 files changed

+554
-364
lines changed

UPGRADE.md

Lines changed: 27 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,32 @@
11
# Upgrade to 3.0
22

3+
## BC BREAK `AbstractPlatform::get*Expression()` methods no loner accept integer values as arguments
4+
5+
The following methods' arguments do not longer accept integer value:
6+
7+
- the `$expression` argument in `::getCountExpression()`,
8+
- the `$decimals` argument in `::getRoundExpression()`,
9+
- the `$seconds` argument in `::getDateAddSecondsExpression()`,
10+
- the `$seconds` argument in `::getDateSubSecondsExpression()`,
11+
- the `$minutes` argument in `::getDateAddMinutesExpression()`,
12+
- the `$minutes` argument in `::getDateSubMinutesExpression()`,
13+
- the `$hours` argument in `::getDateAddHourExpression()`,
14+
- the `$hours` argument in `::getDateAddHourExpression()`,
15+
- the `$days` argument in `::getDateAddDaysExpression()`,
16+
- the `$days` argument in `::getDateSubDaysExpression()`,
17+
- the `$weeks` argument in `::getDateAddWeeksExpression()`,
18+
- the `$weeks` argument in `::getDateSubWeeksExpression()`,
19+
- the `$months` argument in `::getDateAddMonthExpression()`,
20+
- the `$months` argument in `::getDateSubMonthExpression()`,
21+
- the `$quarters` argument in `::getDateAddQuartersExpression()`,
22+
- the `$quarters` argument in `::getDateSubQuartersExpression()`,
23+
- the `$years` argument in `::getDateAddYearsExpression()`,
24+
- the `$years` argument in `::getDateSubYearsExpression()`.
25+
26+
Please use the strings representing numeric SQL literals instead (e.g. `'1'` instead of `1`).
27+
28+
The signature of `AbstractPlatform::getConcatExpression()` changed to `::getConcatExpression(string ...$string)`.
29+
330
## BC BREAK The type of `$start` in `AbstractPlatform::getLocateExpression()` changed from `string|false` to `?string`
431

532
The default value of `$start` is now `null`, not `false`.

0 commit comments

Comments
 (0)