diff options
| author | Teddy Wing | 2017-11-09 14:21:38 +0100 |
|---|---|---|
| committer | cedricnjanga | 2017-11-13 17:52:54 +0100 |
| commit | b172d774513ffbae5c8a88f39b3d4bb7e4ae51e0 (patch) | |
| tree | 30a85cb381f1b9732f2bfdbae9a459ad9eb07ac6 | |
| parent | 08bb5790c7637e338700ae7f8100770a4d1c9d6e (diff) | |
| download | chouette-core-b172d774513ffbae5c8a88f39b3d4bb7e4ae51e0.tar.bz2 | |
ComplianceCheckSet: Rename to `all_statuses_ok`
Suggestion from Robert to have a simpler name. I had just written it the
way it sounded in my head when I wrote the method.
Refs #4757
| -rw-r--r-- | app/models/compliance_check_set.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/app/models/compliance_check_set.rb b/app/models/compliance_check_set.rb index d2155cb9a..cce24410c 100644 --- a/app/models/compliance_check_set.rb +++ b/app/models/compliance_check_set.rb @@ -30,14 +30,14 @@ class ComplianceCheckSet < ActiveRecord::Base end end - if all_statuses_are_ok(statuses) + if all_statuses_ok(statuses) return update(status: 'successful') end end private - def all_statuses_are_ok(statuses) + def all_statuses_ok(statuses) uniform_statuses = statuses.uniq uniform_statuses.length == 1 && uniform_statuses.first == 'OK' end |
