Skip to content

Add support for native MySQL driver (mysqli)#952

Merged
mvorisek merged 9 commits intodevelopfrom
mysqlnd
Jan 7, 2022
Merged

Add support for native MySQL driver (mysqli)#952
mvorisek merged 9 commits intodevelopfrom
mysqlnd

Conversation

@mvorisek
Copy link
Copy Markdown
Member

@mvorisek mvorisek commented Jan 6, 2022

to allow support of simultaneous queries :)

@mvorisek mvorisek force-pushed the mysqlnd branch 7 times, most recently from c8992b6 to 721d028 Compare January 7, 2022 09:41
@mvorisek mvorisek marked this pull request as ready for review January 7, 2022 10:27
public function groupConcat($field, string $delimiter = ',')
{
return $this->expr('group_concat({} separator [])', [$field, $delimiter]);
return $this->expr('group_concat({} separator \'' . str_replace('\'', '\'\'', $delimiter) . '\')', [$field]);
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

php/php-src#7903

was actually a hidden issue

@mvorisek mvorisek added the MAJOR label Jan 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants