Как правильно сделать user_helper?

Question

[Adil1]

Хочу сделать хелпер для аватарок, но чет не получается
В общем что имеем:

module UsersHelper
	def avatar_for(user, options = { size: 50 })
		if user.nil? or user.avatar.nil?
			image_tag('/uploads/1.png', :size => 50)
			size = options[:size]
		else
			image_tag(user.avatar.thumb, :size => 52)
			size = options[:size]
		end
	end
end

во вьюху вставляю
<%= avatar_for @user %>

Но не пашет, использую carierwave и rmagick для обрезки.
Да, кстати, может кто сталкивался с crop? Что посоветуете?

Answer 1

[jj_killer]

Задача ясна, но выглядит это вот так:

module UsersHelper
  def avatar_for(user, options = { size: 50 })
    if user.try(:avatar).blank?
      image_tag('/uploads/1.png', :size => options[:size])
    else
      image_tag(user.avatar.thumb, :size => options[:size])
    end
  end
end

Comments

[Adil1]

Спасибо, работает только если у юзера есть запись в базе с картинкой, если нету ничего не показывает, хотя должен, есть идеи? а то уже запарился искать

[jj_killer]

Ну может у вас user.avatar не nil, а пустая строка. Потому как если nil, то обязательно сработает.

[Adil1]

а с пустой строкой user.avatar.to_f == []? потому как в базе просто пустое поле

[jj_killer]

нет, есть удобный rails хелпер метод, blank?. Обрабатывает и nil, и пустую строку. В вашем случае выглядит так: user.avatar.blank?

[Adil1]

@jj_killer а все понял blank?

[jj_killer]

Только вот еще один трабл, если вы передатите nil'ового user'а в метод хелпера avatar_for, то на user.avatar.nil? вызвется исключение, мол NoMethodError: undefined method `avatar' for nil:NilClass. Так что лучше обернуть user.avatar.blank? в try. Вот так: user.try(:avatar).blank?

[jj_killer]

Хотя нет, не вызовется. Там OR, так что до него не дойдет, так что можно не оборачивать в try.

[jj_killer]

Хотя, можно вообще убрать user.nil? и оставить только: user.try(:avatar).blank? Так будет даже лучше.

[Adil1]

@jj_killer Спасибо!

Answer 2

[jj_killer]

Неясно что вы делаете size = options[:size] тут. Вам сайз надо передать image_tag, а по факту ваш хелпер просто выводит options[:size] вместо image_tag.

Comments

[Adil1]

задача следующая, если у юзера есть аватар то показать его, если нет то показать дефолтную картинку, все работает если у юзера в базе в колонке avatar есть ссылка на изображение, а если нету то он ничего не показывает

[shumi]

Ну да, если у юзера есть аватар - ваш метод возвращает size = options[:size]