Disable slug if a wildcard_url is set

It wouldn't work to update the slug anyway, because the urlname mechanic is different for pages with a wildcard_url. The slug form field will be disabled and the slug can also contain slashes.

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 wildcard url pattern or the last part of an urlname path
       def slug
-        urlname.to_s.split("/").last
+        wildcard_url&.pattern.presence || urlname.to_s.split("/").last
+      end
+
+      def has_wildcard_url?
+        wildcard_url&.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_wildcard_url?
+    }, 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

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