uses more efficient count
This commit is contained in:
@@ -25,11 +25,11 @@ class Admin::StatsController < Admin::BaseController
|
|||||||
|
|
||||||
def proposal_notifications
|
def proposal_notifications
|
||||||
@proposal_notifications = ProposalNotification.all
|
@proposal_notifications = ProposalNotification.all
|
||||||
@proposals_with_notifications = @proposal_notifications.pluck(:proposal_id).uniq
|
@proposals_with_notifications = @proposal_notifications.distinct.count(:proposal_id)
|
||||||
end
|
end
|
||||||
|
|
||||||
def direct_messages
|
def direct_messages
|
||||||
@direct_messages = DirectMessage.all
|
@direct_messages = DirectMessage.count
|
||||||
@users_who_have_sent_message = DirectMessage.pluck(:sender_id).uniq
|
@users_who_have_sent_message = DirectMessage.distinct.count(:sender_id)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|||||||
@@ -7,14 +7,14 @@
|
|||||||
<div class="small-12 medium-3 column">
|
<div class="small-12 medium-3 column">
|
||||||
<p class="featured">
|
<p class="featured">
|
||||||
<%= t("admin.stats.direct_messages.total") %><br>
|
<%= t("admin.stats.direct_messages.total") %><br>
|
||||||
<span id="direct_messages_count" class="number"><%= @direct_messages.count %></span>
|
<span id="direct_messages_count" class="number"><%= @direct_messages %></span>
|
||||||
</p>
|
</p>
|
||||||
</div>
|
</div>
|
||||||
|
|
||||||
<div class="small-12 medium-6 column end">
|
<div class="small-12 medium-6 column end">
|
||||||
<p>
|
<p>
|
||||||
<%= t("admin.stats.direct_messages.users_who_have_sent_message") %><br>
|
<%= t("admin.stats.direct_messages.users_who_have_sent_message") %><br>
|
||||||
<span id="users_who_have_sent_message_count" class="number"><%= @users_who_have_sent_message.count %></span>
|
<span id="users_who_have_sent_message_count" class="number"><%= @users_who_have_sent_message %></span>
|
||||||
</p>
|
</p>
|
||||||
</div>
|
</div>
|
||||||
</div>
|
</div>
|
||||||
|
|||||||
@@ -7,14 +7,14 @@
|
|||||||
<div class="small-12 medium-3 column">
|
<div class="small-12 medium-3 column">
|
||||||
<p class="featured">
|
<p class="featured">
|
||||||
<%= t("admin.stats.proposal_notifications.total") %><br>
|
<%= t("admin.stats.proposal_notifications.total") %><br>
|
||||||
<span id="proposal_notifications_count" class="number"><%= @proposal_notifications.count %></span>
|
<span id="proposal_notifications_count" class="number"><%= @proposal_notifications %></span>
|
||||||
</p>
|
</p>
|
||||||
</div>
|
</div>
|
||||||
|
|
||||||
<div class="small-12 medium-6 column end">
|
<div class="small-12 medium-6 column end">
|
||||||
<p>
|
<p>
|
||||||
<%= t("admin.stats.proposal_notifications.proposals_with_notifications") %><br>
|
<%= t("admin.stats.proposal_notifications.proposals_with_notifications") %><br>
|
||||||
<span id="proposals_with_notifications_count" class="number"><%= @proposals_with_notifications.count %></span>
|
<span id="proposals_with_notifications_count" class="number"><%= @proposals_with_notifications %></span>
|
||||||
</p>
|
</p>
|
||||||
</div>
|
</div>
|
||||||
</div>
|
</div>
|
||||||
|
|||||||
Reference in New Issue
Block a user