htwkmooc/codeocean
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Leverage PostgreSQL count for study group size

Browse Source
This commit is contained in:
Sebastian Serth
2021-01-23 00:28:11 +01:00
parent 3b58cadbeb
commit 72b3524b18
3 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
app/models/study_group.rb
View File

@ -11,6 +11,10 @@ class StudyGroup < ApplicationRecord
external_users + internal_users external_users + internal_users
end end
def user_count
external_users.count + internal_users.count
end
def to_s def to_s
if name.blank? if name.blank?
"StudyGroup " + id.to_s "StudyGroup " + id.to_s

2
app/views/study_groups/index.html.slim
View File

@ -25,7 +25,7 @@ h1 = StudyGroup.model_name.human(count: 2)
td td
code = group.external_id code = group.external_id
td = link_to_if(policy(group.consumer).show?, group.consumer, group.consumer) td = link_to_if(policy(group.consumer).show?, group.consumer, group.consumer)
td = group.users.count td = group.user_count
td = link_to(t('shared.show'), group) if policy(group).show? td = link_to(t('shared.show'), group) if policy(group).show?
td = link_to(t('shared.edit'), edit_study_group_path(group)) if policy(group).edit? td = link_to(t('shared.edit'), edit_study_group_path(group)) if policy(group).edit?
td = link_to(t('shared.destroy'), group, data: {confirm: t('shared.confirm_destroy')}, method: :delete) if policy(group).destroy? td = link_to(t('shared.destroy'), group, data: {confirm: t('shared.confirm_destroy')}, method: :delete) if policy(group).destroy?

2
app/views/study_groups/show.html.slim
View File

@ -7,7 +7,7 @@ h1
= row(label: 'study_group.external_id') do = row(label: 'study_group.external_id') do
code = @study_group.external_id code = @study_group.external_id
= row(label: 'study_group.consumer', value: link_to_if(policy(@study_group).show?, @study_group.consumer, @study_group.consumer)) = row(label: 'study_group.consumer', value: link_to_if(policy(@study_group).show?, @study_group.consumer, @study_group.consumer))
= row(label: 'study_group.member_count', value: @study_group.users.count) = row(label: 'study_group.member_count', value: @study_group.user_count)
h2.mt-4 = t('activerecord.attributes.study_group.members') h2.mt-4 = t('activerecord.attributes.study_group.members')