Яuбy / Говнокод #8269 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  10. 10
  11. 11
  12. 12
  13. 13
  14. 14
  15. 15
  16. 16
  17. 17
  18. 18
  19. 19
  20. 20
  21. 21
  22. 22
# Work fine only on call with has_many association
def self.new_from_params options
  new_rule = Rule.new
  return nil if new_rule.project_id.nil?

  rule = Rule.find_by_id options[:rule_id]
  if rule.nil? or (new_rule.project.id != rule.project.id)
    rule = Rule.new
  end

  new_rule.rule_id = options[:rule_id]
  new_rule.users = (rule.users.to_a + options[:users].to_a).uniq
  new_rule.statuses = (rule.statuses.to_a + options[:statuses].to_a).uniq
  new_rule.tags = (rule.tags.to_a + options[:tags].to_a).uniq
  new_rule.grouped_by = options[:grouped_by].presence || rule.grouped_by.presence
  new_rule.sorted_by = options[:sorted_by].presence || rule.sorted_by.presence
  new_rule.search = options[:search].presence || rule.search.presence
  new_rule.period_start = options[:period_start].presence || rule.period_start.presence
  new_rule.period_end = options[:period_end].presence || rule.period_end.presence

  new_rule
 end

Запостил: emerald emerald, (Updated )

Комментарии (5) RSS

  • реально в чём колбаса? меня не прет читать эту портянку.......
    Ответить
    • Проще функцию написать, чем 8 строк дублировать
      Ответить
    • Боюсь представить что творится с кодом, если эта проверка оправдана.
      Ответить

Добавить комментарий

Семь раз отмерь — один отрежь, guest!

    А не использовать ли нам bbcode?


    8