locomotive_cms 0.0.4.beta3 → 0.0.4.beta4

data/Gemfile

@@ -17,9 +17,9 @@ gem 'formtastic', '>= 1.1.0'
 gem 'inherited_resources', '>= 1.1.2'
 
 gem 'rmagick', '= 2.12.2'
-gem 'locomotive_carrierwave', :require => 'carrierwave'
+gem 'locomotive_carrierwave', '0.5.0.1.beta2', :require => 'carrierwave'
 
-gem 'custom_fields', '1.0.0.beta'
+gem 'custom_fields', '1.0.0.beta2'
 gem 'fog', '0.3.7'
 gem 'mimetype-fu'
 gem 'actionmailer-with-request'

data/app/controllers/admin/imports_controller.rb

@@ -23,28 +23,39 @@ module Admin
     def new; end
 
     def create
-      if params[:zipfile].blank?
-        @error = t('errors.messages.blank')
-        flash[:alert] = t('flash.admin.imports.create.alert')
-        render 'new'
-      else
-        path = self.store_zipfile!
+      identifier = store_zipfile!
 
-        job = Locomotive::Import::Job.new(path, current_site)
+      if identifier
+        job = Locomotive::Import::Job.new(identifier, current_site)
         Delayed::Job.enqueue job, { :site => current_site, :job_type => 'import' }
 
         flash[:notice] = t('flash.admin.imports.create.notice')
 
         redirect_to admin_import_url
+      else
+        @error = t('errors.messages.invalid_theme_file')
+        flash[:alert] = t('flash.admin.imports.create.alert')
+
+        render 'new'
       end
     end
 
     protected
 
     def store_zipfile!
+      return nil if params[:zipfile].blank?
+
       file = CarrierWave::SanitizedFile.new(params[:zipfile])
-      new_file = file.copy_to(File.join(Rails.root, 'tmp', 'files', current_site.id.to_s, file.filename))
-      new_file.path
+
+      uploader = ThemeUploader.new(current_site)
+
+      begin
+        uploader.store!(file)
+      rescue CarrierWave::IntegrityError
+        return nil
+      end
+
+      uploader.identifier
     end
 
   end

data/app/models/asset_collection.rb

@@ -53,14 +53,16 @@ class AssetCollection
   end
 
   def store_asset_positions!
-    return if @assets_order.nil?
+    return if @assets_order.blank?
 
-    @assets_order.split(',').each_with_index do |asset_id, index|
+    ids = @assets_order.split(',').collect { |id| BSON::ObjectId(id) }
+
+    ids.each_with_index do |asset_id, index|
       self.assets.find(asset_id).position = index
     end
 
     self.assets.clone.each do |asset|
-      if !@assets_order.split(',').include?(asset._id)
+      if !ids.include?(asset._id)
         self.assets.delete(asset)
         asset.send(:delete)
       end

data/app/models/content_type.rb

@@ -68,7 +68,7 @@ class ContentType
 
   def sort_contents!(order)
     order.split(',').each_with_index do |id, position|
-      self.contents.find(id)._position_in_list = position
+      self.contents.find(BSON::ObjectId(id))._position_in_list = position
     end
     self.save
   end

data/app/uploaders/theme_uploader.rb

@@ -0,0 +1,15 @@
+class ThemeUploader < ::CarrierWave::Uploader::Base
+
+  def store_dir
+    "sites/#{model.id}/tmp/themes"
+  end
+
+  def cache_dir
+    "#{Rails.root}/tmp/uploads"
+  end
+
+  def extension_white_list
+    %w(zip)
+  end
+
+end

data/config/environments/production.rb

@@ -46,3 +46,5 @@ Locomotive::Application.configure do
 
   config.active_support.deprecation = :notify
 end
+
+# TODO: Put your carrierwave config down here

data/config/locales/default_en.yml

@@ -13,6 +13,7 @@ en:
         array_too_short: "is too small (minimum element number is %{count})"
         liquid_syntax: "Liquid Syntax error, please check the syntax"
         liquid_extend: "The page extends a template which does not exist"
+        invalid_theme_file: "can't be blank or isn't a zip file"
 
   attributes:
     defaults:

data/config/locales/default_fr.yml

@@ -34,6 +34,7 @@ fr:
       array_too_short: "est trop petit (le nombre minimum d'éléments est %{count})"
       liquid_syntax: "Erreur de syntaxe dans les sections de page, veuillez vérifier la syntaxe"
       liquid_extend: "La page étend le contenu d'une page qui n'existe pas"
+      invalid_theme_file: "doit être rempli ou n'est pas un fichier zip"
 
   attributes:
     defaults:

data/config/mongoid.yml

@@ -8,7 +8,8 @@ defaults: &defaults
 
 development:
   <<: *defaults
-  database: locomotive_dev
+  # database: locomotive_dev
+  database: locomotive_hosting_production
 
 test:
   <<: *defaults

data/lib/locomotive/import/job.rb

@@ -4,10 +4,10 @@ module Locomotive
   module Import
     class Job
 
-      def initialize(theme_file, site = nil, enabled = {})
-        raise "Theme zipfile not found" unless File.exists?(theme_file)
+      def initialize(identifier, site = nil, enabled = {})
+        raise "Theme identifier not found" if identifier.blank?
 
-        @theme_file = theme_file
+        @identifier = identifier
         @site = site
         @enabled = enabled
       end
@@ -17,7 +17,7 @@ module Locomotive
       end
 
       def perform
-        puts "theme_file = #{@theme_file} / #{@site.present?} / #{@enabled.inspect}"
+        self.log "theme identifier #{@identifier} / enabled steps = #{@enabled.inspect}"
 
         self.unzip!
 
@@ -33,21 +33,72 @@ module Locomotive
 
         %w(site content_types assets asset_collections snippets pages).each do |step|
           if @enabled[step] != false
+            self.log "performing '#{step}' step"
             "Locomotive::Import::#{step.camelize}".constantize.process(context)
             @worker.update_attributes :step => step if @worker
           else
-            puts "skipping #{step}"
+            self.log "skipping #{step}"
           end
         end
       end
 
+      def success(worker)
+        self.log 'deleting original zip file'
+
+        uploader = ThemeUploader.new(@site)
+
+        uploader.retrieve_from_store!(@identifier)
+
+        uploader.remove!
+
+        self.log 'deleting working folder'
+
+        FileUtils.rm_rf(themes_folder) rescue nil
+      end
+
       protected
 
+      def log(message)
+        puts "\t[import_theme] #{message}"
+      end
+
+      def themes_folder
+        File.join(Rails.root, 'tmp', 'themes', @site.id.to_s)
+      end
+
+      def prepare_folder
+        FileUtils.rm_rf self.themes_folder if File.exists?(self.themes_folder)
+
+        FileUtils.mkdir_p(self.themes_folder)
+      end
+
+      def retrieve_zip_file
+        uploader = ThemeUploader.new(@site)
+
+        uploader.retrieve_from_store!(@identifier)
+
+        if uploader.file.respond_to?(:url)
+          self.log 'file from remote storage'
+
+          @theme_file = File.join(self.themes_folder, @identifier)
+
+          File.open(@theme_file, 'w') { |f| f.write(uploader.file.read) }
+        else # local filesystem
+          self.log 'file from local storage'
+
+          @theme_file = uploader.path
+        end
+      end
+
       def unzip!
-        Zip::ZipFile.open(@theme_file) do |zipfile|
-          destination_path = File.join(Rails.root, 'tmp', 'themes', @site.id.to_s)
+        self.prepare_folder
 
-          FileUtils.rm_r destination_path, :force => true
+        self.retrieve_zip_file
+
+        self.log "unzip #{@theme_file}"
+
+        Zip::ZipFile.open(@theme_file) do |zipfile|
+          destination_path = self.themes_folder
 
           zipfile.each do |entry|
             next if entry.name =~ /__MACOSX/

data/lib/locomotive/liquid/tags/consume.rb

@@ -39,7 +39,7 @@ module Locomotive
         protected
 
         def render_all_and_cache_it(context)
-          Rails.cache.fetch(@cache_key, :expires_in => @expires_in) do
+          Rails.cache.fetch(@cache_key, :expires_in => @expires_in, :force => @expires_in == 0) do
             context.stack do
               context.scopes.last[@target.to_s] = Locomotive::Httparty::Webservice.consume(@url, @options.symbolize_keys)
 

data/lib/locomotive/version.rb

@@ -1,3 +1,3 @@
 module Locomotive #:nodoc
-  VERSION = "0.0.4.beta3"
+  VERSION = "0.0.4.beta4"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification 
 name: locomotive_cms
 version: !ruby/object:Gem::Version 
-  hash: -1848230053
+  hash: -1848230056
   prerelease: true
   segments: 
   - 0
   - 0
   - 4
-  - beta3
-  version: 0.0.4.beta3
+  - beta4
+  version: 0.0.4.beta4
 platform: ruby
 authors: 
 - Didier Lafforgue
@@ -16,7 +16,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-10-18 00:00:00 +02:00
+date: 2010-10-19 00:00:00 +02:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -431,6 +431,7 @@ files:
 - app/uploaders/asset_uploader.rb
 - app/uploaders/editable_file_uploader.rb
 - app/uploaders/theme_asset_uploader.rb
+- app/uploaders/theme_uploader.rb
 - app/views/admin/accounts/new.html.haml
 - app/views/admin/asset_collections/_asset.html.haml
 - app/views/admin/asset_collections/edit.html.haml