Ruby on Rails | Screencasts | Download | Documentation | Weblog | Community | Source

Ticket #9957: minor_refactoring_of_mime_type_responds.diff

File minor_refactoring_of_mime_type_responds.diff, 3.1 kB (added by josh, 1 year ago)

  • actionpack/lib/action_controller/caching.rb

    old new  
    251251 
    252252        private 
    253253          def set_content_type!(controller, extension) 
    254             controller.response.content_type = Mime::EXTENSION_LOOKUP[extension].to_s if extension 
     254            controller.response.content_type = Mime::Type.lookup_by_extension(extension).to_s if extension 
    255255          end 
    256256 
    257257          def path_options_for(controller, options) 

  • actionpack/lib/action_controller/mime_responds.rb

    old new  
    110110 
    111111    class Responder #:nodoc: 
    112112      def initialize(controller) 
    113         @controller    = controller 
    114         @request       = controller.request 
    115         @response      = controller.response 
     113        @controller = controller 
     114        @request    = controller.request 
     115        @response   = controller.response 
    116116 
    117         format = @request.parameters[:format] 
    118         @mime_type_priority = format && Mime::EXTENSION_LOOKUP[format] ? 
    119           [ Mime::EXTENSION_LOOKUP[format] ] : 
    120           @request.accepts 
     117        @mime_type_priority = Array(Mime::Type.lookup_by_extension(@request.parameters[:format]) || @request.accepts) 
    121118 
    122119        @order     = [] 
    123120        @responses = {} 
    …… 
    128125 
    129126        @order << mime_type 
    130127 
    131         if block_given? 
    132           @responses[mime_type] = Proc.new do 
    133             @response.template.template_format = mime_type.to_sym 
    134             @response.content_type = mime_type.to_s 
    135             block.call 
    136           end 
    137         else 
    138           @responses[mime_type] = Proc.new do 
    139             @response.template.template_format = mime_type.to_sym 
    140             @response.content_type = mime_type.to_s 
    141             @controller.send :render, :action => @controller.action_name 
    142           end 
     128        @responses[mime_type] = Proc.new do 
     129          @response.template.template_format = mime_type.to_sym 
     130          @response.content_type = mime_type.to_s 
     131          block_given? ? block.call : @controller.send(:render, :action => @controller.action_name) 
    143132        end 
    144133      end 
    145134 

  • actionpack/lib/action_controller/mime_type.rb

    old new  
    11module Mime 
    22  SET              = [] 
    3   EXTENSION_LOOKUP = Hash.new { |h, k| h[k] = Type.new(k) unless k == "" } 
    4   LOOKUP           = Hash.new { |h, k| h[k] = Type.new(k) unless k == "" } 
     3  EXTENSION_LOOKUP = Hash.new { |h, k| h[k] = Type.new(k) unless k.blank? } 
     4  LOOKUP           = Hash.new { |h, k| h[k] = Type.new(k) unless k.blank? } 
    55 
    66  # Encapsulates the notion of a mime type. Can be used at render time, for example, with: 
    77  #