%)

>sanitize(self.text, :tags=>[]).length == 0 лучше .empty?

енто ты об чом?

о том что при ковераге 100% явная ошибка в коде, значит коверага не 100%

>sanitize(self.text, :tags=>[]).length == 0 лучше .empty?

не. там идея в том что считается содержимое текста, без тегов.

ибо могут быть просто теги в редакторе гнабитые, но видимого текста не будет

и длина уже будет не ноль

.blank?

"<b></b>" = это не бланк

бланк - чистый емпти - пусттой

и не empty

болд

бля

только это не пользуют

я про пустые теги без текста внутри

имеется в виду .length == 0, а не вся конструкция

с ними поле будет не пустым в любом случае

так бы и сказали

но вообще, кто там доколебался до comment.user = currenet_user

как мне еще связать текущего залогиненного юзера с комеентом который он насрал к посту?

>true if (!user.nil? && user.Role.id >= 1) true if тут совершенно лишнее

точнее камент привязать к юзеру

если не так

>true if (!user.nil? && user.Role.id >= 1) true if тут совершенно лишнее

угу, знаю.

@TurnaviotovNikolay а я will_paginate перестал юзать - он же не обновляется и бутстрап только старый поддерживает

а что виестое will_poaginate?

*вместо

а что виестое will_poaginate?

kaminari

а насчет ассоциирования комента с юзером то что не так?

или про comment.update(user: current_user)?

comment.User - User с большой буквы

там в моделях косяки, я везде капсил иза этого

с одной стороны - надоть переделать бы

не вижу связи - но уже интересно

а то кодец попахивает

не вижу связи - но уже интересно

связи кого с чем и что интересно?

ты про мой супербложик7:)

косяки в моделях -> надо капсить

КРИВЫЕ МОДЕЛИ БЛДЖАД!

а, я тогда был зеленый чайник, да

хм, хоть бери грохни да переделай красиво

а то чот страшно такоэ показывать людям )

-- дорогая, почему моя бутылка вискаря на половину пуста? -- потому что ты пессимист

лучше не грохать, а фиксить недочёты

лучше не подстраиваться под хуево написанный код, а рефакторить и рерайтить

лучше делать то, что наиболее эффективно в данный момент

и положить хер на будущее, ага

Буддисты в чяти

Буддистов на котлетки!

лучше делать то, что наиболее эффективно в данный момент

доо.. больше говнокомитов в 35+(С) жару )))

и положить хер на будущее, ага

смотря чье будущее в данный момент важнее

в данный момент эффективным может быть как грохать так и рефакторить при чем тут хер?

с высоты активного фигачинга панели я понял чт о полгода назад был полным идиотом чайником. но на тот момент эта версия бложика была вполне рабочим прототипом которым я гордился что смог

с высоты через полгода - да, код кхм, странен

кстати да, переделать сейчас бложик = вспомнить как минимум нестед моделы

Аналогичные впечатления от викторинки

Аналогичные впечатления от викторинки

чо, после викторинки предыдущий проект "ойблин, это не я, афтар не я"?:)

Да ладно, главное напильник подрачовее :)

мама за тебя будет private ставить?

https://github.com/foxmuldercp/blogapp/blob/master/app/controllers/application_controller.rb#L10

фильтры для кого придумали? code duplication https://github.com/foxmuldercp/blogapp/blob/master/app/controllers/articles_controller.rb

@article.User это вообще что

https://github.com/foxmuldercp/blogapp/blob/master/app/controllers/articles_controller.rb#L62 имодзи в тайтле и пользователи венды будут рады клёвому имени файла

уууу :)

с файлом да, спасибо за мысль

рубокопа нет, рика нет

никого нет

я прошлой зимой про рубокопа еще не знал

ERROR: S client not available

а чято такое рик вот впервые слышу

он какой-то бесполезный, судя по моему опыту

рик?

ага

https://github.com/troessner/reek

у меня рубокоп на дефолтный скаффолденый рельсокод выдал под 600 строк сообщений

у меня рубокоп на дефолтный скаффолденый рельсокод выдал под 600 строк сообщений

там есть ключик --rails

это ему поможет генерировать ещё больше строк

105 files inspected, 860 offenses detected

scaffold я тоже не понимаю зачем юзать, кстати

105 files inspected, 847 offenses detected без --рельс

scaffold я тоже не понимаю зачем юзать, кстати

мне он показался удобным накидывать основу под модель, а дальше уже ее в хвост и в гриву

ну для этого rails g model есть

rubocop | grep 'Line is too long' | wc -l 197

остальной бойлерплейт нафиг не нужен, по-моему

блядь, да откуда этот ебучий линукс вставляет эту ебучую фразу

не, мне больше вьюшки и контроллер интересы

grep | wc -l -> grep -c

но вот я сейчас взял у сеья в панельке, причесал плюс минус одну модель

rubocop | grep 'Line is too long' | wc -l 197

это надо уже давно в рельсу замержить

там с 80 line всё плохо

где-то соблюдают, где-то нет

нигде не соблюдают

и патчи с такими фиксами заворачивают

угу

меня это тоже бесит

у меня на 1280+ 24 экране - 80 символов - это с треть экрана

и патчи с такими фиксами заворачивают

надо патризанскую войну

80 символов это для тех кто выходит из терминала

поправил багу мелкую, поправил отступы ;)

и правит текст с помощью вима

хм. я вот другое думаю