@@ -12,7 +12,6 @@ class Artist < ApplicationRecord
|
|||||||
after_save :update_wiki
|
after_save :update_wiki
|
||||||
after_save :clear_url_string_changed
|
after_save :clear_url_string_changed
|
||||||
validates :name, tag_name: true, uniqueness: true
|
validates :name, tag_name: true, uniqueness: true
|
||||||
validate :validate_wiki, :on => :create
|
|
||||||
belongs_to_creator
|
belongs_to_creator
|
||||||
has_many :members, :class_name => "Artist", :foreign_key => "group_name", :primary_key => "name"
|
has_many :members, :class_name => "Artist", :foreign_key => "group_name", :primary_key => "name"
|
||||||
has_many :urls, :dependent => :destroy, :class_name => "ArtistUrl", :autosave => true
|
has_many :urls, :dependent => :destroy, :class_name => "ArtistUrl", :autosave => true
|
||||||
@@ -391,13 +390,6 @@ class Artist < ApplicationRecord
|
|||||||
wiki_page.save
|
wiki_page.save
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
def validate_wiki
|
|
||||||
if WikiPage.titled(name).exists?
|
|
||||||
errors.add(:name, "conflicts with a wiki page")
|
|
||||||
return false
|
|
||||||
end
|
|
||||||
end
|
|
||||||
end
|
end
|
||||||
|
|
||||||
module TagMethods
|
module TagMethods
|
||||||
|
|||||||
@@ -128,18 +128,6 @@ class ArtistTest < ActiveSupport::TestCase
|
|||||||
assert_equal(artist.name, artist.wiki_page.title)
|
assert_equal(artist.name, artist.wiki_page.title)
|
||||||
end
|
end
|
||||||
|
|
||||||
context "when a wiki page with the same name already exists" do
|
|
||||||
setup do
|
|
||||||
@wiki_page = FactoryBot.create(:wiki_page, :title => "aaa")
|
|
||||||
@artist = FactoryBot.build(:artist, :name => "aaa")
|
|
||||||
end
|
|
||||||
|
|
||||||
should "not validate" do
|
|
||||||
@artist.save
|
|
||||||
assert_equal(["Name conflicts with a wiki page"], @artist.errors.full_messages)
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
should "update the wiki page when notes are assigned" do
|
should "update the wiki page when notes are assigned" do
|
||||||
artist = FactoryBot.create(:artist, :name => "aaa", :notes => "testing")
|
artist = FactoryBot.create(:artist, :name => "aaa", :notes => "testing")
|
||||||
artist.update_attribute(:notes, "kokoko")
|
artist.update_attribute(:notes, "kokoko")
|
||||||
|
|||||||
Reference in New Issue
Block a user