Fix Bug 2274, empty tag

This commit is contained in:
Johannes Hackel 2011-11-22 12:16:00 +01:00
parent 7e4028626f
commit eb65074360
3 changed files with 16 additions and 6 deletions

View file

@ -22,6 +22,9 @@ class PeopleController < ApplicationController
if (params[:q][0] == 35 || params[:q][0] == '#') && params[:q].length > 1
redirect_to tag_path(:name => params[:q].gsub(/[#\.]/, ''), :q => params[:q])
return
elsif (params[:q][0] == 35 || params[:q][0] == '#') && params[:q].length == 1
flash[:error] = I18n.t('tags.show.none', :name => params[:q])
redirect_to multi_path()
end
limit = params[:limit] ? params[:limit].to_i : 15

View file

@ -15,13 +15,18 @@ class TagFollowingsController < ApplicationController
# POST /tag_followings.xml
def create
name_normalized = ActsAsTaggableOn::Tag.normalize(params['name'])
@tag = ActsAsTaggableOn::Tag.find_or_create_by_name(name_normalized)
@tag_following = current_user.tag_followings.new(:tag_id => @tag.id)
if @tag_following.save
flash[:notice] = I18n.t('tag_followings.create.success', :name => name_normalized)
if name_normalized.nil? || name_normalized.empty?
flash[:error] = I18n.t('tag_followings.create.none', :name => name_normalized)
else
flash[:error] = I18n.t('tag_followings.create.failure', :name => name_normalized)
@tag = ActsAsTaggableOn::Tag.find_or_create_by_name(name_normalized)
@tag_following = current_user.tag_followings.new(:tag_id => @tag.id)
if @tag_following.save
flash[:notice] = I18n.t('tag_followings.create.success', :name => name_normalized)
else
flash[:error] = I18n.t('tag_followings.create.failure', :name => name_normalized)
end
end
redirect_to :back

View file

@ -846,10 +846,12 @@ en:
following: "Following #%{tag}"
stop_following: "Stop Following #%{tag}"
followed_by: 'followed by'
none: "The empty tag don't exists!"
tag_followings:
create:
success: "Hooray! You're now following #%{name}."
failure: "Failed to follow #%{name}. Are you already following it?"
none: "You can't follow the empty tag!"
destroy:
success: "Alas! You aren't following #%{name} anymore."
failure: "Failed to stop following #%{name}. Maybe you already stopped following it?"