Skip to content

Fix SQL type error - #28478

Merged
nickvergessen merged 1 commit into
nextcloud:masterfrom
jzombi:patch-1
Aug 20, 2021
Merged

Fix SQL type error#28478
nickvergessen merged 1 commit into
nextcloud:masterfrom
jzombi:patch-1

Conversation

@jzombi

Copy link
Copy Markdown
Contributor

Fixing the following SQL error encountered with PostgreSQL:

SQLSTATE[42883]: Undefined function: 7 ERROR:
No operator matches the given name and argument types. You might need to add explicit type casts.

This prevented the Nextcloud apps/files page to show the files and directories. See #27881

Fixing the following SQL error encountered with PostgreSQL:
SQLSTATE[42883]: Undefined function: 7 ERROR:
No operator matches the given name and argument types. You might need to add explicit type casts.
This prevented the Nextcloud apps/files page to show the files and directories. See nextcloud#27881
@szaimenszaimen added the 3. to review Waiting for reviews label Aug 17, 2021
@szaimenszaimen added this to the Nextcloud 23 milestone Aug 17, 2021
@ChristophWurst
ChristophWurst removed their request for review August 17, 2021 16:41
@kesselb

Copy link
Copy Markdown
Contributor

$table->addColumn('object_id', 'string', [
'notnull' => true,
'length' => 64,
'default' => '',
]);

object_id seems a string. Sounds good to use PARAM_STR_ARRAY here.

@kesselbkesselb added the bug label Aug 17, 2021
@kesselbkesselb linked an issue Aug 17, 2021 that may be closed by this pull request
@solracsf

Copy link
Copy Markdown
Member

Makes sense 👍

@nickvergessen
nickvergessen merged commit 83d46ef into nextcloud:masterAug 20, 2021
@welcome

welcomeBot commented Aug 20, 2021

Copy link
Copy Markdown

Thanks for your first pull request and welcome to the community! Feel free to keep them coming! If you are looking for issues to tackle then have a look at this selection: https://github.com/nextcloud/server/issues?q=is%3Aopen+is%3Aissue+label%3A%22good+first+issue%22

@szaimen

Copy link
Copy Markdown
Contributor

/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.

Cannot access files after upgrade

7 participants

@jzombi@kesselb@solracsf@szaimen@nickvergessen@blizzz@skjnldsv