diff options
author | Felipe Artur <felipefac@gmail.com> | 2016-05-09 17:58:20 +0300 |
---|---|---|
committer | Felipe Artur <felipefac@gmail.com> | 2016-05-09 18:00:32 +0300 |
commit | 32811d98fe2893a0671fe22c4aebf8bd254b7709 (patch) | |
tree | 0b2286c17e8f22e7799f9303ddaead35ac87d06f /app | |
parent | d028863eda8b97f6e4db129ef57f0d3a2130c9b3 (diff) |
Make model sanitization methods one liners
Diffstat (limited to 'app')
-rw-r--r-- | app/models/label.rb | 5 | ||||
-rw-r--r-- | app/models/milestone.rb | 5 |
2 files changed, 4 insertions, 6 deletions
diff --git a/app/models/label.rb b/app/models/label.rb index 0b34911a4e9..c551ceef64e 100644 --- a/app/models/label.rb +++ b/app/models/label.rb @@ -117,9 +117,8 @@ class Label < ActiveRecord::Base LabelsHelper::text_color_for_bg(self.color) end - def title= value - value = Sanitize.clean(value.to_s) if value - write_attribute(:title, Sanitize.clean(value)) + def title=(value) + write_attribute(:title, Sanitize.clean(value.to_s)) if value.present? end private diff --git a/app/models/milestone.rb b/app/models/milestone.rb index ed81791c69c..15a786210fa 100644 --- a/app/models/milestone.rb +++ b/app/models/milestone.rb @@ -129,9 +129,8 @@ class Milestone < ActiveRecord::Base nil end - def title= value - value = Sanitize.clean(value.to_s) if value - write_attribute(:title, value) + def title=(value) + write_attribute(:title, Sanitize.clean(value.to_s)) if value.present? end # Sorts the issues for the given IDs. |