FIX: Properly allow no group to be selected (#33)

assets/javascripts/discourse/controllers/admin-plugins-discourse-subscriptions-products-show-plans-show.js.es6

@@ -1,14 +1,31 @@
 import discourseComputed from "discourse-common/utils/decorators";
 import DiscourseURL from "discourse/lib/url";
 import Controller from "@ember/controller";
+import { alias } from "@ember/object/computed";
 
 const RECURRING = "recurring";
 const ONE_TIME = "one_time";
 
 export default Controller.extend({
   // Also defined in settings.
-  selectedCurrency: Ember.computed.alias("model.plan.currency"),
+  selectedCurrency: alias("model.plan.currency"),
-  selectedInterval: Ember.computed.alias("model.plan.interval"),
+  selectedInterval: alias("model.plan.interval"),
+
+  @discourseComputed("model.plan.metadata.group_name")
+  selectedGroup(groupName) {
+    return groupName || "no-group";
+  },
+
+  @discourseComputed("model.groups")
+  availableGroups(groups) {
+    return [
+      {
+        id: null,
+        name: "no-group",
+      },
+      ...groups,
+    ];
+  },
 
   @discourseComputed
   currencies() {
@@ -57,13 +74,9 @@ export default Controller.extend({
     },
 
     createPlan() {
-      // TODO: set default group name beforehand
+      if (this.model.plan.metadata.group_name === "no-group") {
-      if (this.get("model.plan.metadata.group_name") === undefined) {
+        this.set("model.plan.metadata.group_name", null);
-        this.set("model.plan.metadata", {
-          group_name: this.get("model.groups.firstObject.name"),
-        });
       }
-
       this.get("model.plan")
         .save()
         .then(() => this.redirect(this.productId))
@@ -73,6 +86,9 @@ export default Controller.extend({
     },
 
     updatePlan() {
+      if (this.model.plan.metadata.group_name === "no-group") {
+        this.set("model.plan.metadata.group_name", null);
+      }
       this.get("model.plan")
         .update()
         .then(() => this.redirect(this.productId))

assets/javascripts/discourse/routes/admin-plugins-discourse-subscriptions-products-show-plans-show.js.es6

@@ -20,6 +20,9 @@ export default Route.extend({
         isRecurring: true,
         currency: Discourse.SiteSettings.discourse_subscriptions_currency,
         product: product.get("id"),
+        metadata: {
+          group_name: null,
+        },
       });
     } else {
       plan = AdminPlan.find(id).then((result) => {

assets/javascripts/discourse/templates/admin/plugins-discourse-subscriptions-products-show-plans-show.hbs

@@ -17,8 +17,8 @@
     <label for="interval">{{i18n 'discourse_subscriptions.admin.plans.plan.group'}}</label>
     {{combo-box
       valueProperty="name"
-      content=model.groups 
+      content=availableGroups
-      value=model.plan.metadata.group_name 
+      value=selectedGroup
       onChange=(action (mut model.plan.metadata.group_name))
     }}
     <div class="control-instructions">