• rb
    rb
    сам код патча разместил в app/config/initializers
    /1 ·
  • wyldrodney
    wyldrodney
    Почему костыли? Интересно на что такое наткнулись что пришлось так делать.
    /2 ·
  • rb
    rb
    @wyldrodney, 100-моделей, где в каждой user_id. Задолбались каждый раз в контролерах писать .merge(:user => current_user)
    /3 in reply to /2 ·
  • wyldrodney
    wyldrodney
    @rb, Гм.. ясно.
    /4 in reply to /3 ·
  • kaineer
    kaineer
    @rb, почему не сделать модуль и не инклудить повсеместно?
    /5 in reply to /3 ·
  • rb
    rb
    @kaineer, этот вопрос тоже обсуждался, но число моделей будет расти и можно забыть его заиклудить.... к тому же, нарушается принцип DRY.
    /6 in reply to /5 ·
  • kaineer
    kaineer
    @rb, Где это он нарушается? А на предмет — забыть проинклудить можно проверку гонять, чтобы искала строчку в моделях.
    /7 in reply to /6 ·
  • rb
    rb
    @kaineer, мы фактически должны сделать copy-paste во всех контролерах. Почему бы не добавлять это код автоматом во все модели?
    /8 in reply to /7 ·
  • rb
    rb
    @rb, *моделях
    /9 in reply to /8 ·
  • kaineer
    kaineer
    @rb, А нельзя в том же инициализаторе просто найти все модели проекта и каждому проинклудить модуль?
    /10 in reply to /8 ·
  • kaineer
    kaineer
    А ещё, кто мешает сделать вот так: gist.github.com
    /11 ·