Skip to content

Conversation

@bartv2
Copy link
Contributor

@bartv2 bartv2 commented Feb 26, 2014

see #7415

@PVince81
Copy link
Contributor

It doesn't seem to work, I still get the same error. Already tested this two times.
Are you sure it's the right place ?
The error still shows the first column as unquoted:

An exception occurred while executing 'ALTER TABLE `oc_privatedata` CHANGE user `user` VARCHAR(255) DEFAULT '' NOT NULL, CHANGE app `app` VARCHAR(255) DEFAULT '' NOT NULL, CHANGE key `key` VARCHAR(255) DEFAULT '' NOT NULL, CHANGE value `value` VARCHAR(255) DEFAULT '' NOT NULL': SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near 'key `key` VARCHAR(255) DEFAULT '' NOT NULL, CHANGE value `value` VARCHAR(255) DE' at line 1

@PVince81
Copy link
Contributor

{"app":"PHP","message":"Undefined property: Doctrine\\DBAL\\Schema\\TableDiff::$oldColumnName at \/srv\/www\/htdocs\/owncloud2\/lib\/private\/db\/mdb2schemamanager.php#85","level":3,"time":"2014-02-26T15:06:19+00:00"}

@PVince81
Copy link
Contributor

Looks like you need to iterate over $tableDiff->changedColumns (saw it in the debugger) and quote their names

@bartv2
Copy link
Contributor Author

bartv2 commented Feb 26, 2014

@PVince81 can you do the change?

@PVince81
Copy link
Contributor

Ok, I'll take over this and let you review it

@PVince81
Copy link
Contributor

New PR: #7430 (had trouble force pushing)

@PVince81 PVince81 closed this Feb 26, 2014
@PVince81 PVince81 deleted the quote-oldcolumnname branch February 26, 2014 15:32
@ghost
Copy link

ghost commented Feb 26, 2014

💣 Test Failed. 💣
Refer to this link for build results: https://ci.owncloud.org/job/pull-request-analyser/3371/

@PVince81
Copy link
Contributor

@owncloud-bot retest this please

@PVince81
Copy link
Contributor

@owncloud-bot ah no forget it forget it!

@scrutinizer-notifier
Copy link

The inspection completed: No new issues

@lock lock bot locked as resolved and limited conversation to collaborators Aug 21, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants