@rubylang
v26.08.2016
11:58:09
11:58:09
s26.08.2016
11:58:13
11:58:13
а зачем ее херачить во все классы?
Danila26.08.2016
11:58:20
11:58:20
думаю, там одна строка
s26.08.2016
11:58:32
11:58:32
и что?
GoogleDanila26.08.2016
11:58:32
11:58:32
судя по названию
s26.08.2016
11:58:47
11:58:47
сделать HasDeletableThumb, заинклюдить его куда нужно
не думаю, что это добавит неявности
Danila26.08.2016
11:59:01
11:59:01
бе
s26.08.2016
11:59:04
11:59:04
на мой взгляд – наоборот
Антон26.08.2016
11:59:34
11:59:34
неявность - плехо
если мы наделяем класс способностью, в чем неявность?
если мы наделяем класс способностью через наделятор способностей, в чем неявность?s26.08.2016
12:00:05
12:00:05
:+1:
Danila26.08.2016
12:00:40
12:00:40
чувак, покажи свой метод
s26.08.2016
12:01:03
12:01:03
суть не в методе или его сложности
Антон26.08.2016
12:01:05
12:01:05
интрига :)
s26.08.2016
12:01:05
12:01:05
а подходе
Nork26.08.2016
12:01:29
12:01:29
ладно только от смеха не помрите
Googlev26.08.2016
12:02:17
12:02:17
если мы наделяем класс способностью, в чем неявность?
если мы наделяем класс способностью через наделятор способностей, в чем неявность?
если нет неявности - хорошоа неявность - плохо
Антон26.08.2016
12:02:42
12:02:42
Nork26.08.2016
12:04:13
12:04:13
https://gist.github.com/Syntaxys-dll/4493e9f7901815a39ea7e96c9f4ac35b
v26.08.2016
12:04:35
12:04:35
мне и поиск записи по id в before_Action не нравится
Антон26.08.2016
12:04:53
12:04:53
если ты не видишь, от чего оно работать начинает
чтобы так случилось нужно еще постараться, подумать крепко, выдумать магию
можно to_s переопределить, это не значит что руби неявный вообще языкv26.08.2016
12:05:12
12:05:12
я в применении к рельсам
Антон26.08.2016
12:08:11
12:08:11
https://gist.github.com/Syntaxys-dll/4493e9f7901815a39ea7e96c9f4ac35b
забить
работает? не трогай
не нравится? - запилить ThumbRemover.new(resource).processАнтон26.08.2016
12:08:50
12:08:50
а еще вариант
метод ресурса remove_thumb
пусть ресурс сам отвечает за свою консистентность
Danila26.08.2016
12:13:17
12:13:17
самый норм вариант
Антон26.08.2016
12:14:05
12:14:05
:+1:
Nork26.08.2016
12:30:02
12:30:02
самый норм вариант
в моем случае carrierwave, если я не ошибаюсь он не делает отдельной модели, thumb_picture заполняется из метода контроллера который делает превью изображения из изображения загруженного carrierwave. ?Googles26.08.2016
12:31:36
12:31:36
метод контроллера, который делает превью?
Nork26.08.2016
12:32:07
12:32:07
ага
s26.08.2016
12:32:13
12:32:13
а зачем, превью – по идее это просто одна из версий в аплоадере
Nork26.08.2016
12:34:04
12:34:04
Кейс такой: грузится изображение -> появляется кнопка "сделать превью" -> на изображении выбирается область -> жмем кнопку -> создается превью
v26.08.2016
12:34:16
12:34:16
ээ
Nork26.08.2016
12:35:34
12:35:34
Превью не всмысле уменьшенная версия а превью для новости в разделе статей например, т.е. выбранный кусок изображения который нужно показывать в ленте и нужно выбирать вручную
Антон26.08.2016
12:36:10
12:36:10
c9 для pair programming использовал ктон-ибудь? есть успехи?
v26.08.2016
12:37:46
12:37:46
да понятно, что кроп
но там вроде можно прямо карриервэйв дернуть с нужными размерами
s26.08.2016
12:39:45
12:39:45
да, если файл обработан аплоадером, то ты можешь просто вернуть picture.url(:thumb)
Nork26.08.2016
12:56:24
12:56:24
да, если файл обработан аплоадером, то ты можешь просто вернуть picture.url(:thumb)
разобрался, спасибо, а то я чет минимэджиком в кнотроллере резал)s26.08.2016
12:56:33
12:56:33
))))
ojab26.08.2016
13:00:15
13:00:15
https://gist.github.com/Syntaxys-dll/4493e9f7901815a39ea7e96c9f4ac35b
.camelize.constantize на user input это *cough* слегка *cough* небезопасноojab26.08.2016
13:01:24
13:01:24
ну и смысла так развёрнуто писать нет, resource.update(thumb_picture: nil)
Антон26.08.2016
13:02:13
13:02:13
да, только update! или if update
Nork26.08.2016
13:13:46
13:13:46
.camelize.constantize на user input это *cough* слегка *cough* небезопасно
Это совсем руки оторвать подход и так не делать или по другому можно как то обернуть все это дело? const_get?ojab26.08.2016
13:16:06
13:16:06
controller_name.classify.constantize, как вариант
использовать любой нефильтрованный user input в любых ситуациях — плохой подход и надо этого избегать, если возможно
если с .find(params[:id] понятно, какая ошибка может произойти и какие будут последствия, то с .camelize.constantize этих последствий может быть гораздо больше
Danila26.08.2016
13:26:46
13:26:46
ну вот, а вы говорили не надо метод смотреть
Nikolay26.08.2016
14:22:11
14:22:11
Подскажите пожалуйста: есть админка, длня нее сделан отдельный контроллер, в админке крады для news и articles. У news и articles одинаковый метод в контроллере del_thumb_picture. Я могу его вынести в контроллер админки и передавать в него параметрами название модели и ид чтобы сделать этот метод общим или это как то по другому правильно делается?
я б сказал, что выделять всякие /admin/someshit вообще плохая идея. по хорому админ должен видеть тоже, что и юзер, разве что за исключением количества тех же постов и прочее. у себя я это решил Devise + scopingGoogleАнтон26.08.2016
14:23:11
14:23:11
не должен
v26.08.2016
14:23:17
14:23:17
загнул
Антон26.08.2016
14:23:17
14:23:17
это вот вброс щас был?
Nork26.08.2016
14:23:18
14:23:18
v26.08.2016
14:23:35
14:23:35
админ должен видеть то же, что и юзер, только если захочет
Nikolay26.08.2016
14:23:52
14:23:52
Антон26.08.2016
14:24:06
14:24:06
потому что отдельные блекджеки
v26.08.2016
14:24:12
14:24:12
ну епт
AdminERROR: S client not available
Nikolay26.08.2016
14:24:23
14:24:23
не отдельные жеж
Антон26.08.2016
14:24:47
14:24:47
Nikolay26.08.2016
14:24:50
14:24:50
хотя бы чтобы посмотреть количество постов
ЭЭ ШТО? какое кличество постов? для этого городить /admin/posts/count?бред
Антон26.08.2016
14:25:28
14:25:28
вот крайность: у меня пользователь видит лендинг и 3 страницы
бизнесом управляет контрол на ангуляре и 35 моделей в коде
v26.08.2016
14:25:29
14:25:29
admin/users, ващета
Nikolay26.08.2016
14:25:49
14:25:49
ахаха
v26.08.2016
14:25:58
14:25:58
а там ты смотришь и posts, и прочие траливали
Nikolay26.08.2016
14:28:45
14:28:45
а что мне как админу мешает эти /posts смотреть так же как и все
GoogleNikolay26.08.2016
14:29:03
14:29:03
только вместо posts.where(user: me) posts.all?
v26.08.2016
14:30:53
14:30:53
а что мне как админу мешает эти /posts смотреть так же как и все
то, что у тебя в общем контроллере if заведетсяи if тебе придется втыкать везде
ну или похожую логику реализовывать
Nikolay26.08.2016
14:32:14
14:32:14
я ж говорю - скоупі девайса
v26.08.2016
14:32:19
14:32:19
это если не учитывать того, что тебе будут нужны данные, которые обычный юзверь не видит - еще логики в контроллер, еще логики во вьюхи
Nikolay26.08.2016
14:32:54
14:32:54
вот это уже более весомый аргумент
а напомните мне плз как сделать uniq в массиве хешей по конкретному полю...
ojab26.08.2016
14:45:06
14:45:06
.uniq { |h| h[:name] }
Nikolay26.08.2016
15:18:46
15:18:46
я правильно понял, что к любой фиговине можно прицепить { } ?
ну, кроме nil/true/false
Mikhail26.08.2016
16:04:44
16:04:44
использовать любой нефильтрованный user input в любых ситуациях — плохой подход и надо этого избегать, если возможно
Для таких штук можно сделать какой нибудь констант LIST_CONST = %W(model model1 model2 model3) и проверять наличие парама в этом листеNork26.08.2016
16:15:37
16:15:37
Шифровать в erb название модели, прибавлять шум, разбирать в концерне ) Мне кажется либо я что то не правильно делаю, либо должен быть правильный способ. Но тут момент такой что данная тема сделана только для админки и проверяется роль пользователя.
Nikolay26.08.2016
19:28:00
19:28:00
Nork26.08.2016
21:40:06
21:40:06
Подскажите пожалуйста, можно ли вместо textarea в рельсах использовать div какнить красиво без js?
I26.08.2016
22:20:16
22:20:16
contenteditable=true, но совсем без js не выйдет сохранить
Открыть в Telegram