Fix #3677

CHANGELOG-v3.md

@@ -4,6 +4,7 @@
 
 ### Fixed
 - Fixed an error that occurred when updating to Craft 3.1 if a plugin or module was calling any soft-deletable records’ `find()` methods.
+- Fixed an error that occurred when updating from Craft 2 to Craft 3.1 if there were any RichText fields. ([#3677](https://github.com/craftcms/cms/issues/3677))
 
 ## 3.1.2.2 - 2019-01-19
 

src/migrations/m160707_000001_rename_richtext_assetsource_setting.php

@@ -21,7 +21,7 @@ public function safeUp()
         $fields = (new Query())
             ->select(['id', 'settings'])
             ->from([Table::FIELDS])
-            ->where(['type' => 'craft\\redactor\\Fields'])
+            ->where(['type' => 'craft\\redactor\\Field'])
             ->all($this->db);
 
         foreach ($fields as $field) {

src/migrations/m190121_120000_fix_redactor_settings.php

@@ -0,0 +1,45 @@
+<?php
+
+namespace craft\migrations;
+
+use craft\db\Migration;
+use craft\db\Query;
+use craft\db\Table;
+use craft\helpers\ArrayHelper;
+use craft\helpers\Json;
+
+/**
+ * m190121_120000_fix_redactor_settings migration.
+ */
+class m190121_120000_fix_redactor_settings extends Migration
+{
+    /**
+     * @inheritdoc
+     */
+    public function safeUp()
+    {
+        $fields = (new Query())
+            ->select(['id', 'settings'])
+            ->from([Table::FIELDS])
+            ->where(['type' => 'craft\\redactor\\Field'])
+            ->all($this->db);
+
+        foreach ($fields as $field) {
+            $settings = Json::decode($field['settings']);
+            if (is_array($settings) && array_key_exists('availableAssetSources', $settings)) {
+                $settings['availableVolumes'] = ArrayHelper::remove($settings, 'availableAssetSources');
+                $this->update(Table::FIELDS, ['settings' => Json::encode($settings)], ['id' => $field['id']]);
+            }
+        }
+    }
+
+    /**
+     * @inheritdoc
+     */
+    public function safeDown()
+    {
+        echo "m190121_120000_fix_redactor_settings cannot be reverted.\n";
+
+        return false;
+    }
+}