Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

issue with migrateto21.php #566

Closed
ma2thieu opened this issue Nov 22, 2014 · 3 comments
Closed

issue with migrateto21.php #566

ma2thieu opened this issue Nov 22, 2014 · 3 comments
Assignees
Labels

Comments

@ma2thieu
Copy link

I run into multiple issues with migrateto21.php

PHP Fatal error: Uncaught exception 'PDOException' with message 'SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '? WHERE id = ?' at line 1' in /var/www/dav/vendor/sabre/dav/bin/migrateto21.php:116

I think $stmt = $pdo->query('UPDATE calendarobjects SET uid = ? WHERE id = ?'); should be $stmt = $pdo->prepare('UPDATE calendarobjects SET uid = ? WHERE id = ?');

Also :
PHP Warning: Missing argument 1 for Sabre\VObject\Component::select()
PHP Notice: Undefined variable: name in /var/www/dav/vendor/sabre/vobject/lib/Component.php on line 219

And anyway isn't $uid always null ?
line 139 : maybe $uid = (string)$uid; should be $uid = (string)$item->UID; ?

@evert
Copy link
Member

evert commented Nov 22, 2014

Hi @ma2thieu ,

I'm going through the full migration process again... this doesn't look too good. Thanks for the report.

@evert evert added the bug label Nov 22, 2014
@evert evert self-assigned this Nov 22, 2014
@ma2thieu
Copy link
Author

Hi @evert
Thank you but I think there is still the issue with $uid being always null
(line 139) $uid = (string) $item->UID;

@evert
Copy link
Member

evert commented Nov 23, 2014

Shit you are right :(

Will have to do another release. Good thing this is not as much of an issue. For now it just means that scheduling will not work well for events that were created before the migration, but it won't break anything.

evert added a commit that referenced this issue Nov 23, 2014
@evert evert closed this as completed in 8e7b566 Nov 24, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants