move migration
Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
This commit is contained in:
parent
644686c0ec
commit
d89edb28c4
8 changed files with 84 additions and 102 deletions
|
@ -156,6 +156,7 @@ return array(
|
|||
'OCA\\DAV\\Migration\\Version1005Date20180530124431' => $baseDir . '/../lib/Migration/Version1005Date20180530124431.php',
|
||||
'OCA\\DAV\\Migration\\Version1006Date20180619154313' => $baseDir . '/../lib/Migration/Version1006Date20180619154313.php',
|
||||
'OCA\\DAV\\Migration\\Version1007Date20181007225117' => $baseDir . '/../lib/Migration/Version1007Date20181007225117.php',
|
||||
'OCA\\DAV\\Migration\\Version1008Date20181030113700' => $baseDir . '/../lib/Migration/Version1008Date20181030113700.php',
|
||||
'OCA\\DAV\\RootCollection' => $baseDir . '/../lib/RootCollection.php',
|
||||
'OCA\\DAV\\Server' => $baseDir . '/../lib/Server.php',
|
||||
'OCA\\DAV\\Settings\\CalDAVSettings' => $baseDir . '/../lib/Settings/CalDAVSettings.php',
|
||||
|
|
|
@ -171,6 +171,7 @@ class ComposerStaticInitDAV
|
|||
'OCA\\DAV\\Migration\\Version1005Date20180530124431' => __DIR__ . '/..' . '/../lib/Migration/Version1005Date20180530124431.php',
|
||||
'OCA\\DAV\\Migration\\Version1006Date20180619154313' => __DIR__ . '/..' . '/../lib/Migration/Version1006Date20180619154313.php',
|
||||
'OCA\\DAV\\Migration\\Version1007Date20181007225117' => __DIR__ . '/..' . '/../lib/Migration/Version1007Date20181007225117.php',
|
||||
'OCA\\DAV\\Migration\\Version1008Date20181030113700' => __DIR__ . '/..' . '/../lib/Migration/Version1008Date20181030113700.php',
|
||||
'OCA\\DAV\\RootCollection' => __DIR__ . '/..' . '/../lib/RootCollection.php',
|
||||
'OCA\\DAV\\Server' => __DIR__ . '/..' . '/../lib/Server.php',
|
||||
'OCA\\DAV\\Settings\\CalDAVSettings' => __DIR__ . '/..' . '/../lib/Settings/CalDAVSettings.php',
|
||||
|
|
|
@ -1130,9 +1130,20 @@ class CardDavBackend implements BackendInterface, SyncSupport {
|
|||
}
|
||||
}
|
||||
|
||||
/**
|
||||
* Extract UID from vcard
|
||||
*
|
||||
* @param string $cardData the vcard raw data
|
||||
* @return string the uid or empty if none
|
||||
* @throws BadRequest
|
||||
*/
|
||||
private function getUID($cardData) {
|
||||
$vCard = Reader::read($cardData);
|
||||
$uid = $vCard->UID->getValue();
|
||||
return $uid;
|
||||
if ($vCard->UID) {
|
||||
$uid = $vCard->UID->getValue();
|
||||
return $uid;
|
||||
}
|
||||
// should already be handled, but just in case
|
||||
throw new BadRequest('vCards on CardDAV servers MUST have a UID property');
|
||||
}
|
||||
}
|
||||
|
|
52
apps/dav/lib/Migration/Version1008Date20181030113700.php
Normal file
52
apps/dav/lib/Migration/Version1008Date20181030113700.php
Normal file
|
@ -0,0 +1,52 @@
|
|||
<?php
|
||||
/**
|
||||
* @copyright Copyright (c) 2018, John Molakvoæ (skjnldsv@protonmail.com)
|
||||
*
|
||||
* @author John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
|
||||
*
|
||||
* @license GNU AGPL version 3 or any later version
|
||||
*
|
||||
* This program is free software: you can redistribute it and/or modify
|
||||
* it under the terms of the GNU Affero General Public License as
|
||||
* published by the Free Software Foundation, either version 3 of the
|
||||
* License, or (at your option) any later version.
|
||||
*
|
||||
* This program is distributed in the hope that it will be useful,
|
||||
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
||||
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
||||
* GNU Affero General Public License for more details.
|
||||
*
|
||||
* You should have received a copy of the GNU Affero General Public License
|
||||
* along with this program. If not, see <http://www.gnu.org/licenses/>.
|
||||
*
|
||||
*/
|
||||
|
||||
namespace OCA\DAV\Migration;
|
||||
|
||||
use Closure;
|
||||
|
||||
use Doctrine\DBAL\Types\Type;
|
||||
use OCP\DB\ISchemaWrapper;
|
||||
use OCP\Migration\SimpleMigrationStep;
|
||||
use OCP\Migration\IOutput;
|
||||
|
||||
/**
|
||||
* add column for share notes
|
||||
*
|
||||
* Class Version15000Date20180927120000
|
||||
*/
|
||||
class Version1008Date20181030113700 extends SimpleMigrationStep {
|
||||
public function changeSchema(IOutput $output, Closure $schemaClosure, array $options) {
|
||||
|
||||
/** @var ISchemaWrapper $schema */
|
||||
$schema = $schemaClosure();
|
||||
|
||||
$table = $schema->getTable('cards');
|
||||
$table->addColumn('uid', Type::STRING, [
|
||||
'notnull' => false,
|
||||
'length' => 255
|
||||
]);
|
||||
|
||||
return $schema;
|
||||
}
|
||||
}
|
|
@ -1,87 +0,0 @@
|
|||
<?php
|
||||
/**
|
||||
* @copyright Copyright (c) 2018 Bjoern Schiessle <bjoern@schiessle.org>
|
||||
*
|
||||
* @license GNU AGPL version 3 or any later version
|
||||
*
|
||||
* This program is free software: you can redistribute it and/or modify
|
||||
* it under the terms of the GNU Affero General Public License as
|
||||
* published by the Free Software Foundation, either version 3 of the
|
||||
* License, or (at your option) any later version.
|
||||
*
|
||||
* This program is distributed in the hope that it will be useful,
|
||||
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
||||
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
||||
* GNU Affero General Public License for more details.
|
||||
*
|
||||
* You should have received a copy of the GNU Affero General Public License
|
||||
* along with this program. If not, see <http://www.gnu.org/licenses/>.
|
||||
*
|
||||
*/
|
||||
|
||||
namespace OC\Core\Migrations;
|
||||
|
||||
use Doctrine\DBAL\Types\Type;
|
||||
use OCP\DB\ISchemaWrapper;
|
||||
use OCP\Migration\SimpleMigrationStep;
|
||||
|
||||
/**
|
||||
* add column for share notes
|
||||
*
|
||||
* Class Version15000Date20180927120000
|
||||
*/
|
||||
class Version15000Date20180927120000 extends SimpleMigrationStep {
|
||||
public function changeSchema(\OCP\Migration\IOutput $output, \Closure $schemaClosure, array $options) {
|
||||
|
||||
/** @var ISchemaWrapper $schema */
|
||||
$schema = $schemaClosure();
|
||||
|
||||
if ($schema->hasTable('cards')) {
|
||||
$table = $schema->getTable('cards');
|
||||
$table->addColumn('uid', Type::STRING, [
|
||||
'notnull' => false,
|
||||
'length' => 255
|
||||
]);
|
||||
} else {
|
||||
$table = $schema->createTable('cards');
|
||||
$table->addColumn('id', 'bigint', [
|
||||
'autoincrement' => true,
|
||||
'notnull' => true,
|
||||
'length' => 11,
|
||||
'unsigned' => true,
|
||||
]);
|
||||
$table->addColumn('addressbookid', 'integer', [
|
||||
'notnull' => true,
|
||||
'default' => 0,
|
||||
]);
|
||||
$table->addColumn('carddata', 'blob', [
|
||||
'notnull' => false,
|
||||
]);
|
||||
$table->addColumn('uri', 'string', [
|
||||
'notnull' => false,
|
||||
'length' => 255,
|
||||
]);
|
||||
$table->addColumn('lastmodified', 'bigint', [
|
||||
'notnull' => false,
|
||||
'length' => 11,
|
||||
'unsigned' => true,
|
||||
]);
|
||||
$table->addColumn('etag', 'string', [
|
||||
'notnull' => false,
|
||||
'length' => 32,
|
||||
]);
|
||||
$table->addColumn('size', 'bigint', [
|
||||
'notnull' => true,
|
||||
'length' => 11,
|
||||
'unsigned' => true,
|
||||
]);
|
||||
$table->addColumn('uid', Type::STRING, [
|
||||
'notnull' => false,
|
||||
'length' => 255
|
||||
]);
|
||||
$table->setPrimaryKey(['id']);
|
||||
}
|
||||
|
||||
return $schema;
|
||||
}
|
||||
}
|
|
@ -670,7 +670,6 @@ return array(
|
|||
'OC\\Core\\Migrations\\Version14000Date20180710092004' => $baseDir . '/core/Migrations/Version14000Date20180710092004.php',
|
||||
'OC\\Core\\Migrations\\Version14000Date20180712153140' => $baseDir . '/core/Migrations/Version14000Date20180712153140.php',
|
||||
'OC\\Core\\Migrations\\Version15000Date20180926101451' => $baseDir . '/core/Migrations/Version15000Date20180926101451.php',
|
||||
'OC\\Core\\Migrations\\Version15000Date20180927120000' => $baseDir . '/core/Migrations/Version15000Date20180927120000.php',
|
||||
'OC\\Core\\Migrations\\Version15000Date20181015062942' => $baseDir . '/core/Migrations/Version15000Date20181015062942.php',
|
||||
'OC\\DB\\Adapter' => $baseDir . '/lib/private/DB/Adapter.php',
|
||||
'OC\\DB\\AdapterMySQL' => $baseDir . '/lib/private/DB/AdapterMySQL.php',
|
||||
|
|
|
@ -700,7 +700,6 @@ class ComposerStaticInit53792487c5a8370acc0b06b1a864ff4c
|
|||
'OC\\Core\\Migrations\\Version14000Date20180710092004' => __DIR__ . '/../../..' . '/core/Migrations/Version14000Date20180710092004.php',
|
||||
'OC\\Core\\Migrations\\Version14000Date20180712153140' => __DIR__ . '/../../..' . '/core/Migrations/Version14000Date20180712153140.php',
|
||||
'OC\\Core\\Migrations\\Version15000Date20180926101451' => __DIR__ . '/../../..' . '/core/Migrations/Version15000Date20180926101451.php',
|
||||
'OC\\Core\\Migrations\\Version15000Date20180927120000' => __DIR__ . '/../../..' . '/core/Migrations/Version15000Date20180927120000.php',
|
||||
'OC\\Core\\Migrations\\Version15000Date20181015062942' => __DIR__ . '/../../..' . '/core/Migrations/Version15000Date20181015062942.php',
|
||||
'OC\\DB\\Adapter' => __DIR__ . '/../../..' . '/lib/private/DB/Adapter.php',
|
||||
'OC\\DB\\AdapterMySQL' => __DIR__ . '/../../..' . '/lib/private/DB/AdapterMySQL.php',
|
||||
|
|
|
@ -27,6 +27,7 @@ use OCP\IConfig;
|
|||
use OCP\IDBConnection;
|
||||
use OCP\Migration\IOutput;
|
||||
use OCP\Migration\IRepairStep;
|
||||
use Sabre\VObject\Reader;
|
||||
|
||||
class SetVcardDatabaseUID implements IRepairStep {
|
||||
const MAX_ROWS = 1000;
|
||||
|
@ -70,20 +71,27 @@ class SetVcardDatabaseUID implements IRepairStep {
|
|||
} while (count($rows) > 0);
|
||||
}
|
||||
|
||||
private function getUid($carddata) {
|
||||
preg_match('/UID:(.*)$/m', $carddata, $matches);
|
||||
if (count($matches) > 1) {
|
||||
return $matches[1];
|
||||
/**
|
||||
* Extract UID from vcard
|
||||
*
|
||||
* @param string $cardData the vcard raw data
|
||||
* @return string the uid or empty if none
|
||||
*/
|
||||
private function getUID(string $cardData): string {
|
||||
$vCard = Reader::read($cardData);
|
||||
if ($vCard->UID) {
|
||||
$uid = $vCard->UID->getValue();
|
||||
return $uid;
|
||||
}
|
||||
|
||||
return false;
|
||||
return '';
|
||||
}
|
||||
|
||||
/**
|
||||
* @param int $id
|
||||
* @param string $uid
|
||||
*/
|
||||
private function update($id, $uid) {
|
||||
private function update(int $id, string $uid) {
|
||||
if (!$this->updateQuery) {
|
||||
$builder = $this->connection->getQueryBuilder();
|
||||
|
||||
|
@ -98,16 +106,14 @@ class SetVcardDatabaseUID implements IRepairStep {
|
|||
$this->updateQuery->execute();
|
||||
}
|
||||
|
||||
private function repair() {
|
||||
private function repair(): int {
|
||||
$this->connection->beginTransaction();
|
||||
$entries = $this->getInvalidEntries();
|
||||
$count = 0;
|
||||
foreach ($entries as $entry) {
|
||||
$count++;
|
||||
$uid = $this->getUid($entry['carddata']);
|
||||
if ($uid !== false) {
|
||||
$this->update($entry['id'], $uid);
|
||||
}
|
||||
$uid = $this->getUID($entry['carddata']);
|
||||
$this->update($entry['id'], $uid);
|
||||
}
|
||||
$this->connection->commit();
|
||||
|
||||
|
|
Loading…
Reference in a new issue