mcms_pages 0.0.1 → 0.0.2

data/app/controllers/admin/pages_controller.rb

@@ -17,6 +17,7 @@ class Admin::PagesController < AdminController
 
   # loading controller specific assets
   before_filter :load_assets
+  after_filter :clean_pages, :only => [:update]
 
   # GET /pages
   # GET /pages.json
@@ -85,15 +86,20 @@ class Admin::PagesController < AdminController
     #@page = Page.find(params[:id])
     @page = Page.find_by_slug_or_id(params[:path], params[:id])
 
-    #retrieving layout id from params and assigning to a variable
-    @layout = @page["layout_id"]
+    if @page.nil?
+
+      raise ActiveRecord::RecordNotFound
+
+    else
+
+     #retrieving layout id from params and assigning to a variable
+     @layout = @page["layout_id"]
+
+    end # end if
 
     # calling method to find all pages and page parts
     find_pages_and_parts(@layout)
 
-    # calling method to define parts of the page
-    define_parts
-
     #sends data in different formats
     respond_to do |format|
 
@@ -135,7 +141,7 @@ class Admin::PagesController < AdminController
 
       else #page saving failed, re-render the form
 
-        format.html { render action: "new" }
+        format.html { render action: "new", :template => 'mcms_pages/admin/pages/new' }
         format.json { render json: @page.errors, status: :unprocessable_entity }
 
       end # end if
@@ -153,15 +159,16 @@ class Admin::PagesController < AdminController
     #find the page with given id
     @page = Page.find(params[:id])
 
+    #use updated_at manually, it will update the page table if user hits update button.
+    # This may not be trivial, so it may change.
+    @page.updated_at = Time.now
+
     #retrieving layout id from params and assigning to a variable
     @layout = @page[:layout_id]
 
     # calling method to find all pages and page parts
     find_pages_and_parts(@layout)
 
-    # calling method to define parts of the page
-    define_parts
-
     #sends in data in different format
     respond_to do |format|
 
@@ -169,7 +176,7 @@ class Admin::PagesController < AdminController
         format.html { redirect_to admin_pages_path, notice: 'Page was successfully updated.' }
         format.json { head :no_content }
       else #page saving failed, re-renders edit template
-        format.html { render action: "edit" }
+        format.html { render action: "edit", :template => 'mcms_pages/admin/pages/edit' }
         format.json { render json: @page.errors, status: :unprocessable_entity }
 
       end # end if
@@ -187,6 +194,8 @@ class Admin::PagesController < AdminController
     # find the page with with
     @page = Page.find(params[:id])
 
+    PagePart.delete_page_parts(@page)
+
     #destroy the page
     @page.destroy
 
@@ -209,18 +218,26 @@ class Admin::PagesController < AdminController
     @page = Page.find_by_slug(params[:path])
 
     logger.debug @page
-    render :layout => false
+
+    #sends in data in different format to index action
+    respond_to do |format|
+
+      format.html { render :template => 'mcms_pages/admin/pages/find_child', :layout => false }
+      format.json { head :no_content }
+
+    end # end respond_to block
 
   end
 
-  # @param    : Object(as a string)
+  # @param    : string
   # @return   : None
-  # @purpose  : To render a page part dynamically
+  # @purpose  : To render a page part pages dynamically
   def add_page_part
 
     #retrieve parameters and assign these to  instance variables
     @title = params[:title]
     @index = params[:index]
+    @part = params[:part]
 
     # render js.erb
     respond_to do |format|
@@ -239,10 +256,10 @@ class Admin::PagesController < AdminController
   def load_assets
 
     # class method call to include js files
-    AssetManager.include_local_library [:application, :pageFormAdmin, "ckeditor/init"]
+    Asset.include_local_library [:application, :pageFormAdmin, "ckeditor/init"]
 
     # class method call to include css files
-    AssetManager.include_css ["mcms_pages/pages", "mcms_pages/page_styles"]
+    Asset.include_css ["mcms_pages/pages", "mcms_pages/page_form", "mcms_pages/tabs"]
 
   end # end method
 
@@ -254,7 +271,7 @@ class Admin::PagesController < AdminController
     #renders the form field for each part of layout
     parts.each_with_index do |page_part, index|
 
-      fragment = get_fragments(index, page_part)
+      fragment = get_fragments(page_part)
 
       # check if same type of fragment is already created
       if fragment.nil? or fragment.blank? or fragment.empty?
@@ -275,29 +292,38 @@ class Admin::PagesController < AdminController
 
   def find_pages_and_parts layout
 
-    #calling find_all_pages method from Page model
-    @pages = Page.find_all_pages
-
     # storing the array returned by helper method to
     # decide the page parts in a layout
     @parts = Layout.find_page_parts(layout)
 
-  end
+  end # end method
 
-  def define_parts
+  # @param    : None
+  # @return   : None
+  # @purpose  : To delete orphan record
+  def clean_pages
 
-    # declare an empty hash
-    @part_title = Hash.new
+    # find all records in the table mcms_page_parts
+    page_parts = PagePart.all
 
-    # loop through each part of the page
-    @parts.each_with_index do  |part,index|
+    # loop through all page_parts array
+    page_parts.each do |page_part|
 
-      # retrieve the id of page parts of the page based on title and store it as a hash element
-      @part_title[part] = @page.page_parts.select("mcms_page_parts.id").find_by_title(part).id
+      # find page_part_page corresponding to a page_part
+      page_part_page = PagePartsPage.where("page_part_id = ?", page_part.id).first
+
+      # check if there is no page_part_page corresponding to a page_part
+      if page_part_page.nil? or page_part_page.blank?
+
+        # find page part with relevent id and delete it
+        part = PagePart.find(page_part.id)
+        part.destroy
+
+      end # end if
 
     end # end loop
 
-  end
+  end # end method
 
 end #end class
 

data/app/controllers/admin_controller.rb

@@ -13,10 +13,44 @@
 class AdminController < ApplicationController
 
   #admin part
-  layout "mcms_pages/layouts/mcms_layout"
+  # check if mcms core layout exists
+  if Gem.available?('mcms')
 
+   #if yes then use core layout
+   layout "mcms/main_layout"
+
+  else # if not
+
+    # use page's default layout
+    layout "mcms_pages/layouts/mcms_layout"
+
+  end # end if
+
+  # use before filter to authenticate user
   prepend_before_filter :authenticate_user!
+
+  # check authorization for a user
   load_and_authorize_resource
 
-end
+  # rescue from record not found exception
+  rescue_from ActiveRecord::RecordNotFound, :with => :render_404
+
+
+  # @params   : None
+  # @return   : None
+  # @purpose  : To render 404.html from public folder
+  def render_404
+
+    # render data in different format
+    respond_to do |format|
+
+      format.html { render :file => "#{Rails.root}/public/404.html", :status => :not_found }
+      format.xml  { head :not_found }
+      format.any  { head :not_found }
+
+    end # end respond_to block
+
+  end # end method
+
+end # end class
 

data/app/controllers/pages_controller.rb

@@ -13,8 +13,6 @@
 
 class PagesController < ApplicationController
 
-  # loading controller specific assets
-  before_filter :load_assets
   layout "mcms_pages/layouts/mcms_layout"
 
   #finding page based on url
@@ -85,6 +83,19 @@ class PagesController < ApplicationController
       # Preview a non-persisted page
       @page = Page.new(params[:page])
 
+      # initializing a loop variable
+      i = @page.page_parts.length
+
+      # looping through page_parts_pages to retrieve all page_parts
+      params[:page]["page_parts_pages_attributes"].each do |k,v|
+
+        # retrieve existing page_parts from DB and assign it to @page array
+        @page.page_parts[i] = PagePart.find_by_id(v["page_part_id"]);
+
+        i = i + 1 # incrementing the loop variable
+
+      end # end loop
+
     end # end if
 
     # send data in different format
@@ -100,20 +111,12 @@ class PagesController < ApplicationController
   #defining controller specific method as protected
   protected
 
-  #calling library class AssetManager's methods to include specific js and css files
-  def load_assets
-
-    # class method call to include css files
-    AssetManager.include_css ["mcms_pages/pages", "mcms_pages/page_styles"]
-
-  end # end method
-
   # @param   : boolean(optional)
   # @return  : none
   # @purpose : To find a page if it exists or show the page not found error
   def get_page(page_not_found = true)
 
-    @menu_pages = Page.where(:published => true)
+    @menu_pages = Page.where("published = ? and show_in_menu = ?", true, true)
 
     # case statement based on current action and assign the result to instance variable
     case action_name
@@ -145,6 +148,15 @@ class PagesController < ApplicationController
     # find page based on link_url and assign it to an instance variable
     @page = Page.where(:link_url => "/404").first
 
+     # send data in different format
+     respond_to do |format|
+
+       format.html{ render :template => 'mcms_pages/pages/show'}# show.html.erb
+       format.json { render json: @page }
+
+     end # end respond_to block
+
+
   end # end method
 
 end #end class

data/app/helpers/pages_helper.rb

@@ -11,7 +11,7 @@
 =end
 
 module PagesHelper
-  
+
   # @Parameters  : integer(column count of a layout)
   # @Return      : Array
   # @purpose     : To give layout a name on the basis of parameter
@@ -23,19 +23,19 @@ module PagesHelper
       when 2
 
         return "two_column_layout"
-          
+
       when 3
 
         return "three_column_layout"
-        
+
       when 1
 
         return "one_column_layout"
 
       else
-        
+
         return "default_layout"
-        
+
     end # end case statement
 
   end #end method
@@ -49,70 +49,78 @@ module PagesHelper
 
   end
 
-
   # @param    : Integer
   # @return   : Array
   # @purpose  : To find page parts according to param
-  def get_fragments(part, title)
+  def get_fragments title
 
-    #using case statement to get fragments
-    case part
+     if title != "main body"
 
-      when 0
+       # get page body fragment with title
+       part = PagePart.find_page_parts_with_title(title)
 
-        if title == "header"
-          
-          # get header fragment
-          part = PagePart.find_page_parts_with_title_header
+     end
 
-        end
-        
-      when 1
-        
-        # get footer fragment
-        part = PagePart.find_page_parts_with_title_footer
+  end # end method
 
-      when 2
+  # @param    : Integer(part_index of page part)
+  # @return   : Object
+  # @purpose  : To build a form object for dynamic creating the page part
+  def build_page_part index
 
-        if title == "left body"
-          
-          # get left body fragment
-          part = PagePart.find_page_parts_with_title_left_body
-
-        elsif title == "right body"
-          
-          # get right body fragment
-          part = PagePart.find_page_parts_with_title_right_body
-          
-        end
-        
-      when 3
-        
-        if title == "right body"
+    # create a PagePart object and assign it to a variable
+    page = PagePart.new
 
-          # get right body fragment
-          part = PagePart.find_page_parts_with_title_right_body
+    #make the page part form
+    builder = ActionView::Helpers::FormBuilder.new( "page[page_parts_attributes][#{index}]", page, self, {}, proc {})
 
-        end
-      
-    end # end case
+    #return form object
+    return builder
 
   end # end method
 
   # @param    : Integer(part_index of page part)
   # @return   : Object
   # @purpose  : To build a form object for dynamic creating the page part
-  def build_page_part index
+  def build_page_part_page index
 
     # create a PagePart object and assign it to a variable
-    page = PagePart.new
+    page = PagePartsPage.new
 
     #make the page part form
-    builder = ActionView::Helpers::FormBuilder.new( "page[page_parts_attributes][#{index}]", page, self, {}, proc {})
+    builder = ActionView::Helpers::FormBuilder.new( "page[page_parts_pages_attributes][#{index}]", page, self, {}, proc {})
 
     #return form object
     return builder
 
   end # end method
 
+  # @param    : Array(page array to be arranged)
+  # @return   : Hash
+  # @purpose  : To arrange the result and store it in a hash to display the value properly
+  def arrange_page_part page
+
+    # find the length of page_parts of current page
+    part_length = page.page_parts.length
+
+    # create an empty hash
+    part_pages = Hash.new
+
+    #loop through the length of page_part
+    for i in 0..part_length-1
+
+      #find each page_part array and assign it in a variable
+      part = page.page_parts[i]
+
+      #assign page_part array to hash
+      part_pages[part.title] = part
+
+    end # end loop
+
+    # return hash
+    part_pages
+
+  end # end method
+
 end # end module
+

data/app/models/ckeditor/asset.rb

@@ -1,6 +1,7 @@
 class Ckeditor::Asset < ActiveRecord::Base
 
-  self.table_name = "mcms_assets"
+  #self.table_name = "mcms_assets"
+  set_table_name "mcms_assets"
 
   include Ckeditor::Orm::ActiveRecord::AssetBase
   include Ckeditor::Backend::Paperclip

data/app/models/page.rb

@@ -32,8 +32,12 @@ class Page < ActiveRecord::Base
   #include rails standard validation for title
   validates :title, :presence => true, :uniqueness => true
 
+  # validating link url using rails standard validator with regular expression
+  validates_format_of :link_url,
+            :with => /^(http|https):\/\/[a-z0-9]+([\-\.]{1}[a-z0-9]+)*\.[a-z]{2,5}(:[0-9]{1,5})?(\/.*)?$/ix unless :link_url.length > 0
+
   # Nested pages, i.e. page underneath another page
-  #acts_as_nested_set :dependent => :destroy
+  acts_as_nested_set :dependent => :destroy
 
   #Associating table "mcms_pages" with table "mcms_page_parts" using has_many relation
 #  has_many :page_parts,

data/app/models/page_part.rb

@@ -28,72 +28,53 @@ class PagePart < ActiveRecord::Base
   #validating title field to have non-blank value
   validates :title, :presence  => true
 
-  # @param    : none
+  # @param    : string(title of the page part)
   # @return   : array
-  # @purpose  : To find all page_parts with title header
-  def self.find_page_parts_with_title_header
+  # @purpose  : To find all page_parts with given title
+  def self.find_page_parts_with_title title
 
     #find all page parts with title header and assign it to a variable
-    header_part = self.find_all_by_title("header")
+    part_body = self.find_all_by_title(title)
 
     # calling method to find unique pages and assigning them to an array
-    page_title = PagePartsPage.find_unique_page_parts_pages(header_part)
+    page_title = PagePartsPage.find_unique_page_parts_pages(part_body)
 
     # returning the array
     return page_title
-    
+
   end # end method
 
   # @param    : none
   # @return   : array
-  # @purpose  : To find all page_parts with title header
-  def self.find_page_parts_with_title_footer
-
-    #find all page parts with title header and assign it to a variable
-    footer_part = self.find_all_by_title("footer")
-
-    # calling method to find unique pages and assigning them to an array
-    page_title = PagePartsPage.find_unique_page_parts_pages(footer_part)
+  # @purpose  : It will just return the title of page part
+  def self.find_page_parts_with_title_main_body
 
-    # returning the array
-    return page_title
+    return "main body"
 
   end # end method
 
-  # @param    : none
-  # @return   : array
-  # @purpose  : To find all page_parts with title right body
-  def self.find_page_parts_with_title_right_body
-
-     #find all page parts with title header and assign it to a variable
-    right_body = self.find_all_by_title("right body")
+  # @param    : Object
+  # @return   : none
+  # @purpose  : To find all independent page_parts and delete it
+  def self.delete_page_parts page
 
-    # calling method to find unique pages and assigning them to an array
-    page_title = PagePartsPage.find_unique_page_parts_pages(right_body)
+    # loop through page_parts_pages of page and check its page part
+    page.page_parts_pages.each do |page_part_page|
 
-    # returning the array
-    return page_title
+      # find page_part by its id and assign it to a variable
+      page_part = self.find_by_id(page_part_page.page_part_id)
 
-  end # end method
+      # if there is only one page_part for a page then delete it
+      if page_part.pages.count == 1 and page_part.pages.first.title == page.title
 
-  # @param    : none
-  # @return   : array
-  # @purpose  : To find all page_parts with title left body
-  def self.find_page_parts_with_title_left_body
+        # delete page_part
+        page_part.destroy
 
-     #find all page parts with title header and assign it to a variable
-    left_body = self.find_all_by_title("left body")
+      end # end if
 
-    # calling method to find unique pages and assigning them to an array
-    page_title = PagePartsPage.find_unique_page_parts_pages(left_body)
-
-    # returning the array
-    return page_title
+    end # end loop
 
   end # end method
 
-  def self.find_page_parts_with_title_main_body
-    return "main body"
-  end
-  
-end  # end class
+end  # end class
+

data/app/views/mcms_pages/admin/layouts/index.html.erb

@@ -4,7 +4,7 @@
   @Company Name              : Mindfire Solutions Private Limited
   @Creator Name              : Vikram Kumar Mishra
   @Date Created              : 2012-06-15
-  @Date Modified             : 
+  @Date Modified             :
   @Last Modification Details :
   @Purpose                   : To display all the layouts and respective image and let the user choose the specific layout.
 
@@ -30,19 +30,19 @@
       <div class="layout-image">
 
         <%= image_tag("layouts/#{layout.name.tr(' ', '-')}.png", :size => "400x400", :alt => "#{layout.name}", :title => "click to view large") %>
-        
+
       </div>
       <div class="layout-link">
 
         <!-- if parent_id is not 0 then send parent_id through url -->
         <% if not @page_parent_id == 0 %>
-        
+
           <%= link_to("Use This Layout", new_admin_page_path(:layout_id => "#{layout.id}", :parent_id => "#{@page_parent_id}")) %>
 
         <% else %>
-          
+
           <%= link_to("Use This Layout", new_admin_page_path(:layout_id => "#{layout.id}")) %>
-        
+
         <% end %>
 
       </div>
@@ -51,4 +51,7 @@
 
   <% end %>
 
-</div>
+</div>
+
+<%= render :partial => "mcms_pages/layouts/head" %>
+

data/app/views/mcms_pages/admin/pages/_action.html.erb

@@ -11,9 +11,28 @@
 -->
 
 
-<section id="application-actions">
+<div class="repos application-actions">
 
-  <%= link_to "Add a New Page", new_admin_page_path, :id => "page-new" %>
+  <div class="top-bar">
 
-</section>
+    <h2>Quick tasks </h2>
+    <%= link_to "Add a New Page", new_admin_page_path, :id => "page-new", :class =>"button new-repo" %>
+
+  </div>
+
+  <ul class="repo-list">
+
+    <li class="public source">
+      <a href="/" target="_blank">
+        <span class="mini-icon"></span>
+        <span class="owner">Switch to Website</span><span class="repo"></span>
+        <span class="arrow"></span>
+      </a>
+    </li>
+  </ul>
+
+  <div class="bottom-bar">
+  </div>
+
+</div>