Enterprise permalink generator handles blank permalink case

This commit is contained in:
Rob Harrington
2015-01-21 14:40:28 +11:00
parent 7520552fd7
commit 36430d3bad
2 changed files with 6 additions and 0 deletions

View File

@@ -298,6 +298,7 @@ class Enterprise < ActiveRecord::Base
def self.find_available_permalink(test_permalink)
test_permalink = test_permalink.parameterize
test_permalink = "my-enterprise" if test_permalink.blank?
existing = Enterprise.select(:permalink).order(:permalink).where("permalink LIKE ?", "#{test_permalink}%").map(&:permalink)
if existing.empty?
test_permalink

View File

@@ -824,6 +824,11 @@ describe Enterprise do
expect(Enterprise.find_available_permalink("Some Unused Permalink")).to eq "some-unused-permalink"
end
it "sets the permalink to 'my-enterprise' if parametized permalink is blank" do
expect(Enterprise.find_available_permalink("")).to eq "my-enterprise"
expect(Enterprise.find_available_permalink("$$%{$**}$%}")).to eq "my-enterprise"
end
it "finds and index value based on existing permalinks" do
expect(Enterprise.find_available_permalink("permalink")).to eq "permalink2"
end