Disable slug if an url_pattern is set

This will not have an effect if the user is editing the url_pattern. Also change the mechanic of the slug little bit for url_pattern page layouts. They can have multiple wildcards it good to see the whole url pattern as slug instead of part behind the last slash.

Author: kulturbande

app/models/alchemy/page/page_naming.rb

@@ -42,9 +42,13 @@ def update_urlname!
         end
       end
 
-      # Returns always the last part of a urlname path
+      # Returns url pattern or the last part of an urlname path
       def slug
-        urlname.to_s.split("/").last
+        page_layout_url_pattern.presence || urlname.to_s.split("/").last
+      end
+
+      def has_url_pattern?
+        page_layout_url_pattern.present?
       end
 
       private

app/views/alchemy/admin/pages/_form.html.erb

@@ -18,7 +18,10 @@
     </div>
 
     <%= f.input :name, autofocus: true %>
-    <%= f.input :urlname, as: 'string', input_html: {value: @page.slug}, label: Alchemy::Page.human_attribute_name(:slug) %>
+    <%= f.input :urlname, as: 'string', input_html: {
+      value: @page.slug,
+      disabled: @page.has_url_pattern?
+    }, label: Alchemy::Page.human_attribute_name(:slug) %>
 
     <%= f.fields_for :draft_version, @page.draft_version do |v| %>
       <alchemy-char-counter max-chars="60">

spec/models/alchemy/page_spec.rb

@@ -1978,6 +1978,22 @@ module Alchemy
           expect(page.slug).to be_nil
         end
       end
+
+      context "with a page layout that has a url_pattern" do
+        let(:page) { build(:alchemy_page, page_layout: "product_detail") }
+
+        it "returns the url_pattern instead of the urlname" do
+          expect(page.slug).to eq(":id")
+        end
+      end
+
+      context "with a multi-segment url_pattern" do
+        let(:page) { build(:alchemy_page, page_layout: "blog_post") }
+
+        it "returns the full url_pattern" do
+          expect(page.slug).to eq(":year/:slug")
+        end
+      end
     end
 
     context "page status methods" do