typo 3.99.0

data/db/migrate/024_cleanup_contents.rb

@@ -0,0 +1,54 @@
+# BareMigration doesn't handle inheritance yet.  Therefore, we need
+# to mimic it manually using conditions on find().  Since this script
+# doesn't need to set type or id, there's no need to move either of
+# the protected fields out of the way like we do in #20-23.
+
+# BareMigration ALSO doesn't handle associations... yet.
+# For now, that means your keywords won't be automatically converted
+# to tags.  That might not be a bad thing...
+
+
+class Bare24Article < ActiveRecord::Base
+  include BareMigration
+  include TypoGuid
+
+  set_table_name :contents
+  has_and_belongs_to_many :tags,
+      :class_name => 'Bare24Tag', :foreign_key => 'article_id',
+      :join_table => 'articles_tags', :association_foreign_key => 'tag_id'
+
+  def keywords_to_tags
+    Bare24Article.transaction do
+      tags.clear
+      keywords.to_s.split.uniq.each do |tagword|
+        tags << Bare24Tag.get(tagword)
+      end
+    end
+  end
+end
+
+class Bare24Tag < ActiveRecord::Base
+  include BareMigration
+  has_and_belongs_to_many :articles,
+      :class_name => 'Bare24Article', :foreign_key => 'tag_id',
+      :join_table => 'articles_tags', :association_foreign_key => 'articles_id'
+end
+
+class CleanupContents < ActiveRecord::Migration
+  def self.up
+    STDERR.puts "Updating all articles"
+    # This is needed when migrating from 2.5.x, because we skip GUID
+    # generation and tagging during earlier migrations.
+    Bare24Article.transaction do
+      Bare24Article.find(:all, :conditions => "type = 'Article'").each do |a|
+        a.create_guid
+#       a.keywords_to_tags      (not needed?)
+        a.save!
+      end
+    end
+  end
+
+  def self.down
+    # nothing
+  end
+end

data/db/migrate/025_add_itunes_metadata.rb

@@ -0,0 +1,29 @@
+class Bare25Resource < ActiveRecord::Base
+  include BareMigration
+end
+
+class AddItunesMetadata < ActiveRecord::Migration
+  def self.up
+    STDERR.puts "Adding podcast metadata fields"
+    modify_tables_and_update([:add_column, :resources, :itunes_metadata, :boolean],
+                             [:add_column, :resources, :itunes_author, :string],
+                             [:add_column, :resources, :itunes_subtitle, :string],
+                             [:add_column, :resources, :itunes_duration, :integer],
+                             [:add_column, :resources, :itunes_summary, :text],
+                             [:add_column, :resources, :itunes_keywords, :string],
+                             [:add_column, :resources, :itunes_category, :string],
+                             [:add_column, :resources, :itunes_explicit, :boolean])
+  end
+
+  def self.down
+    STDERR.puts "Removing podcast metadata fields"
+    modify_tables_and_update([:remove_column, :resources, :itunes_metadata, :boolean],
+                             [:remove_column, :resources, :itunes_author, :string],
+                             [:remove_column, :resources, :itunes_subtitle, :string],
+                             [:remove_column, :resources, :itunes_duration, :integer],
+                             [:remove_column, :resources, :itunes_summary, :text],
+                             [:remove_column, :resources, :itunes_keywords, :string],
+                             [:remove_column, :resources, :itunes_category, :string],
+                             [:remove_column, :resources, :itunes_explicit, :boolean])
+  end
+end

data/db/migrate/026_add_redirect_table.rb

@@ -0,0 +1,13 @@
+class AddRedirectTable < ActiveRecord::Migration
+  def self.up
+    STDERR.puts "Adding Redirect Table"
+    create_table :redirects do |t|
+      t.column :from_path, :string
+      t.column :to_path, :string
+    end
+  end
+
+  def self.down
+    drop_table :redirects
+  end
+end

data/db/migrate/027_set_comment_published_flag.rb

@@ -0,0 +1,19 @@
+class Bare27Content < ActiveRecord::Base
+  include BareMigration
+  # See #24 for a description of how we have to manually handle STI
+end
+
+class SetCommentPublishedFlag < ActiveRecord::Migration
+  def self.up
+    STDERR.puts "Setting published flag on each comment"
+    Bare27Content.transaction do
+      Bare27Content.find(:all, :conditions => "type = 'Comment'").each do |c|
+        c.published = true
+        c.save!
+      end
+    end
+  end
+
+  def self.down
+  end
+end

data/db/migrate/028_rename_redirect_to.rb

@@ -0,0 +1,12 @@
+class RenameRedirectTo < ActiveRecord::Migration
+  def self.up
+    # The original version of the redirects table used 'to' as a column name
+    # Postgres is okay with that, but not mysql.
+    rename_column :redirects, :to, :to_path rescue nil
+  end
+
+  def self.down
+    # don't rename column back to broken name.
+    # there's little chance this column will be used before now anyway.
+  end
+end

data/db/migrate/029_add_user_notification.rb

@@ -0,0 +1,29 @@
+class Bare29User < ActiveRecord::Base
+  include BareMigration
+end
+
+class AddUserNotification < ActiveRecord::Migration
+  def self.up
+    modify_tables_and_update([:add_column, Bare29User, :notify_via_email,         :boolean],
+                             [:add_column, Bare29User, :notify_on_new_articles,   :boolean],
+                             [:add_column, Bare29User, :notify_on_comments,       :boolean],
+                             [:add_column, Bare29User, :notify_watch_my_articles, :boolean]) do |u|
+      # Definitions:
+      #  notify_via_email: use email to deliver notifications
+      #  notify_on_new_articles: send a notification message (email, etc) when new articles added.
+      #  notify_on_comments: send a notification message when new comments are added to watched articles.
+      #  notify_watch_my_articles: tell the notifiation system to watch my articles.
+      u.notify_via_email = true
+      u.notify_on_new_articles = false
+      u.notify_on_comments = true
+      u.notify_watch_my_articles = true
+    end
+  end
+
+  def self.down
+    modify_tables_and_update([:remove_column, Bare29User, :notify_via_email,         :boolean],
+                             [:remove_column, Bare29User, :notify_on_new_articles,   :boolean],
+                             [:remove_column, Bare29User, :notify_on_comments,       :boolean],
+                             [:remove_column, Bare29User, :notify_watch_my_articles, :boolean])
+  end
+end

data/db/migrate/030_index_sessions.rb

@@ -0,0 +1,9 @@
+class IndexSessions < ActiveRecord::Migration
+  def self.up
+    add_index :sessions, :sessid
+  end
+
+  def self.down
+    remove_index :sessions, :sessid
+  end
+end

data/db/migrate/031_add_notifications_table.rb

@@ -0,0 +1,14 @@
+class AddNotificationsTable < ActiveRecord::Migration
+  def self.up
+    create_table :notifications, :id => false do |t|
+      t.column :notify_user_id, :integer
+      t.column :notify_content_id, :integer
+      t.column :created_at, :datetime
+      t.column :updated_at, :datetime
+    end
+  end
+
+  def self.down
+    drop_table :notifications
+  end
+end

data/db/migrate/032_add_jabber_notification.rb

@@ -0,0 +1,11 @@
+class AddJabberNotification < ActiveRecord::Migration
+  def self.up
+    add_column :users, :notify_via_jabber, :boolean
+    add_column :users, :jabber, :string
+  end
+
+  def self.down
+    remove_column :users, :notify_via_jabber
+    remove_column :users, :jabber
+  end
+end

data/db/migrate/033_add_count_caching.rb

@@ -0,0 +1,34 @@
+class Bare33Content < ActiveRecord::Base
+  include BareMigration
+
+  def count_children_of_type(type)
+    self.class.find(:all,
+                    :conditions => ["article_id = ? and type = ?",
+                                               self.id,       type ]).size
+  end
+
+  def correct_counts
+    self.comments_count   = self.count_children_of_type('Comment')
+    self.trackbacks_count = self.count_children_of_type('Trackback')
+  end
+end
+
+
+class AddCountCaching < ActiveRecord::Migration
+  def self.up
+    STDERR.puts "Adding comments_count, trackbacks_count"
+    modify_tables_and_update([:add_column, Bare33Content, :comments_count, :integer],
+                             [:add_column, Bare33Content, :trackbacks_count, :integer]) do |a|
+      if not $schema_generator
+        a.correct_counts
+      end
+    end
+  end
+
+  def self.down
+    STDERR.puts "Removing counts columns"
+    remove_column :contents, :comments_count
+    remove_column :contents, :trackbacks_count
+  end
+end
+

data/db/migrate/034_boolify_published.rb

@@ -0,0 +1,36 @@
+class Bare34Content < ActiveRecord::Base
+  include BareMigration
+end
+
+class BoolifyPublished < ActiveRecord::Migration
+  def self.up
+    STDERR.puts "Boolifying contents.published"
+    modify_tables_and_update([:rename_column, Bare34Content, :published, :old_pub],
+                             [:add_column,    Bare34Content, :published, :boolean, { :default => true }]) do |c|
+      unless $schema_generator
+        if c.old_pub.nil?
+          c.published = true
+        else
+          c.published = (!c.old_pub.to_i.zero? ? true : false)
+        end
+      end
+    end
+    remove_column :contents, :old_pub
+  end
+
+  def self.down
+    STDERR.puts "Un-Boolifying contents.published"
+    modify_tables_and_update([:rename_column, Bare34Content, :published, :old_pub],
+                             [:add_column,    Bare34Content, :published, :integer]) do |c|
+      unless $schema_generator
+        say "Old published: #{c.old_pub}"
+        if c.old_pub.nil?
+          c.published = 1
+        else
+          c.published = c.old_pub ? 1 : 0
+        end
+      end
+    end
+    remove_column :contents, :old_pub
+  end
+end

data/db/migrate/035_boolify_content_allow_foo.rb

@@ -0,0 +1,36 @@
+class Bare35Content < ActiveRecord::Base
+  include BareMigration
+end
+
+class BoolifyContentAllowFoo < ActiveRecord::Migration
+  def self.up
+    STDERR.puts "Boolifying contents.allow_(comments|pings)"
+
+    modify_tables_and_update([:rename_column, Bare35Content, :allow_pings,    :old_ap],
+                             [:add_column,    Bare35Content, :allow_pings,    :boolean],
+                             [:rename_column, Bare35Content, :allow_comments, :old_ac],
+                             [:add_column,    Bare35Content, :allow_comments, :boolean]) do |c|
+      unless $schema_generator
+        c.allow_pings    = !c.old_ap.to_i.zero? ? true : false unless c.old_ap.nil?
+        c.allow_comments = !c.old_ac.to_i.zero? ? true : false unless c.old_ac.nil?
+      end
+    end
+    remove_column :contents, :old_ap
+    remove_column :contents, :old_ac
+  end
+
+  def self.down
+    STDERR.puts "Un-Boolifying contents.allow_(comments|pings)"
+    modify_tables_and_update([:rename_column, Bare35Content, :allow_pings,    :old_ap],
+                             [:add_column,    Bare35Content, :allow_pings,    :integer],
+                             [:rename_column, Bare35Content, :allow_comments, :old_ac],
+                             [:add_column,    Bare35Content, :allow_comments, :integer]) do |c|
+      unless $schema_generator
+        c.allow_pings    = c.old_ap ? 1 : 0 unless c.old_ap.nil?
+        c.allow_comments = c.old_ac ? 1 : 0 unless c.old_ac.nil?
+      end
+    end
+    remove_column :contents, :old_ap
+    remove_column :contents, :old_ac
+  end
+end

data/db/migrate/036_add_tag_display_name.rb

@@ -0,0 +1,53 @@
+class AddTagDisplayName < ActiveRecord::Migration
+  class Tag < ActiveRecord::Base
+    has_and_belongs_to_many :articles
+  end
+
+  class Content < ActiveRecord::Base
+  end
+
+  class Article < Content
+    has_and_belongs_to_many :tags
+  end
+
+  def self.up
+    STDERR.puts 'Adding display name to tags'
+    modify_tables_and_update(:add_column, Tag, :display_name, :string) do
+      unless $schema_generator
+        Tag.find(:all).each do |tag|
+          tag.display_name = tag.name
+          tag.name = tag.name.tr(' ', '').downcase
+          if tag.name != tag.display_name
+            # we need to make sure we're not attempting to create duplicate-named tags
+            # if so, we need to coalesce them
+            # Hopefully this code isn't necessary, but somebody may have tags "Monty Python" and "montypython"
+            # Please note that this code isn't going to be tested very well, if at all, because of my limited
+            # testing setup. But in my quickie tests it appears to be working right
+            if origtag = Tag.find(:first, :conditions => [%{name = ? AND id != ?}, tag.name, tag.id])
+              tag.articles.each do |article|
+                # replace our tag with origtag in article.tags
+                article.tags = article.tags.collect { |x| x.id == tag.id ? origtag : x }
+              end
+              tag.destroy
+            else
+              # ok, original tag
+              tag.save!
+            end
+          else
+            tag.save!
+          end
+        end
+      end
+    end
+  end
+
+  def self.down
+    STDERR.puts 'Removing display name from tags'
+    unless $schema_generator
+      Tag.find_and_update(:all) do |tag|
+        tag.name = tag.display_name
+      end
+    end
+    remove_column :tags, :display_name
+  end
+end

data/db/migrate/037_enlarge_ip_field.rb

@@ -0,0 +1,9 @@
+class EnlargeIpField < ActiveRecord::Migration
+  def self.up
+    change_column :contents, :ip, :string, :limit => 40
+  end
+
+  def self.down
+    change_column :contents, :ip, :string
+  end
+end

data/db/migrate/038_add_blog_object.rb

@@ -0,0 +1,52 @@
+class Bare38Blog < ActiveRecord::Base
+  include BareMigration
+end
+
+class Bare38Setting < ActiveRecord::Base
+  include BareMigration
+end
+
+class AddBlogObject < ActiveRecord::Migration
+  def self.up
+    begin
+      STDERR.puts "Adding a blogs table"
+      create_table :blogs do |t|
+        t.column :dummy, :string unless $schema_generator
+      end
+      unless $schema_generator
+        Bare38Blog.reset_column_information
+
+        add_column :settings, :blog_id, :integer
+        Bare38Setting.reset_column_information
+
+        Bare38Setting.transaction do
+          STDERR.puts "Creating default blog"
+          default_blog = Bare38Blog.create!
+
+          STDERR.puts "Connecting settings to the default blog"
+
+          STDERR.puts "New Default blog has id: " + default_blog.id.to_s
+          STDERR.puts "Migrating #{Bare38Setting.find(:all).size} settings to the new Blog"
+
+          Bare38Setting.find(:all).each do |setting|
+            setting.blog_id = default_blog.id
+            setting.save!
+          end
+        end
+        remove_column :blogs, :dummy
+      end
+    rescue Exception => e
+      STDERR.puts("Rolling back the changes")
+      drop_table(:blogs) rescue nil
+      remove_column(:settings, :blog_id) rescue nil
+      raise e
+    end
+  end
+
+  def self.down
+    STDERR.puts "Unlinking settings and removing the blogs table"
+    Bare38Setting.delete_all(["blog_id != ?", Bare38Blog.find(:first)])
+    remove_column :settings, :blog_id
+    drop_table :blogs
+  end
+end

data/db/migrate/039_serialize_blog_attributes.rb

@@ -0,0 +1,160 @@
+class SerializeBlogAttributes < ActiveRecord::Migration
+  class BareSetting < ActiveRecord::Base
+    include BareMigration
+    belongs_to :blog, :class_name => "SerializeBlogAttributes::BareBlog"
+  end
+
+  class BareBlog < ActiveRecord::Base
+    include BareMigration
+    serialize :settings
+
+    class SettingSpec
+      attr_accessor :key, :type, :default
+
+      def initialize(key, type, default)
+        self.key     = key
+        self.type    = type
+        self.default = default
+      end
+
+      def normalize_value(line)
+        if !line || line.value.nil?
+          return default
+        end
+        value = line.value
+        case type
+        when :boolean
+          ! value.match(/^(?:f(?:alse)?|0|)$/)
+        when :integer
+          value.to_i
+        when :string
+          value
+        else
+          YAML::load(value)
+        end
+      end
+
+      def stringify_value(value)
+        case type
+        when :boolean
+          value.to_s
+        when :integer
+          value.to_s
+        when :string
+          value
+        else
+          value.to_yaml
+        end
+      end
+    end
+
+    def self.fields(key = nil)
+      @@fields ||= { }
+      key ? @@fields[key.to_sym] : @@fields
+    end
+
+    def self.setting(key, type, default)
+      fields[key.to_sym] = SettingSpec.new(key.to_sym, type, default)
+    end
+
+    # Description
+    setting :blog_name,                  :string, 'My Shiny Weblog!'
+    setting :blog_subtitle,              :string, ''
+    setting :geourl_location,            :string, ''
+
+    # Spam
+    setting :sp_global,                  :boolean, false
+    setting :sp_article_auto_close,      :integer, 0
+    setting :sp_allow_non_ajax_comments, :boolean, true
+    setting :sp_url_limit,               :integer, 0
+
+    # Podcasting
+    setting :itunes_explicit,            :boolean, false
+    setting :itunes_author,              :string, ''
+    setting :itunes_subtitle,            :string, ''
+    setting :itunes_summary,             :string, ''
+    setting :itunes_owner,               :string, ''
+    setting :itunes_email,               :string, ''
+    setting :itunes_name,                :string, ''
+    setting :itunes_copyright,           :string, ''
+
+    # Mostly Behaviour
+    setting :text_filter,                :string, ''
+    setting :comment_text_filter,        :string, ''
+    setting :limit_article_display,      :integer, 10
+    setting :limit_rss_display,          :integer, 10
+    setting :default_allow_pings,        :boolean, false
+    setting :default_allow_comments,     :boolean, true
+    setting :link_to_author,             :boolean, false
+    setting :show_extended_on_rss,       :boolean, true
+    setting :theme,                      :string, 'azure'
+    setting :use_gravatar,               :boolean, false
+    setting :ping_urls,                  :string, "http://rpc.technorati.com/rpc/ping\nhttp://ping.blo.gs/\nhttp://rpc.weblogs.com/RPC2"
+    setting :send_outbound_pings,        :boolean, true
+    setting :email_from,                 :string, 'scott@sigkill.org'
+
+    # Jabber config
+    setting :jabber_address,             :string, ''
+    setting :jabber_password,            :string, ''
+  end
+
+
+  def self.up
+    add_column :blogs, :settings, :text
+    unless $schema_generator
+      begin
+        BareSetting.transaction do
+          BareBlog.find(:all).each do |blog|
+            blog.settings = { }
+            BareSetting.with_scope(:find => { :conditions => "blog_id = #{blog.id}"}) do
+              BareBlog.fields.each do |key, spec|
+                next unless setting = BareSetting.find_by_name(key.to_s, :limit => 1)
+                blog.settings[key.to_s] =
+                  spec.normalize_value(setting)
+                BareSetting.delete(setting.id)
+              end
+            end
+            blog.save
+          end
+        end
+      rescue Exception => e
+        remove_column :blogs, :settings rescue nil
+        raise e
+      end
+    end
+    drop_table :settings
+  end
+
+  def self.down
+    begin
+      create_settings
+      unless $schema_generator
+        BareSetting.transaction do
+          BareBlog.find(:all).each do |blog|
+            blog.settings ||= { }
+            BareSetting.with_scope(:create => { :blog_id => blog.id }) do
+              BareBlog.fields.each do |key, spec|
+                next unless blog.settings.has_key?(key.to_s)
+                BareSetting.create!(:name => key.to_s,
+                                    :value => spec.stringify_value(blog.settings[key.to_s]))
+              end
+            end
+          end
+        end
+      end
+      remove_column :blogs, :settings rescue nil
+    rescue Exception => e
+      drop_table :settings rescue nil
+      raise e
+    end
+  end
+
+  def self.create_settings
+    create_table :settings do |t|
+      t.column :name,     :string, :limit => 255
+      t.column :value,    :string, :limit => 255
+      t.column :position, :integer
+      t.column :blog_id,  :integer
+    end
+  end
+end