FIX: handle non-logged in user trying to skip wizard (#154)

* FIX: handle non-logged in user trying to skip wizard

* update version, fixed formatting

* make sure skipping user is logged in

assets/javascripts/wizard/models/custom.js.es6

@@ -2,6 +2,7 @@ import { default as computed } from "discourse-common/utils/decorators";
 import getUrl from "discourse-common/lib/get-url";
 import WizardField from "wizard/models/wizard-field";
 import { ajax } from "wizard/lib/ajax";
+import { popupAjaxError } from "discourse/lib/ajax-error";
 import Step from "wizard/models/step";
 import EmberObject from "@ember/object";
 import Site from "./site";
@@ -24,15 +25,19 @@ const CustomWizard = EmberObject.extend({
 
 CustomWizard.reopenClass({
   skip(wizardId) {
-    ajax({ url: `/w/${wizardId}/skip`, type: "PUT" }).then((result) => {
+    ajax({ url: `/w/${wizardId}/skip`, type: "PUT" })
-      CustomWizard.finished(result);
+      .then((result) => {
-    });
+        CustomWizard.finished(result);
+      })
+      .catch(popupAjaxError);
   },
 
   restart(wizardId) {
-    ajax({ url: `/w/${wizardId}/skip`, type: "PUT" }).then(() => {
+    ajax({ url: `/w/${wizardId}/skip`, type: "PUT" })
-      window.location.href = `/w/${wizardId}`;
+      .then(() => {
-    });
+        window.location.href = `/w/${wizardId}`;
+      })
+      .catch(popupAjaxError);
   },
 
   finished(result) {

controllers/custom_wizard/wizard.rb

@@ -5,6 +5,7 @@ class CustomWizard::WizardController < ::ApplicationController
   layout 'wizard'
 
   before_action :ensure_plugin_enabled
+  before_action :ensure_logged_in, only: [:skip]
   helper_method :wizard_page_title
   helper_method :wizard_theme_id
   helper_method :wizard_theme_lookup

plugin.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 # name: discourse-custom-wizard
 # about: Create custom wizards
-# version: 1.15.2
+# version: 1.15.3
 # authors: Angus McLeod
 # url: https://github.com/paviliondev/discourse-custom-wizard
 # contact emails: angus@thepavilion.io