avoid empty null default with value that will be inserted anyways

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

Author: blizzz

apps/workflowengine/lib/Migration/PopulateNewlyIntroducedDatabaseFields.php

@@ -25,7 +25,6 @@
 namespace OCA\WorkflowEngine\Migration;
 
 use Doctrine\DBAL\Driver\Statement;
-use OCA\WorkflowEngine\Entity\File;
 use OCP\IDBConnection;
 use OCP\Migration\IOutput;
 use OCP\Migration\IRepairStep;
@@ -51,17 +50,6 @@ public function run(IOutput $output) {
 
 		$result->closeCursor();
 
-		$this->populateEntityCol();
-	}
-
-	protected function populateEntityCol() {
-		$qb = $this->dbc->getQueryBuilder();
-
-		$qb->update('flow_operations')
-			->set('entity', $qb->createNamedParameter(File::class))
-			->where($qb->expr()->emptyString('entity'))
-			->execute();
-
 	}
 
 	protected function populateScopeTable(Statement $ids): void {

apps/workflowengine/lib/Migration/Version2000Date20190808074233.php

@@ -7,6 +7,7 @@
 use Closure;
 use Doctrine\DBAL\Schema\Table;
 use Doctrine\DBAL\Types\Type;
+use OCA\WorkflowEngine\Entity\File;
 use OCP\DB\ISchemaWrapper;
 use OCP\Migration\IOutput;
 use OCP\Migration\SimpleMigrationStep;
@@ -116,7 +117,7 @@ protected function ensureEntityColumns(Table $table) {
 			$table->addColumn('entity', Type::STRING, [
 				'notnull' => true,
 				'length' => 256,
-				'default' => '',
+				'default' => File::class,
 			]);
 		}
 		if(!$table->hasColumn('events')) {