Commit 2ee7c9dbf924cb70db11cd47112033c30628a959
Exists in
master
and in
1 other branch
Merge pull request #307 from lest/patch-1
use super in overrided attribute readers
Showing
2 changed files
with
3 additions
and
3 deletions
Show diff stats
app/models/app.rb
| ... | ... | @@ -99,12 +99,12 @@ class App |
| 99 | 99 | |
| 100 | 100 | # Legacy apps don't have notify_on_errs and notify_on_deploys params |
| 101 | 101 | def notify_on_errs |
| 102 | - !(self[:notify_on_errs] == false) | |
| 102 | + !(super == false) | |
| 103 | 103 | end |
| 104 | 104 | alias :notify_on_errs? :notify_on_errs |
| 105 | 105 | |
| 106 | 106 | def notify_on_deploys |
| 107 | - !(self[:notify_on_deploys] == false) | |
| 107 | + !(super == false) | |
| 108 | 108 | end |
| 109 | 109 | alias :notify_on_deploys? :notify_on_deploys |
| 110 | 110 | ... | ... |