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

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
  23. 23
  24. 24
  25. 25
  26. 26
  27. 27
  28. 28
  29. 29
  30. 30
  31. 31
  32. 32
  33. 33
  34. 34
def self.process(site, object = nil, &block)
    dont_unblock = false
    block_res = true

    site.lock!
    if site.assets_blocked?
      dont_unblock = true
      raise Asset::BlockedException
    end
    site.assets_blocked = true
    site.save!

    block_res = yield if block_given?

    return false unless block_res
    return object.save if object

    true
  ensure
    site.update_attribute(:assets_blocked, false) unless dont_unblock
  end

  # во всех местах, где с ассетами делалось хоть что-нибудь, копипастился такой кусок:

      result =
      begin
        @asset = @site.assets.new
        @asset.process do
          @asset.attached = params[:file]
          @asset.folder_path = params[:path]
        end
      rescue Asset::BlockedException
        render json: { errors: [I18n.t("layout.assets.errors.assets_blocked")] } and return
      end

По мотивам http://govnokod.ru/13027: транзакции изобретены не нами, давайте сделаем их сами!

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

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

  • - Как у вас всё запутано! Чего только не бывает на свете... - философски заметил Сергей, спасая меж тем продукты от вражеского нашествия.
    Ответить

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

Я, guest, находясь в здравом уме и твердой памяти, торжественно заявляю:

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


    8