diff options
| author | Louise Crow <louise.crow@gmail.com> | 2015-01-23 14:39:43 +0000 |
|---|---|---|
| committer | Gareth Rees <gareth@mysociety.org> | 2015-02-20 11:59:04 +0000 |
| commit | 93458e4f59108144e90ac11ec582f1bba61e808f (patch) | |
| tree | c076288741837d200893dc259e788f98c88ee802 | |
| parent | b9905fb9233991113483899551c73d79a583732c (diff) | |
Standard separation of methods.
Comment above a method should apply to that method only
| -rw-r--r-- | app/models/public_body.rb | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/app/models/public_body.rb b/app/models/public_body.rb index b8163b07d..f26246660 100644 --- a/app/models/public_body.rb +++ b/app/models/public_body.rb @@ -235,7 +235,7 @@ class PublicBody < ActiveRecord::Base return self.has_tag?('defunct') end - # Can an FOI (etc.) request be made to this body, and if not why not? + # Can an FOI (etc.) request be made to this body? def is_requestable? if self.defunct? return false @@ -248,6 +248,7 @@ class PublicBody < ActiveRecord::Base end return !self.request_email.empty? && self.request_email != 'blank' end + # Strict superset of is_requestable? def is_followupable? if self.request_email.nil? @@ -255,6 +256,7 @@ class PublicBody < ActiveRecord::Base end return !self.request_email.empty? && self.request_email != 'blank' end + # Also used as not_followable_reason def not_requestable_reason if self.defunct? |