Confine token save to Settings model to avoid conflicts.

src/CraftQL.php

@@ -111,21 +111,4 @@ protected function settingsHtml()
             'settings' => $this->getSettings()
         ]);
     }
-
-    /**
-     * Save the settings, some custom work here to make sure token names
-     * are saved correctly
-     */
-    public function setSettings(array $settings)
-    {
-        parent::setSettings($settings);
-
-        if (isset($_POST['settings']['token'])) {
-            foreach ($_POST['settings']['token'] as $tokenId => $values) {
-                $token = Token::find()->where(['id' => $tokenId])->one();
-                $token->name = @$values['name'];
-                $token->save();
-            }
-        }
-    }
 }

src/Models/Settings.php

@@ -30,4 +30,23 @@ function tokens()
         $tokens = Token::find()->where(['userId' => \Craft::$app->user->id])->all();
         return $tokens;
     }
+
+    /**
+     * Make sure token names are saved correctly along with this model.
+     * 
+     * @inheritdoc
+     */
+    public function beforeValidate()
+    {
+        if (isset($_POST['settings']['token'])) {
+            foreach ($_POST['settings']['token'] as $tokenId => $values) {
+                $token = Token::find()->where(['id' => $tokenId])->one();
+                $token->name = @$values['name'];
+                $token->save();
+            }
+        }
+
+        return parent::beforeValidate();
+    }
+
 }