1.5.0: Made column changes to be one per migration to support sqlite

- Fixed bug causing seeding to not work

models/Menu.php

@@ -258,11 +258,16 @@ public function beforeSave()
      */
     public function setUrlAttribute($value)
     {
-        if ( $this->is_external ) {
+        $urlValue = null;
+        if ( $this->is_external && !empty($this->external_url) ) {
             $urlValue = $this->external_url;
-        } else {
+        } elseif (!$this->is_external && !empty($this->internal_url)) {
             $urlValue = $this->internal_url;
         }
+        // Allow seeding
+        if (empty($urlValue) && !empty($value)) {
+            $urlValue = $value;
+        }
         $this->attributes['url'] = $urlValue;
     }
 }

updates/add_enabled_parameters_query_fields_1.php

@@ -0,0 +1,25 @@
+<?php namespace BenFreke\MenuManager\Updates;
+
+use Schema;
+use October\Rain\Database\Updates\Migration;
+
+class AddEnabledParametersQueryFields1 extends Migration
+{
+
+    public function up()
+    {
+        Schema::table('benfreke_menumanager_menus', function($table)
+        {
+            $table->integer('enabled')->default(1);
+        });
+    }
+
+    public function down()
+    {
+        Schema::table('benfreke_menumanager_menus', function($table)
+        {
+            $table->dropColumn('enabled');
+        });
+    }
+
+}

updates/add_enabled_parameters_query_fields_2.php

@@ -0,0 +1,25 @@
+<?php namespace BenFreke\MenuManager\Updates;
+
+use Schema;
+use October\Rain\Database\Updates\Migration;
+
+class AddEnabledParametersQueryFields2 extends Migration
+{
+
+    public function up()
+    {
+        Schema::table('benfreke_menumanager_menus', function($table)
+        {
+            $table->string('parameters')->nullable();
+        });
+    }
+
+    public function down()
+    {
+        Schema::table('benfreke_menumanager_menus', function($table)
+        {
+            $table->dropColumn('parameters');
+        });
+    }
+
+}

updates/add_enabled_parameters_query_fields.php → updates/add_enabled_parameters_query_fields_3.php

@@ -3,15 +3,13 @@
 use Schema;
 use October\Rain\Database\Updates\Migration;
 
-class AddEnabledParametersQueryFields extends Migration
+class AddEnabledParametersQueryFields3 extends Migration
 {
 
     public function up()
     {
         Schema::table('benfreke_menumanager_menus', function($table)
         {
-            $table->integer('enabled')->default(1);
-            $table->string('parameters')->nullable();
             $table->string('query_string')->nullable();
         });
     }
@@ -20,8 +18,6 @@ public function down()
     {
         Schema::table('benfreke_menumanager_menus', function($table)
         {
-            $table->dropColumn('enabled');
-            $table->dropColumn('parameters');
             $table->dropColumn('query_string');
         });
     }

updates/version.yaml

@@ -17,7 +17,9 @@
   - add_link_target_field.php
 1.1.1:
   - Added ability to enable/disable individual menu links; Added ability for url parameters & query string; Fixed issue of "getLinkHref()" pulling through full page url with parameters rather than the ACTUAL page url
-  - add_enabled_parameters_query_fields.php
+  - add_enabled_parameters_query_fields_1.php
+  - add_enabled_parameters_query_fields_2.php
+  - add_enabled_parameters_query_fields_3.php
 1.1.2:
   - Reformatted code for better maintainability and better practises
 1.1.3: