1. Ruby / Говнокод #8269

    −99

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    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, 21 Октября 2011

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

    • ava airrussia 21.10.2011 22:05 # +3

      реально в чём колбаса? меня не прет читать эту портянку.......
      Ответить

      • ava guest 22.10.2011 00:17 # 0

        Проще функцию написать, чем 8 строк дублировать
        Ответить

    • ava Nook 20.11.2011 00:51 # 0

      Первая проверка на project_id шикарна.
      Ответить

      • ava akostrikov 22.07.2013 11:21 # 0

        Боюсь представить что творится с кодом, если эта проверка оправдана.
        Ответить

    • ava bormand 22.07.2013 11:47 # 0

      > .nil?
      Эм. В руби можно вызывать методы у nil'а?
      Ответить

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