replaced ActsAsTaggableOnSteroids by ActsAsTaggableOn

Author: rueckerl

Gemfile

@@ -10,6 +10,7 @@ gem 'rails', '~> 4.2.4'
 gem 'sass-rails'
 
 platforms :ruby do
+#  gem 'pg'
   gem 'sqlite3'
 end
 
@@ -44,6 +45,7 @@ gem 'exception_notification', '~> 2.5.2'
 gem 'omniauth'
 gem 'omniauth-google-oauth2'
 gem 'omniauth-openid'
+gem 'acts-as-taggable-on', '~> 3.5'
 
 # Bundle gems for the local environment. Make sure to
 # put test-only gems in this group so their generators

app/helpers/navigation_helper.rb

@@ -17,6 +17,6 @@ def get_tags_for_published_posts
       published_posts = Post.where.not(:published_at => nil)
       published_tag_names = published_posts.collect { |post| post.cached_tag_list.split(',') }.flatten.uniq
       published_tag_names.each { |tag| tag.strip! }
-      Tag.where(:name => published_tag_names).sort_by { |tag| tag.taggings.size }.reverse
+      ActsAsTaggableOn::Tag.where(:name => published_tag_names).sort_by { |tag| tag.taggings.size }.reverse
     end
 end

app/models/post.rb

@@ -44,8 +44,8 @@ def build_for_preview(params)
       post.generate_slug
       post.set_dates
       post.apply_filter
-      TagList.from(params[:tag_list]).each do |tag|
-        post.tags << Tag.new(:name => tag)
+      post.tag_list.each do |tag|
+        post.tags << ActsAsTaggableOn::Tag.find_or_create_with_like_by_name(tag)
       end
       post
     end
@@ -57,14 +57,11 @@ def find_recent(options = {})
       conditions = ['published_at < ?', Time.zone.now]
       limit = options[:limit] ||= DEFAULT_LIMIT
 
-      options = {
-        :order      => order,
-        :conditions => conditions,
-        :limit      => limit
-      }.merge(options)
-
       if tag
-        find_tagged_with(tag, options)
+        result = where(conditions)
+        result = result.tagged_with(tag)
+        result = result.includes(:tags) if include_tags
+        result.order(order).limit(limit)
       else
         result = where(conditions)
         result = result.includes(:tags) if include_tags
@@ -136,12 +133,4 @@ def generate_slug
     self.slug.slugorize!
   end
 
-  def tag_list=(value)
-    value = value.split(',') if value.is_a?(String)
-    value.map!{ |tag_name| Tag::filter_name(tag_name) }
-
-    # TODO: Contribute this back to acts_as_taggable_on_steroids plugin
-    value = value.join(", ") if value.respond_to?(:join)
-    super(value)
-  end
 end

app/models/stats.rb

@@ -8,6 +8,6 @@ def comment_count
   end
 
   def tag_count
-    Tag.count
+    ActsAsTaggableOn::Tag.count
   end
 end

app/models/tag.rb

@@ -1,7 +1,7 @@
 <%= form.inputs do -%>
   <%= form.input :title -%>
   <%= form.input :body, :hint => "<a href='http://redcloth.org/textile'>Textile enabled</a>. Use Ctrl+E to switch between preview and edit mode.".html_safe -%>
-  <%= form.input :tag_list, :as => 'string', :required => false, :hint => 'Comma separated: ruby, rails&hellip;'.html_safe -%>
+  <%= form.input :tag_list, :input_html => { :value => @post.tag_list.join(', ')}, :as => 'string', :required => false, :hint => 'Comma separated: ruby, rails&hellip;'.html_safe -%>
 <% end -%>
 <%= form.inputs do -%>
   <%= form.input :published_at_natural, :label => 'Published at', :as => 'string', :hint => 'Examples: now, yesterday, 1 hour from now, '.html_safe + link_to("etc".html_safe, "http://chronic.rubyforge.org/") + '. Leave blank for an unpublished draft.' -%>

app/models/tagging.rb

@@ -0,0 +1,28 @@
+class DropOldTaggingTables < ActiveRecord::Migration
+  def up
+    if ActiveRecord::Base.connection.table_exists? 'tags'
+      drop_table 'tags'
+    end
+
+    if ActiveRecord::Base.connection.table_exists? 'taggings'
+      drop_table 'taggings'
+    end
+
+    if index_exists?(:taggings, 'index_taggings_on_taggable_id_and_taggable_type')
+      remove_index :taggings, :name => 'index_taggings_on_taggable_id_and_taggable_type'
+    end
+
+    if index_exists?(:taggings, 'index_taggings_on_tag_id')
+      remove_index :taggings, :name => 'index_taggings_on_tag_id'
+    end
+
+    if index_exists?(:tags, 'index_tags_on_name')
+      remove_index :tags, :name => 'index_tags_on_name'
+    end
+
+  end
+
+  def down
+    raise ActiveRecord::IrreversibleMigration
+  end
+end

app/views/admin/posts/_form.html.erb

@@ -0,0 +1,32 @@
+# This migration comes from acts_as_taggable_on_engine (originally 1)
+class ActsAsTaggableOnMigration < ActiveRecord::Migration
+  def self.up
+    create_table :tags do |t|
+      t.string :name
+    end
+
+    create_table :taggings do |t|
+      t.references :tag
+
+      # You should make sure that the column created is
+      # long enough to store the required class names.
+      t.references :taggable, polymorphic: true
+      t.references :tagger, polymorphic: true
+      t.string :taggable_type, null: false
+
+      # Limit is created to prevent MySQL error on index
+      # length for MyISAM table type: http://bit.ly/vgW2Ql
+      t.string :context, limit: 128
+
+      t.datetime :created_at
+    end
+
+    add_index :taggings, :tag_id
+    add_index :taggings, [:taggable_id, :taggable_type, :context]
+  end
+
+  def self.down
+    drop_table :taggings
+    drop_table :tags
+  end
+end

config/initializers/acts_as_taggable_on_steroids.rb

@@ -0,0 +1,20 @@
+# This migration comes from acts_as_taggable_on_engine (originally 2)
+class AddMissingUniqueIndices < ActiveRecord::Migration
+  def self.up
+    add_index :tags, :name, unique: true
+
+    remove_index :taggings, :tag_id
+    remove_index :taggings, [:taggable_id, :taggable_type, :context]
+    add_index :taggings,
+              [:tag_id, :taggable_id, :taggable_type, :context, :tagger_id, :tagger_type],
+              unique: true, name: 'taggings_idx'
+  end
+
+  def self.down
+    remove_index :tags, :name
+
+    remove_index :taggings, name: 'taggings_idx'
+    add_index :taggings, :tag_id
+    add_index :taggings, [:taggable_id, :taggable_type, :context]
+  end
+end

db/migrate/20151218121115_drop_old_tagging_tables.rb

@@ -0,0 +1,15 @@
+# This migration comes from acts_as_taggable_on_engine (originally 3)
+class AddTaggingsCounterCacheToTags < ActiveRecord::Migration
+  def self.up
+    add_column :tags, :taggings_count, :integer, default: 0
+
+    ActsAsTaggableOn::Tag.reset_column_information
+    ActsAsTaggableOn::Tag.find_each do |tag|
+      ActsAsTaggableOn::Tag.reset_counters(tag.id, :taggings)
+    end
+  end
+
+  def self.down
+    remove_column :tags, :taggings_count
+  end
+end

db/migrate/20151218121130_acts_as_taggable_on_migration.acts_as_taggable_on_engine.rb

@@ -0,0 +1,10 @@
+# This migration comes from acts_as_taggable_on_engine (originally 4)
+class AddMissingTaggableIndex < ActiveRecord::Migration
+  def self.up
+    add_index :taggings, [:taggable_id, :taggable_type, :context]
+  end
+
+  def self.down
+    remove_index :taggings, [:taggable_id, :taggable_type, :context]
+  end
+end

db/migrate/20151218121131_add_missing_unique_indices.acts_as_taggable_on_engine.rb

@@ -0,0 +1,10 @@
+# This migration comes from acts_as_taggable_on_engine (originally 5)
+# This migration is added to circumvent issue #623 and have special characters
+# work properly
+class ChangeCollationForTagNames < ActiveRecord::Migration
+  def up
+    if ActsAsTaggableOn::Utils.using_mysql?
+      execute("ALTER TABLE tags MODIFY name varchar(255) CHARACTER SET utf8 COLLATE utf8_bin;")
+    end
+  end
+end

db/migrate/20151218121132_add_taggings_counter_cache_to_tags.acts_as_taggable_on_engine.rb

@@ -11,7 +11,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 20150414113518) do
+ActiveRecord::Schema.define(version: 20151218121134) do
 
   create_table "comments", force: true do |t|
     t.integer  "post_id",      null: false
@@ -80,18 +80,22 @@
   create_table "taggings", force: true do |t|
     t.integer  "tag_id"
     t.integer  "taggable_id"
+    t.string   "taggable_type",             null: false
+    t.integer  "tagger_id"
+    t.string   "tagger_type"
+    t.string   "context",       limit: 128
     t.datetime "created_at"
   end
 
-  add_index "taggings", ["tag_id"], name: "index_taggings_on_tag_id"
-  add_index "taggings", ["taggable_id"], name: "index_taggings_on_taggable_id_and_taggable_type"
+  add_index "taggings", ["tag_id", "taggable_id", "taggable_type", "context", "tagger_id", "tagger_type"], name: "taggings_idx", unique: true, using: :btree
+  add_index "taggings", ["taggable_id", "taggable_type", "context"], name: "index_taggings_on_taggable_id_and_taggable_type_and_context", using: :btree
 
   create_table "tags", force: true do |t|
     t.string  "name"
-    t.integer "taggings_count", default: 0, null: false
+    t.integer "taggings_count", default: 0
   end
 
-  add_index "tags", ["name"], name: "index_tags_on_name"
+  add_index "tags", ["name"], name: "index_tags_on_name", unique: true, using: :btree
 
   create_table "undo_items", force: true do |t|
     t.string   "type",       null: false