imagine_cms 4.1.4 → 4.2.0

data/app/controllers/cms/content_controller.rb

@@ -43,7 +43,7 @@ module Cms # :nodoc:
       end
       
       if template_found
-        render :template => @content_path
+        render template: @content_path
       elsif show_from_db
         return
       else
@@ -62,8 +62,10 @@ module Cms # :nodoc:
     
     def rendering_error(exception = nil)
       logger.error "500 from #{request.referer} (exception: #{exception})"
+      logger.error exception.annoted_source_code if exception.class.name == 'ActionView::Template::Error'
+      
       @exception = exception.message
-      render :template => 'imagine_cms/errors/500', :status => 500
+      render template: 'imagine_cms/errors/500', status: 500, formats: [ :html ]
     end
     
     def show_from_db
@@ -136,9 +138,9 @@ module Cms # :nodoc:
             params[:offset] = db_path.pop
             db_path.pop
             
-            # don't cache to disk, but do allow browsers and proxies to cache briefly
+            # don't cache to disk on server, but do allow browsers and proxies to cache briefly
             @allow_caching = false
-            expires_in 5.minutes, :public => true
+            expires_in 1.minutes, public: true if Rails.env.production?
           end
         end
         
@@ -189,8 +191,7 @@ module Cms # :nodoc:
             if @page_list_segment
               name = params[:page_list_name]
               key = "obj-page_list-#{name.gsub(/[^\w]/, '_')}"
-              pages = page_list_items(@pg, key).compact.uniq
-              render :inline => render_page_list_segment(name, pages) and return true
+              render :inline => render_page_list_segment(name, @page_list_pages[key]) and return true
             end
             # end of page list segment code
             
@@ -219,41 +220,35 @@ module Cms # :nodoc:
     helper_method :disable_caching
     
     def search
-      @terms = []
       @pages = []
+      @terms = []
+      @terms = params[:q].split(/[^\w\-]+/).reject { |t| t.length < 3 } if params[:q]
       
-      if params[:q]
-        @terms = params[:q].split(/[^\w\-]+/).reject { |t| t.length < 3 }
-      end
-      
-      CmsPage.index_all
       unless @terms.empty?
-        term_variants = []
-        @terms.each do |term|
-          term_variants << [ term, term.singularize, term.pluralize ].uniq.map { |v| v.gsub(/[\[\]\|\:\>\(\)\?]/, '').gsub(/\+/, '\+') }.join('|')
-        end
+        CmsPage.index_all
         
-        conds = [ 'published_version >= 0' ]
-        vars  = []
-        term_variants.each do |term_variant|
-          conds << "(title regexp ?)"
-          vars  << "[[:<:]](#{term_variant})[[:>:]]"
+        base_query = CmsPage.includes(:tags).references(:cms_page_tags).where('published_version >= 0').limit(100)
+        
+        query = base_query.dup
+        @terms.each do |term|
+          term_variants = [ term, term.singularize, term.pluralize ].uniq.map { |v| v.gsub(/[\[\]\|\:\>\(\)\?]/, '').gsub(/\+/, '\+') }.join('|')
+          query = query.where("(title regexp ?)", "[[:<:]](#{term_variants})[[:>:]]")
         end
-        @pages.concat CmsPage.where([ conds.join(' and ') ].concat(vars))
+        @pages.concat query.all
         
-        conds = [ 'published_version >= 0' ]
-        vars  = []
-        term_variants.each do |term_variant|
-          conds << "(title regexp ? or search_index regexp ?)"
-          vars  << "[[:<:]](#{term_variant})[[:>:]]" << "[[:<:]](#{term_variant})[[:>:]]"
+        query = base_query.dup.where('search_index is not null')
+        @terms.each do |term|
+          term_variants = [ term, term.singularize, term.pluralize ].uniq.map { |v| v.gsub(/[\[\]\|\:\>\(\)\?]/, '').gsub(/\+/, '\+') }.join('|')
+          query = query.where("cms_page_tags.name regexp :regex or search_index regexp :regex",
+                              regex: "[[:<:]](#{term_variants})[[:>:]]")
         end
-        @pages.concat CmsPage.where([ conds.join(' and ') ].concat(vars))
+        query.each { |pg| @pages << pg unless @pages.include?(pg) }
         
         # fulltext doesn't work with innodb... may need to make a separate myisam
         # table just for search. (this would be better because it would sort by relevance)
         # @pages.concat CmsPage.where('match (title, search_index) against (?)', params[:q])
       end
-      @pages = @pages.uniq.reject { |pg| pg.search_index.blank? }.first(100)
+      @pages = @pages.first(100)
       
       @pg = CmsPage.new
       @pg.template = CmsTemplate.find_by_name('Search') || CmsTemplate.new
@@ -278,24 +273,12 @@ module Cms # :nodoc:
       render :nothing => true and return unless @pg && params[:page_list_name]
       key = "obj-page_list-#{params[:page_list_name].gsub(/[^\w]/, '_')}"
       
-      load_page_objects or return true
-      
-      options ||= {}
-      today = Time.mktime(Time.now.year, Time.now.month, Time.now.day)
-      case @page_objects["#{key}-date-range"]
-        when 'all'
-        when 'past'
-          options[:end_date] = today
-        when 'future'
-          options[:start_date] = today
-        when 'custom'
-          options[:start_date] = @page_objects["#{key}-date-range-custom-start"]
-          options[:end_date] = @page_objects["#{key}-date-range-custom-end"]
-      end
+      load_page_objects or return false
       
-      str = render_cms_page_to_string(@pg)
+      # populate @page_list_pages
+      render_cms_page_to_string(@pg)
       
-      @pages = page_list_items(@pg, key, options).first(20)
+      @pages = @page_list_pages[key].first(20) rescue []  # why only first 20?
       @page_contents = {}
       
       unless @pages.empty?

data/app/controllers/management/cms_controller.rb

@@ -5,7 +5,7 @@ class Management::CmsController < Management::ApplicationController # :nodoc:
   before_filter :check_permissions
   before_filter :block_basic_users, :except => [
     :index, :edit_page_content,
-    :include_codepress, :disable_caching, :garbage_collect,
+    :disable_caching, :garbage_collect,
     :select_page, :list_pages_select, :request_review,
     :toolbar_preview, :toolbar_edit,
     
@@ -86,16 +86,16 @@ class Management::CmsController < Management::ApplicationController # :nodoc:
     if request.post?
       @temp.assign_attributes(cms_template_params)
       
-      begin
+      # begin
         @pg = CmsPage.new
         @page_objects = HashObject.new
         render_to_string :inline => @temp.content
-      rescue Exception => e
-        message = e.message
-        flash.now[:error] = "<pre>#{ERB::Util.html_escape(message)}</pre>".html_safe
-        logger.debug e
-        return
-      end
+      # rescue Exception => e
+      #   message = e.message
+      #   flash.now[:error] = "<pre>#{ERB::Util.html_escape(message)}</pre>".html_safe
+      #   logger.debug e
+      #   return
+      # end
       
       # this must come after the render_to_string so that we capture template
       # options embedded in snippets
@@ -176,7 +176,8 @@ class Management::CmsController < Management::ApplicationController # :nodoc:
       @pg.template ||= @parent.template
     end
     
-    @attrs = CmsPageObject.where(:obj_type => 'attribute').uniq.pluck(:name).sort
+    @attrs = CmsPageObject.where(obj_type: 'attribute').uniq.pluck(:name).sort
+    @taglist = CmsPageTag.uniq.pluck(:name).sort
     
     if params[:mode] == 'ajax_new' || params[:mode] == 'ajax_edit'
       @pg.published_version = -1 if params[:mode] == 'ajax_new'
@@ -212,51 +213,45 @@ class Management::CmsController < Management::ApplicationController # :nodoc:
       
       if @pg.send(@pg.new_record? ? :save : :save_without_revision)
         # now try to save tags
-        # begin
-          existing_tags = @pg.tags.map(&:name)
-          tags_to_delete = @pg.tags.to_a
-          params[:tags].split(',').map(&:strip).reject(&:blank?).each do |t|
-            if existing_tags.include?(t)
-              # still in use, don't delete
-              tags_to_delete = tags_to_delete.reject { |tag| tag.name == t }
-            else
-              # doesn't exist, create
-              @pg.tags.create(:name => t)
-            end
+        existing_tags = @pg.tags.map(&:name)
+        tags_to_delete = [] ; @pg.tags.each { |t| tags_to_delete << t }
+        params[:tags].split(',').map(&:strip).reject(&:blank?).each do |t|
+          if existing_tags.include?(t)
+            # still in use, don't delete
+            tags_to_delete = tags_to_delete.reject { |tag| tag.name == t }
+          else
+            # doesn't exist, create
+            @pg.tags.create(name: t)
           end
-          tags_to_delete.each { |t| t.destroy }
-        # rescue Exception => e
-        #   logger.debug e
-        # end
+        end
+        tags_to_delete.each { |t| t.destroy }
         
         # now try to save page objects (just attributes in this case)
-        # begin
-          objects_to_delete = @pg.objects.where("obj_type = 'attribute' or obj_type = 'option'").all
+        objects_to_delete = @pg.objects.where("obj_type = 'attribute' or obj_type = 'option'").all
+        
+        (params[:page_objects] || {}).each do |key,val|
+          next if val.blank?
           
-          (params[:page_objects] || {}).each do |key,val|
-            next if val.blank?
-            
-            if key =~ /^obj-(\w+?)-(.+?)$/
-              obj = @pg.objects.where(:name => $2, :obj_type => $1).first
-              obj ||= @pg.objects.build(:name => $2, :obj_type => $1)
-              obj.content = val
-              obj.save
-              objects_to_delete = objects_to_delete.reject { |obj| obj.name == $2 }
-            end
+          if key =~ /^obj-(\w+?)-(.+?)$/
+            obj = @pg.objects.where(:name => $2, :obj_type => $1).first
+            obj ||= @pg.objects.build(:name => $2, :obj_type => $1)
+            obj.content = val
+            obj.save
+            objects_to_delete = objects_to_delete.reject { |obj| obj.name == $2 }
           end
-          
-          objects_to_delete.each { |t| t.destroy }
-        # rescue Exception => e
-        #   logger.debug e
-        # end
+        end
         
-        render :update do |page|
-          case params[:return_to]
-          when 'preview'
+        objects_to_delete.each { |t| t.destroy }
+        
+        case params[:return_to]
+        when 'preview'
+          render :update do |page|
             page.redirect_to "#{@pg.path.blank? ? '' : '/' + @pg.path}/version/#{@pg.published_version > 0 ? @pg.published_version : @pg.version}"
-          else
-            flash[:notice] = 'Page saved.'
-            session[:cms_pages_path] = @pg.path
+          end
+        else
+          flash[:notice] = 'Page saved.'
+          session[:cms_pages_path] = @pg.path
+          render :update do |page|
             page.redirect_to :action => 'pages'
           end
         end
@@ -343,8 +338,8 @@ class Management::CmsController < Management::ApplicationController # :nodoc:
     @page_objects = HashObject.new(params[:page_objects] || {})
     
     if request.get?
-      @pg.revert_to(params[:version]) if params[:version]
-      @pg.objects.where(:cms_page_version => @pg.version).each do |obj|
+      @pg.version = params[:version] if params[:version] && params[:version].to_i != @pg.version
+      @pg.objects.where(cms_page_version: @pg.version).each do |obj|
         key = "obj-#{obj.obj_type.to_s}-#{obj.name}"
         @page_objects[key] = obj.content.html_safe
       end
@@ -362,10 +357,10 @@ class Management::CmsController < Management::ApplicationController # :nodoc:
       @page_title = @pg.title
       
       @cms_head ||= ''
-      @cms_head << "<script type=\"text/javascript\" src=\"#{url_for(:action => 'page_tags_for_lookup')}\"></script>"
+      @cms_head << "<script type=\"text/javascript\" src=\"#{url_for(action: 'page_tags_for_lookup')}\"></script>"
       
       @template_content = substitute_placeholders(@pg.template.content, @pg)
-      render :layout => 'application'
+      render layout: 'application'
     
     elsif request.post?
       CmsPage.transaction do
@@ -441,7 +436,7 @@ class Management::CmsController < Management::ApplicationController # :nodoc:
           key =~ /^obj-(\w+?)-(.+?)$/
           obj = @pg.objects.build(:name => $2, :obj_type => $1)
           
-          # do a little bit of "censorship" to fix up Word pastes
+          # do a little bit of "censorship" to fix up Word pastes and strange things from the editor
           if val.is_a?(String)
             # all meta and link tags
             val.gsub!(/(<\/?)(meta|link)(.*?)>/m, '')
@@ -459,9 +454,6 @@ class Management::CmsController < Management::ApplicationController # :nodoc:
             # not even sure what these are supposed to be
             val.gsub!(/<xml>(.*?)<\/xml>/m, '')
             
-            # pirate styles not welcome
-            val.gsub!(/<style>(.*?)<\/style>/m, '')
-            
             # images pointing to the local drive??
             val.gsub!(/<img src="file:(.*?)>/, '')
             
@@ -478,6 +470,15 @@ class Management::CmsController < Management::ApplicationController # :nodoc:
             
             # but it's easier just remove all tags with colons in them
             val.gsub!(/(<\/?)[\w\d]+:[\w\d]+(.*?)>/, '')
+            
+            
+            ### other non-ms word specific stuff ###
+            
+            # pirate styles not welcome
+            val.gsub!(/<style(?:.*?)>(.*?)<\/style>/m, '')
+            
+            # fix strange <br>s from the editor
+            val.gsub!(/<br>(<\/h\d>|<\/p>)/, '\1')
           end
           
           obj.content = val
@@ -604,19 +605,19 @@ class Management::CmsController < Management::ApplicationController # :nodoc:
                            :focusOnLoad => focusOnLoad.to_s, :style => 'border: 2px dashed gray; padding: 5px',
                            :minHeight => '100px' }.update(html_options))
       content << content_tag(:div, ''.html_safe, :id => "page_object_config_#{key}")
-      content << javascript_tag("addLoadEvent(function () { scanForPageObjects(#{@pg.id}, '#{key}', #{@pg.version}); });")
+      content << javascript_tag("jQuery(document).ready(function () { scanForPageObjects(#{@pg.id}, '#{key}', #{@pg.version}); });")
       content << observe_field("page_objects_#{key}", :function => "scanForPageObjects(#{@pg.id}, '#{key}', #{@pg.version});", :frequency => 2)
       content
     when :page_list
+      # set defaults unless values are present in template
       @page_objects["#{key}-min-item-count"] ||= 3 unless options[:item_min_count]
       @page_objects["#{key}-max-item-count"] ||= 5 unless options[:item_count]
-      @page_objects["#{key}-item-offset"] ||= 0 unless options[:item_offset]
       @page_objects["#{key}-sort-first-field"] ||= options[:primary_sort_key]
       @page_objects["#{key}-sort-first-direction"] ||= options[:primary_sort_direction]
       @page_objects["#{key}-sort-second-field"] ||= options[:secondary_sort_key]
       @page_objects["#{key}-sort-second-direction"] ||= options[:secondary_sort_direction]
       
-      render_to_string(:partial => 'page_list', :locals => { :name => name, :key => key }).html_safe
+      render_to_string(partial: 'page_list', locals: { name: name, key: key, options: options }).html_safe
     when :snippet
       @snippet = CmsSnippet.find_by_name(name)
       if @snippet
@@ -630,6 +631,25 @@ class Management::CmsController < Management::ApplicationController # :nodoc:
   end
   helper_method :insert_object
   
+  # shortcuts
+  def text_editor(name, options = {}, html_options = {})
+    insert_object(name, :text, options, html_options)
+  end
+  alias :texteditor :text_editor
+  helper_method :text_editor, :texteditor
+  
+  def page_list(name, options = {}, html_options = {})
+    insert_object(name, :page_list, options, html_options)
+  end
+  alias :pagelist :page_list
+  helper_method :page_list, :pagelist
+  
+  def snippet(name, options = {}, html_options = {})
+    insert_object(name, :snippet, options, html_options)
+  end
+  helper_method :snippet
+  
+  
   def disable_caching ; end
   helper_method :disable_caching
   
@@ -655,7 +675,7 @@ class Management::CmsController < Management::ApplicationController # :nodoc:
     @pg = CmsPage.find_by_id(params[:id])
     target_dir = File.join(Rails.root, 'public', 'assets', 'content', @pg.path)
     
-    if File.exists?(target_dir)
+    if File.exist?(target_dir)
       redirect_to :action => 'select_gallery', :id => @pg, :gallery_id => params[:gallery_id]
     else
       render :partial => 'upload_image'
@@ -764,7 +784,7 @@ class Management::CmsController < Management::ApplicationController # :nodoc:
     
     count = 1
     localdir = File.join(target_dir, 'gallery_1')
-    while File.exists?(localdir) && count < 100
+    while File.exist?(localdir) && count < 100
       count += 1
       localdir = File.join(target_dir, "gallery_#{count}")
     end
@@ -821,7 +841,7 @@ class Management::CmsController < Management::ApplicationController # :nodoc:
       thumbfile = File.join(Rails.root, 'public', @dirname, File.basename(img, File.extname(img))) + '-thumb.jpg'
       @thumbs << File.join(@dirname, File.basename(img, File.extname(img))) + '-thumb.jpg'
       
-      next if File.exists?(thumbfile)
+      next if File.exist?(thumbfile)
       
       im = MiniMagick::Image.from_file(File.join(Rails.root, 'public', img))
       im.resize "80x80" # hardcoded!
@@ -976,41 +996,44 @@ class Management::CmsController < Management::ApplicationController # :nodoc:
       new_captions << original_captions[img.to_i] || ''
     end
     
-    File.open(File.join(gallery_dir, 'captions.yml'), 'w') { |f| YAML.dump(new_captions, f) }
+    yaml = YAML.dump(new_captions)
+    File.open(File.join(gallery_dir, 'captions.yml'), 'w') { |f| f << yaml }
     session[:gallery_images_sorted] = nil
     
     redirect_to :action => 'gallery_management', :id => @pg, :gallery_id => params[:gallery_id]
   end
   
   def image_details
-    gallery_dir = File.join(Rails.root, 'public', 'assets', 'content', params[:path].to_s, params[:gallery_id])
+    @pg = CmsPage.find_by_id(params[:id])
     
     # create blank captions.yml if it doesn't already exist
-    create_captions_file(params[:id])
+    create_captions_file(@pg.id)
     
+    gallery_dir = File.join(Rails.root, 'public', 'assets', 'content', @pg.path, params[:gallery_id])
     captions = YAML.load(File.open(File.join(gallery_dir, 'captions.yml')).read)
-    image_id = params[:image].split('.')[0].to_i
+    image_id = params[:image].split('.').first.to_i
     @caption = captions[image_id]
     
-    render :partial => 'image_details'
+    render partial: 'image_details'
   end
   
   def update_caption
     if request.post?
-      gallery_dir = File.join(Rails.root, 'public', 'assets', 'content', params[:path].to_s, params[:gallery_id])
-      
-      image_id = params[:image].split('.')[0].to_i
+      @pg = CmsPage.find_by_id(params[:id])
       
       # create blank captions.yml if it doesn't already exist
-      create_captions_file(params[:id])
+      create_captions_file(@pg.id)
       
+      gallery_dir = File.join(Rails.root, 'public', 'assets', 'content', @pg.path, params[:gallery_id])
       captions = YAML.load_file(File.join(gallery_dir, 'captions.yml')).to_a
+      image_id = params[:image].split('.').first.to_i
       captions[image_id] = params[:caption]
       
-      File.open(File.join(gallery_dir, 'captions.yml'), "w") { |f| f << captions.to_yaml }
+      yaml = YAML.dump(captions)
+      File.open(File.join(gallery_dir, 'captions.yml'), "w") { |f| f << yaml }
     end
     
-    redirect_to :action => 'gallery_management', :id => params[:id], :gallery_id => params[:gallery_id]
+    redirect_to action: 'gallery_management', id: params[:id], gallery_id: params[:gallery_id]
   end
   
   def add_to_gallery
@@ -1104,9 +1127,15 @@ class Management::CmsController < Management::ApplicationController # :nodoc:
   
   def delete_photo
     if request.post?
-      gallery_dir = File.join(Rails.root, 'public', 'assets', 'content', params[:path].to_s, params[:gallery_id])
+      @pg = CmsPage.find_by_id(params[:id])
       
-      image_id = params[:image].split('.')[0].to_i
+      # create blank captions.yml if it doesn't already exist
+      create_captions_file(@pg.id)
+      
+      gallery_dir = File.join(Rails.root, 'public', 'assets', 'content', @pg.path, params[:gallery_id])
+      captions = YAML.load(File.open(File.join(gallery_dir, 'captions.yml')).read).to_a
+      
+      image_id = params[:image].split('.').first.to_i
       
       begin ; File.delete(File.join(gallery_dir, image_id.to_s + '.jpg')) ; rescue ; end
       begin ; File.delete(File.join(gallery_dir, image_id.to_s + '-thumb.jpg')) ; rescue ; end
@@ -1114,16 +1143,12 @@ class Management::CmsController < Management::ApplicationController # :nodoc:
       
       all_images = Dir.glob(File.join(gallery_dir, '*.{jpg,jpeg,png,gif}'))
       images = []
-      all_images.each { |img| images << img if !File.basename(img).include?('thumb') and File.basename(img).split('.')[0].to_i > image_id }
+      all_images.each { |img| images << img if !File.basename(img).include?('thumb') && File.basename(img).split('.').first.to_i > image_id }
       
       image_names = []
-      images.each_with_index { |img, index| image_names << File.basename(img).split('.')[0].to_i }
+      images.each_with_index { |img, index| image_names << File.basename(img).split('.').first.to_i }
       image_names.sort!
       
-      # create blank captions.yml if it doesn't already exist
-      create_captions_file(params[:id])
-      captions = YAML.load(File.open(File.join(gallery_dir, 'captions.yml')).read).to_a
-      
       new_captions = []
       for i in 0...image_id do
         new_captions[i] = captions[i] || ''
@@ -1139,7 +1164,8 @@ class Management::CmsController < Management::ApplicationController # :nodoc:
         image_id += 1
       end
       
-      File.open(File.join(gallery_dir, 'captions.yml'), "w") { |f| f.write(YAML.dump(new_captions)) }
+      yaml = YAML.dump(new_captions)
+      File.open(File.join(gallery_dir, 'captions.yml'), "w") { |f| f << yaml }
     end
     
     redirect_to :action => 'gallery_management', :id => params[:id], :gallery_id => params[:gallery_id]
@@ -1414,16 +1440,12 @@ class Management::CmsController < Management::ApplicationController # :nodoc:
     end
     
     def create_captions_file(pg_id, options = {})
-      gallery_id = (!options[:gallery_id] ? params[:gallery_id] : options[:gallery_id])
-      
-      @pg = CmsPage.find_by_id(pg_id)
-      galleries_dir = File.join(Rails.root, 'public', 'assets', 'content', @pg.path)
-      gallery_dir = File.join(galleries_dir, gallery_id)
-      captions_location = File.join(gallery_dir, 'captions.yml')
-      
-      return if File.exists?(captions_location)
+      @pg ||= CmsPage.find_by_id(pg_id)
+      gallery_id = options[:gallery_id] || params[:gallery_id]
+      captions_file =  File.join(Rails.root, 'public', 'assets', 'content', @pg.path, gallery_id, 'captions.yml')
+      return if File.exist?(captions_file)
       
-      File.open(captions_location, 'w') { |f| YAML.dump([0], f) }
+      File.open(captions_file, 'w') { |f| YAML.dump([0], f) }
     end
     
     # prerequisites: @pg (CmsPage)
@@ -1434,7 +1456,7 @@ class Management::CmsController < Management::ApplicationController # :nodoc:
       
       ret = {}
       
-      if File.exists?(settings_location)
+      if File.exist?(settings_location)
         File.open(settings_location, 'r') { |f| ret = YAML.load(f.read) }
       else
         File.open(settings_location, 'w') { |f| YAML.dump({}, f) }
@@ -1473,7 +1495,7 @@ class Management::CmsController < Management::ApplicationController # :nodoc:
       raise "Source file required" if src_file.blank?
       
       dest = File.join(dest, File.basename(src_file)) if File.directory?(dest)
-      if !File.exists?(dest) || force == 1
+      if !File.exist?(dest) || force == 1
         logger.debug "Reading source file #{src_file}"
         im = Magick::Image::read(src_file)[0]
         im_overlay = Magick::Image::read(File.join(ImagineCms::Engine.root, 'app', 'assets', 'images', 'management', overlay))[0]
@@ -1500,7 +1522,7 @@ class Management::CmsController < Management::ApplicationController # :nodoc:
       Dir.glob("#{galleries_dir}/gallery_*").each do |g|
         begin
           management_dir = File.join(g, 'management')
-          FileUtils.mkdir_p(management_dir) unless File.exists?(management_dir)
+          FileUtils.mkdir_p(management_dir) unless File.exist?(management_dir)
           
           images = Dir.glob("#{g}/*.{jpg,jpeg,png,gif}")
           preview_images = []
@@ -1508,7 +1530,7 @@ class Management::CmsController < Management::ApplicationController # :nodoc:
           
           # gallery preview image
           preview_image_location = File.join(management_dir, 'preview.jpg')
-          unless File.exists?(preview_image_location)
+          unless File.exist?(preview_image_location)
             preview_image = preview_images.first
             create_preview_image(preview_image, preview_image_location, options[:force], 'gallery_preview_overlay.png', 130)
           end