artists: don't show "Status Active" for undeleted artists.
An "active" artist entry is one that isn't deleted. Saying that every undeleted artist is "active" confuses people.
This commit is contained in:
@@ -287,18 +287,6 @@ class Artist < ApplicationRecord
|
|||||||
include BanMethods
|
include BanMethods
|
||||||
extend SearchMethods
|
extend SearchMethods
|
||||||
|
|
||||||
def status
|
|
||||||
if is_banned? && !is_deleted?
|
|
||||||
"Banned"
|
|
||||||
elsif is_banned?
|
|
||||||
"Banned Deleted"
|
|
||||||
elsif is_deleted?
|
|
||||||
"Deleted"
|
|
||||||
else
|
|
||||||
"Active"
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
def self.available_includes
|
def self.available_includes
|
||||||
[:members, :urls, :wiki_page, :tag_alias, :tag]
|
[:members, :urls, :wiki_page, :tag_alias, :tag]
|
||||||
end
|
end
|
||||||
|
|||||||
@@ -1,6 +1,18 @@
|
|||||||
<%# artist %>
|
<%# artist %>
|
||||||
<ul>
|
<ul>
|
||||||
<li><strong>Status</strong> <%= artist.status %></li>
|
<% if artist.is_banned? || artist.is_deleted? %>
|
||||||
|
<li>
|
||||||
|
<strong>Status</strong>
|
||||||
|
|
||||||
|
<% if artist.is_banned? %>
|
||||||
|
Banned
|
||||||
|
<% end %>
|
||||||
|
|
||||||
|
<% if artist.is_deleted? %>
|
||||||
|
Deleted
|
||||||
|
<% end %>
|
||||||
|
</li>
|
||||||
|
<% end %>
|
||||||
|
|
||||||
<% if artist.other_names.present? %>
|
<% if artist.other_names.present? %>
|
||||||
<li>
|
<li>
|
||||||
|
|||||||
Reference in New Issue
Block a user