Changeset 1631

Show
Ignore:
Timestamp:
02/13/08 08:06:18 (3 months ago)
Author:
pdcawley
Message:

Applied some optimization "guerilla" patches and tightened up the order sanitization regexes.

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved
  • trunk/app/controllers/admin/content_controller.rb

    r1552 r1631  
    88 
    99  def list 
    10     if params[:order] and params[:order] =~ /title|created_at|author|state
     10    if params[:order] and params[:order] =~ /\A(?:title|created_at|author|state)\Z
    1111      if params[:sense] and params[:sense] == 'desc' 
    1212        order = params[:order] + " asc" 
    1313      else 
    14         order = params[:order] + " desc"         
     14        order = params[:order] + " desc" 
    1515      end 
    1616    else 
  • trunk/app/controllers/admin/pages_controller.rb

    r1552 r1631  
    88 
    99  def list 
    10     if params[:order] and params[:order] =~ /title|created_at|state
     10    if params[:order] and params[:order] =~ /\A(?:title|created_at|state)\Z
    1111      if params[:sense] and params[:sense] == 'desc' 
    1212        order = params[:order] + " asc" 
    1313      else 
    14         order = params[:order] + " desc"         
     14        order = params[:order] + " desc" 
    1515      end 
    1616    else