alchemy_cms 2.6.1 → 2.6.2

data/lib/alchemy/upgrader/two_point_five.rb

@@ -0,0 +1,79 @@
+module Alchemy
+  module Upgrader::TwoPointFive
+
+  private
+  
+    def convert_picture_storage
+      desc "Convert the picture storage"
+      converted_images = []
+      images = Dir.glob Rails.root.join 'uploads/pictures/**/*.*'
+      if images.blank?
+        log "No pictures found", :skip
+      else
+        images.each do |image|
+          image_uid = image.gsub(/#{Rails.root.to_s}\/uploads\/pictures\//, '')
+          image_id = image_uid.split('/').last.split('.').first
+          picture = Alchemy::Picture.find_by_id(image_id)
+          if picture && picture.image_file_uid.blank?
+            picture.image_file_uid = image_uid
+            picture.image_file_size = File.new(image).size
+            if picture.save!
+              log "Converted #{image_uid}"
+            end
+          else
+            log "Picture with id #{image_id} not found or already converted.", :skip
+          end
+        end
+      end
+    end
+
+    def removed_standard_set_notice
+      warn = <<-WARN
+We removed the standard set from Alchemy core!
+In order to get the standard set back, install the `alchemy-demo_kit` gem.
+WARN
+      todo warn
+    end
+
+    def renamed_t_method
+      warn = <<-WARN
+We renamed alchemy's `t` method override into `_t` to avoid conflicts with Rails own t method!
+If you use the `t` method to translate alchemy scoped keys, then you have to use the `_t` method from now on.
+WARN
+      todo warn
+    end
+
+    def migrated_to_devise
+      warn = <<-WARN
+We changed the authentication provider from Authlogic to Devise.
+
+If you are upgrading from an old Alchemy version < 2.5.0, then you have to make changes to your Devise configuration.
+
+1. Run:
+
+$ rails g alchemy:devise
+
+And alter the encryptor to authlogic_sha512
+and the stretches value from 10 to 20
+
+# config/initializers/devise.rb
+config.stretches = Rails.env.test? ? 1 : 20
+config.encryptor = :authlogic_sha512
+
+2. Add the encryptable module to your Alchemy config.yml:
+
+# config/alchemy/config.yml
+devise_modules:
+  - :database_authenticatable
+  - :trackable
+  - :validatable
+  - :timeoutable
+  - :recoverable
+  - :encryptable
+
+WARN
+      todo warn
+    end
+    
+  end
+end

data/lib/alchemy/upgrader/two_point_four.rb

@@ -0,0 +1,25 @@
+module Alchemy
+  module Upgrader::TwoPointFour
+
+  private
+  
+    def removed_richmedia_essences_notice
+      warn = <<-WARN
+We removed the EssenceAudio, EssenceFlash and EssenceVideo essences from Alchemy core!
+
+In order to get the essences back, install the `alchemy-richmedia-essences` gem.
+
+gem 'alchemy-richmedia-essences'
+
+We left the tables in your database, you can simply drop them if you don't use these essences in your project.
+
+drop_table :alchemy_essence_audios
+drop_table :alchemy_essence_flashes
+drop_table :alchemy_essence_videos
+
+WARN
+      todo warn
+    end
+
+  end
+end

data/lib/alchemy/upgrader/two_point_one.rb

@@ -0,0 +1,95 @@
+module Alchemy
+  module Upgrader::TwoPointOne
+
+  private
+
+    # Creates Language model if it does not exist (Alchemy CMS prior v1.5)
+    # Also creates missing associations between pages and languages
+    def upgrade_to_language
+      desc "Creating languages for pages"
+      Alchemy::Page.all.each do |page|
+        if !page.language_code.blank? && page.language.nil?
+          root = page.get_language_root
+          lang = Alchemy::Language.find_or_create_by_language_code(
+            :name => page.language_code.capitalize,
+            :language_code => page.language_code,
+            :frontpage_name => root.name,
+            :page_layout => root.page_layout,
+            :public => true
+          )
+          page.language = lang
+          if page.save(:validate => false)
+            log "Set language for page #{page.name} to #{lang.name}."
+          end
+        else
+          log("Language for page #{page.name} already set.", :skip)
+        end
+      end
+    end
+
+    def upgrade_layoutpages
+      desc "Setting language of layoutpages"
+      default_language = Alchemy::Language.get_default
+      raise "No default language found." if default_language.nil?
+      layoutpages = Alchemy::Page.layoutpages
+      if layoutpages.any?
+        layoutpages.each do |page|
+          if page.language.class == String || page.language.nil?
+            page.language = default_language
+            if page.save(:validate => false)
+              log "Set language for page #{page.name} to #{default_language.name}."
+            end
+          else
+            log "Language for page #{page.name} already set.", :skip
+          end
+        end
+      else
+        log "No layoutpages found.", :skip
+      end
+    end
+
+    def upgrade_essence_link_target_default
+      desc "Setting new link_target default"
+      essences = (Alchemy::EssencePicture.all + Alchemy::EssenceText.all)
+      if essences.any?
+        essences.each do |essence|
+          case essence.link_target
+          when '1'
+            if essence.update_attribute(:link_target, 'blank')
+              log("Updated #{essence.preview_text} link target to #{essence.link_target}.")
+            end
+          when '0'
+            essence.update_attribute(:link_target, nil)
+            log("Updated #{essence.preview_text} link target to #{essence.link_target.inspect}.")
+          else
+            log("#{essence.preview_text} already upgraded.", :skip)
+          end
+        end
+      else
+        log("No essences to upgrade found.", :skip)
+      end
+    end
+
+    # Updates all essence_type of Content if not already namespaced.
+    def upgrade_to_namespaced_essence_type
+      desc "Namespacing essence_type columns"
+      depricated_contents = Alchemy::Content.where("essence_type LIKE ?", "Essence%")
+      if depricated_contents.any?
+        success = 0
+        errors = []
+        depricated_contents.each do |c|
+          if c.update_attribute(:essence_type, c.essence_type.gsub(/^Essence/, 'Alchemy::Essence'))
+            success += 1
+          else
+            errors << c.errors.full_messages
+          end
+        end
+        log("Namespaced #{success} essence_type columns.") if success > 0
+        log("#{errors.count} errors while namespacing essence_type columns.\n#{errors.join('\n')}", :error) if errors.count > 0
+      else
+        log "No essence_type columns to be namespaced found.", :skip
+      end
+    end
+
+  end
+end

data/lib/alchemy/upgrader/two_point_six.rb

@@ -0,0 +1,32 @@
+module Alchemy
+  module Upgrader::TwoPointSix
+
+  private
+
+    def convert_attachment_storage
+      desc "Convert the attachment storage"
+      converted_files = []
+      files = Dir.glob Rails.root.join 'uploads/attachments/**/*.*'
+      if files.blank?
+        log "No attachments found", :skip
+      else
+        files.each do |file|
+          file_uid = file.gsub(/#{Rails.root.to_s}\/uploads\/attachments\//, '')
+          file_id = file_uid.split('/')[1].to_i
+          attachment = Alchemy::Attachment.find_by_id(file_id)
+          if attachment && attachment.file_uid.blank?
+            attachment.file_uid = file_uid
+            attachment.file_size = File.new(file).size
+            attachment.file_name = attachment.sanitized_filename
+            if attachment.save!
+              log "Converted #{file_uid}"
+            end
+          else
+            log "Attachment with id #{file_id} not found or already converted.", :skip
+          end
+        end
+      end
+    end
+
+  end
+end

data/lib/alchemy/upgrader/two_point_three.rb

@@ -0,0 +1,25 @@
+module Alchemy
+  module Upgrader::TwoPointThree
+
+  private
+  
+    def gallery_pictures_change_notice
+      note =<<NOTE
+We have changed the way Alchemy handles EssencePictures in elements.
+
+It is now possible to have single EssencePictures and galleries side by side in the same element.
+All element editor views containing render_picture_editor with option `maximum_amount_of_images => 1` must be changed into render_essence_editor_by_name.
+In the yml description of these elements add a new content for this picture.
+
+In order to upgrade your elements in the database run:
+
+rails g alchemy:gallery_pictures_migration
+
+and alter `db/seeds.rb`, so that it contains all elements that have essence pictures.
+
+NOTE
+      todo note
+    end
+
+  end
+end

data/lib/alchemy/upgrader/two_point_two.rb

@@ -0,0 +1,69 @@
+module Alchemy
+  module Upgrader::TwoPointTwo
+
+  private
+  
+    def convert_essence_texts_displayed_as_select_into_essence_selects
+      desc "Converting EssenceTexts displayed as select into EssenceSelects"
+      contents_found = 0
+      elements = Alchemy::Element.descriptions.select { |e| e['contents'].present? && !e['contents'].detect { |c| c['settings'].present? && c['settings']['display_as'] == 'select' }.nil? }
+      contents = elements.collect { |el| el['contents'] }.flatten.select { |c| c['settings'] && c['settings']['display_as'] == 'select' }.flatten
+      content_names = contents.collect { |c| c['name'] }
+      Alchemy::Content.essence_texts.where(
+        :name => content_names,
+        :alchemy_elements => {:name => elements.collect { |e| e['name'] }}
+      ).joins(:element).each do |content|
+        new_content = Alchemy::Content.new(:name => content.name, :element_id => content.element.id)
+        if new_content.create_essence!('name' => content.name, 'type' => 'EssenceSelect')
+          new_content.essence.value = content.ingredient
+          if new_content.essence.save
+            contents_found += 1
+            log "Converted #{content.name}'s essence_type into EssenceSelect"
+            content.destroy
+          else
+            log "Could not save essence: #{new_content.essence.errors.full_messages.join(', ')}", :error
+          end
+        else
+          log "Could not create content: #{new_content.errors.full_messages.join(', ')}", :error
+        end
+      end
+      if contents_found > 0
+        todo "Please open your elements.yml file and change all type values from these contents:\n\n#{content_names.join(', ')}\n\ninto EssenceSelect."
+      else
+        log "No EssenceTexts with display_as select setting found.", :skip
+      end
+    end
+
+    def convert_essence_texts_displayed_as_checkbox_into_essence_booleans
+      desc "Converting EssenceTexts displayed as checkbox into EssenceBooleans"
+      contents_found = 0
+      elements = Alchemy::Element.descriptions.select { |e| e['contents'].present? && !e['contents'].detect { |c| c['settings'].present? && c['settings']['display_as'] == 'checkbox' }.nil? }
+      contents = elements.collect { |el| el['contents'] }.flatten.select { |c| c['settings'] && c['settings']['display_as'] == 'checkbox' }.flatten
+      content_names = contents.collect { |c| c['name'] }
+      Alchemy::Content.essence_texts.where(
+        :name => content_names,
+        :alchemy_elements => {:name => elements.collect { |e| e['name'] }}
+      ).joins(:element).each do |content|
+        new_content = Alchemy::Content.new(:name => content.name, :element_id => content.element.id)
+        if new_content.create_essence!('name' => content.name, 'type' => 'EssenceBoolean')
+          new_content.essence.value = content.ingredient
+          if new_content.essence.save
+            contents_found += 1
+            log "Converted #{content.name}'s essence_type into EssenceBoolean"
+            content.destroy
+          else
+            log "Could not save essence: #{new_content.essence.errors.full_messages.join(', ')}", :error
+          end
+        else
+          log "Could not create content: #{new_content.errors.full_messages.join(', ')}", :error
+        end
+      end
+      if contents_found > 0
+        todo "Please open your elements.yml file and change all type values from these contents:\n\n#{content_names.join(', ')}\n\ninto EssenceBoolean."
+      else
+        log "No EssenceTexts with display_as checkbox setting found.", :skip
+      end
+    end
+
+  end
+end

data/lib/alchemy/upgrader/two_point_zero.rb

@@ -0,0 +1,13 @@
+module Alchemy
+  module Upgrader::TwoPointZero
+
+  private
+
+    def strip_alchemy_from_schema_version_table
+      ActiveRecord::Base.connection.execute(
+        "UPDATE schema_migrations SET version = REPLACE(version,'-alchemy','')"
+      )
+    end
+
+  end
+end

data/lib/alchemy/version.rb

@@ -1,6 +1,6 @@
 module Alchemy
 
-  VERSION = "2.6.1"
+  VERSION = "2.6.2"
 
   def self.version
     VERSION

data/lib/tasks/alchemy/upgrade.rake

@@ -12,7 +12,7 @@ namespace :alchemy do
     task :list => [:environment] do
       puts "\nAvailable upgrade tasks"
       puts "-----------------------\n"
-      methods = (Alchemy::Upgrader.private_methods - Object.private_methods - Alchemy::Upgrader.superclass.private_methods)
+      methods = Alchemy::Upgrader.all_upgrade_tasks
       if methods.any?
         methods.each { |method| puts method }
         puts "\nUsage:"

data/spec/libraries/mount_point_spec.rb

@@ -0,0 +1,62 @@
+require 'spec_helper'
+require 'ostruct'
+
+describe Alchemy::MountPoint do
+
+  describe '.get' do
+
+    it "returns the path of alchemy's mount point" do
+      Alchemy::MountPoint.stub!(:mount_point).and_return('/cms')
+      Alchemy::MountPoint.get.should == '/cms'
+    end
+
+    it "removes the leading slash if root mount point" do
+      Alchemy::MountPoint.stub!(:mount_point).and_return('/')
+      Alchemy::MountPoint.get.should == ''
+    end
+
+    context "with remove_leading_slash_if_blank set to false" do
+      before {
+        Alchemy::MountPoint.stub!(:mount_point).and_return('/')
+      }
+
+      it "does not remove the leading white slash of path" do
+        Alchemy::MountPoint.get(false).should == '/'
+      end
+
+      context "and with mount point not root" do
+        before {
+          Alchemy::MountPoint.stub!(:mount_point).and_return('/cms')
+        }
+
+        it "does not remove the leading white slash of path" do
+          Alchemy::MountPoint.get(false).should == '/cms'
+        end
+      end
+    end
+  end
+
+  describe '.routes' do
+    it "returns the routes object from alchemy engine" do
+      Alchemy::MountPoint.routes.should be_instance_of(Journey::Route)
+    end
+  end
+
+  describe '.mount_point' do
+    it 'returns the raw mount point path from routes' do
+      Alchemy::MountPoint.stub!(:routes).and_return(OpenStruct.new(path: OpenStruct.new(spec: '/cms')))
+      Alchemy::MountPoint.mount_point.should == '/cms'
+    end
+
+    context "Alchemy routes could not be found" do
+      before {
+        Alchemy::MountPoint.stub!(:routes).and_return(nil)
+      }
+
+      it "falls back to root path" do
+        Alchemy::MountPoint.mount_point.should == '/'
+      end
+    end
+  end
+
+end

data/spec/libraries/shell_spec.rb

@@ -0,0 +1,123 @@
+require 'spec_helper'
+
+module Alchemy
+
+  # Class fixture
+  class MyToDoList
+    extend Shell
+  end
+
+  describe Shell do
+
+    before { MyToDoList.stub!(:puts) }
+
+    describe '.todo' do
+      it "should add given string as a todo by delegating to .add_todo" do
+        MyToDoList.should_receive(:add_todo).with("new todo")
+        MyToDoList.todo("new todo")
+      end
+    end
+
+    describe '.todos' do
+      it "should be an Array" do
+        expect(MyToDoList.todos).to be_a(Array)
+      end
+    end
+
+    describe '.add_todo' do
+      it "should add the given string to the .todos array" do
+        MyToDoList.add_todo('1')
+        MyToDoList.add_todo('2')
+        expect(MyToDoList.todos).to eq(['1', '2'])
+      end
+    end
+
+    describe '.display_todos' do
+      context 'if there are todos in the list' do
+        before do
+          MyToDoList.stub!(:todos).and_return(['My first todo', 'My second todo'])
+        end
+
+        it "should log them" do
+          MyToDoList.should_receive(:log).any_number_of_times
+          MyToDoList.display_todos
+        end
+
+        it "should iterate through the todos with an index" do
+          MyToDoList.todos.should_receive(:each_with_index)
+          MyToDoList.display_todos
+        end
+      end
+
+      context 'if there are todos in the list' do
+        before do
+          MyToDoList.stub!(:todos).and_return([])
+        end
+
+        it "should not log anything" do
+          MyToDoList.should_not_receive(:log)
+          MyToDoList.display_todos
+        end
+      end
+    end
+
+    describe '.log' do
+
+      context 'if the message type is "skip"' do
+        it "the output color should be yellow and cleared again" do
+          MyToDoList.should_receive(:color).with(:yellow)
+          MyToDoList.should_receive(:color).with(:clear)
+          MyToDoList.log('in yellow, please', :skip)
+        end
+      end
+
+      context 'if the message type is "error"' do
+        it "the output color should be yellow and cleared again" do
+          MyToDoList.should_receive(:color).with(:red)
+          MyToDoList.should_receive(:color).with(:clear)
+          MyToDoList.log('in red, please', :error)
+        end
+      end
+
+      context 'if the message type is "message"' do
+        it "the output color should just be cleared" do
+          MyToDoList.should_receive(:color).with(:clear)
+          MyToDoList.log('cleared, please', :message)
+        end
+      end
+
+      context 'if no message type is given' do
+        it "the output color should be green" do
+          MyToDoList.should_receive(:color).with(:green)
+          MyToDoList.should_receive(:color).with(:clear)
+          MyToDoList.log('in green, please')
+        end
+      end
+    end
+
+    describe '.color' do
+
+      context 'if given name is a constant of Thor::Shell::Color' do
+        before do
+          Thor::Shell::Color.stub!(:const_defined?).and_return(true)
+        end
+
+        it "should call the constant" do
+          String.any_instance.should_receive(:constantize).and_return('')
+          MyToDoList.send(:color, :red)
+        end
+      end
+
+      context 'if given name is not a defined constant of Thor::Shell::Color' do
+        before do
+          Thor::Shell::Color.stub!(:const_defined?).and_return(false)
+        end
+
+        it "should return en empty string" do
+          expect(MyToDoList.send(:color, :not_existing)).to eq('')
+        end
+      end
+    end
+
+  end
+end