Skip to content

Allow casting query functions - #27852

Merged
juliusknorr merged 2 commits into
masterfrom
bugfix/noid/allow-casting-query-functions
Jul 13, 2021
Merged

Allow casting query functions#27852
juliusknorr merged 2 commits into
masterfrom
bugfix/noid/allow-casting-query-functions

Conversation

@nickvergessen

@nickvergessennickvergessen commented Jul 7, 2021

Copy link
Copy Markdown
Member

Regression from #25656 breaking in apps nextcloud/user_usage_report#69
So I first added a test with a sample query and then fixed it.

Fixnextcloud/user_usage_report#69

Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessennickvergessen added bug 3. to review Waiting for reviews labels Jul 7, 2021
@nickvergessennickvergessen added this to the Nextcloud 23 milestone Jul 7, 2021
@nickvergessen

Copy link
Copy Markdown
MemberAuthor

/backport to stable22

@PVince81PVince81 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍 makes sense

@juliusknorr
juliusknorr merged commit 0031152 into masterJul 13, 2021
@juliusknorr
juliusknorr deleted the bugfix/noid/allow-casting-query-functions branch July 13, 2021 06:48
@backportbot-nextcloud

Copy link
Copy Markdown

The backport to stable22 failed. Please do this backport manually.

@szaimenszaimen mentioned this pull request Aug 10, 2021
@nickvergessen

Copy link
Copy Markdown
MemberAuthor

/backport to stable22

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Breaking with Nextcloud 22.0.0 - SQLSTATE[42000]: Syntax error or access violation

3 participants

@nickvergessen@PVince81@juliusknorr