Fix #4924: Moderator dashboard shouldn't show a "Promote" link that's unusable by non-moderators
Remove these links entirely.
This commit is contained in:
@@ -206,25 +206,6 @@ module ApplicationHelper
|
|||||||
link_to(text, user, class: user_class, data: data)
|
link_to(text, user, class: user_class, data: data)
|
||||||
end
|
end
|
||||||
|
|
||||||
def mod_link_to_user(user, positive_or_negative)
|
|
||||||
html = ""
|
|
||||||
html << link_to_user(user)
|
|
||||||
|
|
||||||
if CurrentUser.is_gold? && user != CurrentUser.user
|
|
||||||
if positive_or_negative == :positive
|
|
||||||
html << " [" + link_to("+", new_user_feedback_path(:user_feedback => {:category => "positive", :user_id => user.id})) + "]"
|
|
||||||
|
|
||||||
if CurrentUser.is_moderator? && !user.is_builder?
|
|
||||||
html << " [" + link_to("promote", edit_admin_user_path(user)) + "]"
|
|
||||||
end
|
|
||||||
else
|
|
||||||
html << " [" + link_to("–".html_safe, new_user_feedback_path(:user_feedback => {:category => "negative", :user_id => user.id})) + "]"
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
html.html_safe
|
|
||||||
end
|
|
||||||
|
|
||||||
def embed_wiki(title, **options)
|
def embed_wiki(title, **options)
|
||||||
wiki = WikiPage.find_by(title: title)
|
wiki = WikiPage.find_by(title: title)
|
||||||
text = format_text(wiki&.body)
|
text = format_text(wiki&.body)
|
||||||
|
|||||||
@@ -9,7 +9,7 @@
|
|||||||
<tbody>
|
<tbody>
|
||||||
<% @dashboard.artists.each do |activity| %>
|
<% @dashboard.artists.each do |activity| %>
|
||||||
<tr>
|
<tr>
|
||||||
<td><%= mod_link_to_user(activity.user, :positive) %></td>
|
<td><%= link_to_user(activity.user) %></td>
|
||||||
<td><%= link_to activity.count, artist_versions_path(:search => {:updater_id => activity.user.id}) %></td>
|
<td><%= link_to activity.count, artist_versions_path(:search => {:updater_id => activity.user.id}) %></td>
|
||||||
</tr>
|
</tr>
|
||||||
<% end %>
|
<% end %>
|
||||||
|
|||||||
@@ -21,7 +21,7 @@
|
|||||||
<% end %>
|
<% end %>
|
||||||
</div>
|
</div>
|
||||||
</td>
|
</td>
|
||||||
<td><%= mod_link_to_user(activity.comment.creator, :negative) %></td>
|
<td><%= link_to_user(activity.comment.creator) %></td>
|
||||||
<td><%= activity.count %></td>
|
<td><%= activity.count %></td>
|
||||||
<td><%= activity.comment.score %></td>
|
<td><%= activity.comment.score %></td>
|
||||||
</tr>
|
</tr>
|
||||||
|
|||||||
@@ -9,7 +9,7 @@
|
|||||||
<tbody>
|
<tbody>
|
||||||
<% @dashboard.notes.each do |activity| %>
|
<% @dashboard.notes.each do |activity| %>
|
||||||
<tr>
|
<tr>
|
||||||
<td><%= mod_link_to_user(activity.user, :positive) %></td>
|
<td><%= link_to_user(activity.user) %></td>
|
||||||
<td><%= link_to activity.count, note_versions_path(:search => {:updater_id => activity.user.id}) %></td>
|
<td><%= link_to activity.count, note_versions_path(:search => {:updater_id => activity.user.id}) %></td>
|
||||||
</tr>
|
</tr>
|
||||||
<% end %>
|
<% end %>
|
||||||
|
|||||||
@@ -9,7 +9,7 @@
|
|||||||
<tbody>
|
<tbody>
|
||||||
<% @dashboard.tags.each do |activity| %>
|
<% @dashboard.tags.each do |activity| %>
|
||||||
<tr>
|
<tr>
|
||||||
<td><%= mod_link_to_user(activity.user, :positive) %></td>
|
<td><%= link_to_user(activity.user) %></td>
|
||||||
<td><%= link_to activity.count, post_versions_path(:search => {:updater_id => activity.user.id}) %></td>
|
<td><%= link_to activity.count, post_versions_path(:search => {:updater_id => activity.user.id}) %></td>
|
||||||
</tr>
|
</tr>
|
||||||
<% end %>
|
<% end %>
|
||||||
|
|||||||
@@ -9,7 +9,7 @@
|
|||||||
<tbody>
|
<tbody>
|
||||||
<% @dashboard.posts.each do |activity| %>
|
<% @dashboard.posts.each do |activity| %>
|
||||||
<tr>
|
<tr>
|
||||||
<td><%= mod_link_to_user(activity.user, :positive) %></td>
|
<td><%= link_to_user(activity.user) %></td>
|
||||||
<td><%= link_to activity.count, posts_path(:tags => "user:#{activity.user.name}") %></td>
|
<td><%= link_to activity.count, posts_path(:tags => "user:#{activity.user.name}") %></td>
|
||||||
</tr>
|
</tr>
|
||||||
<% end %>
|
<% end %>
|
||||||
|
|||||||
@@ -9,7 +9,7 @@
|
|||||||
<tbody>
|
<tbody>
|
||||||
<% @dashboard.wiki_pages.each do |activity| %>
|
<% @dashboard.wiki_pages.each do |activity| %>
|
||||||
<tr>
|
<tr>
|
||||||
<td><%= mod_link_to_user(activity.user, :positive) %></td>
|
<td><%= link_to_user(activity.user) %></td>
|
||||||
<td><%= link_to activity.count, wiki_page_versions_path(:search => {:updater_id => activity.user.id}) %></td>
|
<td><%= link_to activity.count, wiki_page_versions_path(:search => {:updater_id => activity.user.id}) %></td>
|
||||||
</tr>
|
</tr>
|
||||||
<% end %>
|
<% end %>
|
||||||
|
|||||||
Reference in New Issue
Block a user