@@ -3,4 +3,6 @@ %label{:for => "page_meta_tags"} Tags %td.field - = f.text_field :meta_tags, :value => @page.tag_list, :class => 'textbox', :maxlength => 160 \ No newline at end of file + = f.text_field :meta_tags, :value => @page.tag_list, :class => 'textbox', :maxlength => 160 + - unless model.errors.on_base.nil? + %span.error= model.errors.on_base \ No newline at end of file app/views/admin/pages/_tag_field.html.haml @@ -1,5 +1,10 @@ TaggingMethods = Proc.new do + def valid_with_tags? + return (self.errors.empty? && valid_without_tags?) + end + alias_method_chain :valid?, :tags + def tag_with tags self.save if self.new_record? # just skip the whole method if the tags string hasn't changed @@ -12,7 +17,9 @@ TaggingMethods = Proc.new do begin tag = MetaTag.find_or_initialize_by_name(tag.strip.squeeze(" ")) meta_tags << tag unless meta_tags.include?(tag) - rescue ActiveRecord::StatementInvalid => e + rescue ActiveRecord::RecordInvalid => e + errors.add_to_base("Tags can not contain special characters") + rescue ActiveRecord::StatementInvalid => e # With SQLite3 - a duplicate tagging will result in the following message: # SQLite3::SQLException: SQL logic error or missing database: INSERT INTO taggings ("meta_tag_id", "taggable_type", "taggable_id") VALUES(11, 'Page', 74) # raise unless e.to_s =~ /duplicate/i lib/tagging_methods.rb 0efa0180ff35909381f17c36505524f2f5360ce6 Benny Degezelle benny@gorilla-webdesign.be http://github.com/jomz/radiant-tags-extension/commit/82cb81fa23f6c589e69ff84e8b7a9af170286949 82cb81fa23f6c589e69ff84e8b7a9af170286949 2009-09-22T13:13:07-07:00 2009-09-22T13:13:07-07:00 display error msg on meta_tags field when tags were invalid 073d0f495193cedf81496611fc5fee476d9b4715 Benny Degezelle benny@gorilla-webdesign.be