Skip to content

Commit

Permalink
Make PersonCard#memberships docs less ambiguous
Browse files Browse the repository at this point in the history
The use of the word "current" was a bit misleading in this context, it
could mean either the current page this class is being used on or the
current term that's in session right now.
  • Loading branch information
chrismytton committed Feb 23, 2017
1 parent fa310a1 commit fc67a72
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion lib/person_cards.rb
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ def identifiers
Section::Identifiers.new(person, top_identifiers: top_identifiers).data
end

# List of memberships this person holds in the current term
# List of memberships this person held in this term
# @return [Array<EveryPolitician::Popolo::Membership>]
def memberships
person.memberships.where(legislative_period_id: term.id)
Expand Down

0 comments on commit fc67a72

Please sign in to comment.