jpmobile 5.2.4 → 6.1.1

data/lib/jpmobile/email.rb

@@ -20,7 +20,7 @@ module Jpmobile
         Mobile.carriers.each do |const|
           c = Mobile.const_get(const)
           if c::MAIL_ADDRESS_REGEXP &&
-             header.match(/(\S+@[A-Za-z0-9\-\.\_]+)/) &&
+             header.match(/(\S+@[A-Za-z0-9\-._]+)/) &&
              Regexp.last_match(1).match(/^#{c::MAIL_ADDRESS_REGEXP}$/)
             return c
           end
@@ -33,14 +33,12 @@ module Jpmobile
         nil
       end
 
-      attr_writer :japanese_mail_address_regexp
+      attr_writer :japanese_mail_address_regexp, :converting_content_type
 
       def japanese_mail?(header)
         @japanese_mail_address_regexp and header.match(@japanese_mail_address_regexp)
       end
 
-      attr_writer :converting_content_type
-
       def converting_content_type
         @converting_content_type ||= ['text/plain', 'text/html']
       end

data/lib/jpmobile/emoticon.rb

@@ -41,7 +41,7 @@ module Jpmobile
     # +str+ のなかでDoCoMo絵文字をUnicode数値文字参照に置換した文字列を返す。
     def self.external_to_unicodecr_docomo(str)
       str.gsub(DOCOMO_SJIS_REGEXP) do |match|
-        sjis = match.unpack('n').first
+        sjis = match.unpack1('n')
         unicode = DOCOMO_SJIS_TO_UNICODE[sjis]
         unicode ? ('&#x%04x;' % unicode) : match
       end
@@ -50,7 +50,7 @@ module Jpmobile
     # +str+ のなかでau絵文字をUnicode数値文字参照に置換した文字列を返す。
     def self.external_to_unicodecr_au(str)
       str.gsub(AU_SJIS_REGEXP) do |match|
-        sjis = match.unpack('n').first
+        sjis = match.unpack1('n')
         unicode = AU_SJIS_TO_UNICODE[sjis]
         unicode ? ('&#x%04x;' % unicode) : match
       end
@@ -61,7 +61,7 @@ module Jpmobile
       str = Jpmobile::Util.ascii_8bit(in_str)
       str.gsub(Jpmobile::Util.jis_string_regexp) do |jis_string|
         jis_string.gsub(/[\x21-\x7e]{2}/) do |match|
-          jis = match.unpack('n').first
+          jis = match.unpack1('n')
           unicode = AU_EMAILJIS_TO_UNICODE[jis]
           unicode ? Jpmobile::Util.ascii_8bit("\x1b\x28\x42&#x%04x;\x1b\x24\x42" % unicode) : match
         end
@@ -72,7 +72,7 @@ module Jpmobile
     def self.external_to_unicodecr_softbank(str)
       # SoftBank Unicode
       str.gsub(SOFTBANK_UNICODE_REGEXP) do |match|
-        unicode = match.unpack('U').first
+        unicode = match.unpack1('U')
         '&#x%04x;' % (unicode + 0x1000)
       end
     end
@@ -80,7 +80,7 @@ module Jpmobile
     def self.external_to_unicodecr_softbank_sjis(str)
       # SoftBank Shift_JIS
       str.gsub(SOFTBANK_SJIS_REGEXP) do |match|
-        sjis = match.unpack('n').first
+        sjis = match.unpack1('n')
         unicode = SOFTBANK_SJIS_TO_UNICODE[sjis]
         '&#x%04x;' % (unicode + 0x1000)
       end
@@ -96,20 +96,15 @@ module Jpmobile
         unicodes = match.unpack('U*')
         unicodes = unicodes.first if unicodes.size == 1
 
-        if (emoticon = UNICODE_EMOTICON_TO_CARRIER_EMOTICON[unicodes]) == GETA_CODE
-          GETA
-        elsif emoticon
-          case emoticon
-          when GETA_CODE
-            GETA
-          when Integer
-            '&#x%04x;' % emoticon
-          when String
-            emoticon
-          end
-        else
-          # 変換できなければ〓に
+        emoticon = UNICODE_EMOTICON_TO_CARRIER_EMOTICON[unicodes]
+        case emoticon
+        when GETA_CODE, nil
+          # 〓が指定されているか変換できなければ〓に
           GETA
+        when Integer
+          '&#x%04x;' % emoticon
+        when String
+          emoticon
         end
       end
     end
@@ -122,15 +117,13 @@ module Jpmobile
 
         emoticon = GOOGLE_EMOTICON_TO_CARRIER_EMOTICON[unicodes]
         case emoticon
-        when GETA_CODE
+        when GETA_CODE, nil
+          # 〓が指定されているか変換できなければ〓に
           GETA
         when Integer
           '&#x%04x;' % emoticon
         when String
           emoticon
-        else
-          # 変換できなければ〓に
-          GETA
         end
       end
     end
@@ -209,7 +202,7 @@ module Jpmobile
     # +str+ のなかでUTF-8で表記された絵文字をUnicode数値文字参照に置換する。
     def self.utf8_to_unicodecr(str)
       str.gsub(UTF8_REGEXP) do |match|
-        '&#x%04x;' % match.unpack('U').first
+        '&#x%04x;' % match.unpack1('U')
       end
     end
 

data/lib/jpmobile/filter.rb

@@ -37,6 +37,11 @@ module Jpmobile
       }
     end
 
+    ALLOWED_CONTENT_TYPE_REGEXP = Regexp.union(
+      Regexp.escape('text/html'),
+      Regexp.escape('application/xhtml+xm'),
+    )
+
     class << self
       def hankaku_format(str)
         replace_chars(str, zen_to_han_table)
@@ -49,8 +54,8 @@ module Jpmobile
       private
 
       def replace_chars(str, table)
-        @regexp_cache ||= {}
-        str.gsub(@regexp_cache[table.object_id] ||= Regexp.union(table.keys), table)
+        @regexp_cache ||= {}.compare_by_identity
+        str.gsub(@regexp_cache[table] ||= Regexp.union(table.keys), table)
       end
 
       def han_to_zen_table
@@ -90,8 +95,8 @@ module Jpmobile
     end
 
     def apply_outgoing?
-      @controller.request.mobile? and
-        [nil, 'text/html', 'application/xhtml+xml'].include?(@controller.response.content_type)
+      @controller.request.mobile? &&
+        @controller.response.content_type&.match?(ALLOWED_CONTENT_TYPE_REGEXP)
     end
 
     def to_internal(str)

data/lib/jpmobile/hook_action_view.rb

@@ -11,18 +11,12 @@ end
 # :stopdoc:
 # accept-charset に charset を変更できるようにする
 module Jpmobile
-  module ActionView
-    module HtmlOptionsWithAcceptCharset
-      private
-
-      def html_options_for_form(url_for_options, options, *parameters_for_url)
-        super.tap do |o|
-          o['accept-charset'] = (Rails.application.config.jpmobile.form_accept_charset_conversion && request && request.mobile && request.mobile.default_charset) || o['accept-charset']
-        end
+  module HtmlOptionsWithAcceptCharset
+    def html_options_for_form(url_for_options, options)
+      super.tap do |o|
+        o['accept-charset'] = (Rails.application.config.jpmobile.form_accept_charset_conversion && request && request.mobile && request.mobile.default_charset) || o['accept-charset']
       end
     end
   end
 end
-
-::ActionView::Helpers::FormTagHelper.send :prepend, Jpmobile::ActionView::HtmlOptionsWithAcceptCharset
 #:startdoc:

data/lib/jpmobile/hook_test_request.rb

@@ -1,2 +1,2 @@
 require 'action_controller/test_case'
-ActionController::TestRequest.send :include, Jpmobile::RequestWithMobileTesting
+ActionController::TestRequest.include Jpmobile::RequestWithMobileTesting

data/lib/jpmobile/mail.rb

@@ -173,7 +173,7 @@ module Mail
       text_body_part = find_part_by_content_type('text/plain').first
       html_body_part = find_part_by_content_type('text/html').first
       html_body_part.transport_encoding = 'quoted-printable' if html_body_part
-      inline_images  = []
+      inline_images = []
       attached_files = []
       attachments.each do |p|
         if p.content_type.match(%r{^image/}) && p.content_disposition.match(/^inline/)
@@ -456,17 +456,17 @@ module Mail
 
     def decoded_with_jpmobile
       if @mobile
-        return value unless value =~ /\=\?[^?]+\?([QB])\?[^?]*?\?\=/mi
+        return value unless value =~ /=\?[^?]+\?([QB])\?[^?]*?\?=/mi
 
         Encodings.collapse_adjacent_encodings(value).each {|line|
-          line.gsub!(/\=\?[^?]+\?([QB])\?[^?]*?\?\=/mi) do |string|
+          line.gsub!(/=\?[^?]+\?([QB])\?[^?]*?\?=/mi) do |string|
             case Regexp.last_match(1)
             when 'B', 'b' then decode_b_value_for_mobile(string)
             when 'Q', 'q' then Encodings.q_value_decode(string)
             else line
             end
           end
-        }.join('')
+        }.join
       else
         decoded_without_jpmobile
       end
@@ -479,7 +479,7 @@ module Mail
     alias_method :decoded, :decoded_with_jpmobile
 
     def decode_b_value_for_mobile(str)
-      match = str.match(/\=\?(.+)?\?[Bb]\?(.*)\?\=/m)
+      match = str.match(/=\?(.+)?\?[Bb]\?(.*)\?=/m)
       if match
         charset = match[1]
         str = Ruby19.decode_base64(match[2])
@@ -579,8 +579,8 @@ module Mail
 
     def get_display_name_with_jpmobile
       get_display_name_without_jpmobile
-    rescue NoMethodError => ex
-      raise ex unless ex.message.match?(/undefined method `gsub' for nil:NilClass/)
+    rescue NoMethodError => e
+      raise e unless e.message.match?(/undefined method `gsub' for nil:NilClass/)
 
       name = unquote(tree.display_name.text_value.strip.to_s)
       strip_all_comments(name.to_s)

data/lib/jpmobile/mailer.rb

@@ -1,7 +1,7 @@
 require 'jpmobile/mail'
 require 'jpmobile/lookup_context'
 
-Jpmobile::Email.japanese_mail_address_regexp = Regexp.new(/\.jp(?:[^a-zA-Z\.\-]|$)/)
+Jpmobile::Email.japanese_mail_address_regexp = Regexp.new(/\.jp(?:[^a-zA-Z.\-]|$)/)
 
 module Jpmobile
   module Mailer

data/lib/jpmobile/mobile/abstract_mobile.rb

@@ -177,7 +177,7 @@ module Jpmobile::Mobile
     end
 
     def to_mail_subject_encoded?(str)
-      str.match(/\=\?#{mail_charset}\?B.+\?\=/i)
+      str.match(/=\?#{mail_charset}\?B.+\?=/i)
     end
 
     def to_mail_body_encoded?(str)

data/lib/jpmobile/mobile/android.rb

@@ -6,6 +6,6 @@ module Jpmobile::Mobile
     include Jpmobile::Mobile::GoogleEmoticon
 
     # 対応するUser-Agentの正規表現
-    USER_AGENT_REGEXP = /Android/
+    USER_AGENT_REGEXP = /Android/.freeze
   end
 end

data/lib/jpmobile/mobile/au.rb

@@ -6,9 +6,9 @@ module Jpmobile::Mobile
   class Au < AbstractMobile
     # 対応するUser-Agentの正規表現
     # User-Agent文字列中に "UP.Browser" を含むVodafoneの端末があるので注意が必要
-    USER_AGENT_REGEXP = %r{^(?:KDDI|UP.Browser/.+?)-(.+?) }
+    USER_AGENT_REGEXP = %r{^(?:KDDI|UP.Browser/.+?)-(.+?) }.freeze
     # 対応するメールアドレスの正規表現
-    MAIL_ADDRESS_REGEXP = /.+@ezweb\.ne\.jp/
+    MAIL_ADDRESS_REGEXP = /.+@ezweb\.ne\.jp/.freeze
     # 簡易位置情報取得に対応していないデバイスID
     # http://www.au.kddi.com/ezfactory/tec/spec/eznavi.html
     LOCATION_UNSUPPORTED_DEVICE_ID = %w[PT21 TS25 KCTE TST9 KCU1 SYT5 KCTD TST8 TST7 KCTC SYT4 KCTB KCTA TST6 KCT9 TST5 TST4 KCT8 SYT3 KCT7 MIT1 MAT3 KCT6 TST3 KCT5 KCT4 SYT2 MAT1 MAT2 TST2 KCT3 KCT2 KCT1 TST1 SYT1].freeze

data/lib/jpmobile/mobile/black_berry.rb

@@ -4,6 +4,6 @@ module Jpmobile::Mobile
   # ==BlackBerry
   class BlackBerry < SmartPhone
     # 対応するUser-Agentの正規表現
-    USER_AGENT_REGEXP = /BlackBerry/
+    USER_AGENT_REGEXP = /BlackBerry/.freeze
   end
 end

data/lib/jpmobile/mobile/ddipocket.rb

@@ -4,7 +4,7 @@ module Jpmobile::Mobile
   # スーパクラスはWillcom。
   class Ddipocket < Willcom
     # 対応するUser-Agentの正規表現
-    USER_AGENT_REGEXP = %r{^Mozilla/3.0\(DDIPOCKET}
+    USER_AGENT_REGEXP = %r{^Mozilla/3.0\(DDIPOCKET}.freeze
 
     MAIL_ADDRESS_REGEXP = nil # DdipocketはEmail判定だとWillcomと判定させたい
   end

data/lib/jpmobile/mobile/docomo.rb

@@ -4,9 +4,9 @@ module Jpmobile::Mobile
   # ==DoCoMo携帯電話
   class Docomo < AbstractMobile
     # 対応するUser-Agentの正規表現
-    USER_AGENT_REGEXP = /^DoCoMo/
+    USER_AGENT_REGEXP = /^DoCoMo/.freeze
     # 対応するメールアドレスの正規表現
-    MAIL_ADDRESS_REGEXP = /.+@docomo\.ne\.jp/
+    MAIL_ADDRESS_REGEXP = /.+@docomo\.ne\.jp/.freeze
     # メールのデフォルトのcharset
     MAIL_CHARSET = 'Shift_JIS'.freeze
     # テキスト部分の content-transfer-encoding
@@ -44,9 +44,8 @@ module Jpmobile::Mobile
     # 端末製造番号があれば返す。無ければ +nil+ を返す。
     def serial_number
       case @env['HTTP_USER_AGENT']
-      when /ser([0-9a-zA-Z]{11})$/ # mova
-        Regexp.last_match(1)
-      when /ser([0-9a-zA-Z]{15});/ # FOMA
+      when /ser([0-9a-zA-Z]{11})$/, # mova
+           /ser([0-9a-zA-Z]{15});/ # FOMA
         Regexp.last_match(1)
       else
         nil
@@ -160,9 +159,8 @@ module Jpmobile::Mobile
     # モデル名を返す。
     def model_name
       case @env['HTTP_USER_AGENT']
-      when %r{^DoCoMo/2.0 (.+)\(}
-        Regexp.last_match(1)
-      when %r{^DoCoMo/1.0/(.+?)/}
+      when %r{^DoCoMo/2.0 (.+)\(},
+           %r{^DoCoMo/1.0/(.+?)/}
         Regexp.last_match(1)
       else
         nil

data/lib/jpmobile/mobile/emobile.rb

@@ -3,9 +3,9 @@
 module Jpmobile::Mobile
   # ==EMOBILE携帯電話
   class Emobile < AbstractMobile
-    USER_AGENT_REGEXP = %r{^emobile/|^Mozilla/5.0 \(H11T; like Gecko; OpenBrowser\) NetFront/3.4$|^Mozilla/4.0 \(compatible; MSIE 6.0; Windows CE; IEMobile 7.7\) S11HT$}
+    USER_AGENT_REGEXP = %r{^emobile/|^Mozilla/5.0 \(H11T; like Gecko; OpenBrowser\) NetFront/3.4$|^Mozilla/4.0 \(compatible; MSIE 6.0; Windows CE; IEMobile 7.7\) S11HT$}.freeze
     # 対応するメールアドレスの正規表現
-    MAIL_ADDRESS_REGEXP = /.+@emnet\.ne\.jp/
+    MAIL_ADDRESS_REGEXP = /.+@emnet\.ne\.jp/.freeze
     # EMnet対応端末から通知されるユニークなユーザIDを取得する。
     def em_uid
       @request.env['HTTP_X_EM_UID']

data/lib/jpmobile/mobile/ipad.rb

@@ -6,6 +6,6 @@ module Jpmobile::Mobile
     include Jpmobile::Mobile::UnicodeEmoticon
 
     # 対応するUser-Agentの正規表現
-    USER_AGENT_REGEXP = /iPad/
+    USER_AGENT_REGEXP = /iPad/.freeze
   end
 end

data/lib/jpmobile/mobile/iphone.rb

@@ -6,6 +6,6 @@ module Jpmobile::Mobile
     include Jpmobile::Mobile::UnicodeEmoticon
 
     # 対応するUser-Agentの正規表現
-    USER_AGENT_REGEXP = /iPhone/
+    USER_AGENT_REGEXP = /iPhone/.freeze
   end
 end

data/lib/jpmobile/mobile/softbank.rb

@@ -5,9 +5,9 @@ module Jpmobile::Mobile
   # Vodafoneのスーパクラス。
   class Softbank < AbstractMobile
     # 対応するuser-agentの正規表現
-    USER_AGENT_REGEXP = /^(?:SoftBank|Semulator)/
+    USER_AGENT_REGEXP = /^(?:SoftBank|Semulator)/.freeze
     # 対応するメールアドレスの正規表現　ディズニーモバイル対応
-    MAIL_ADDRESS_REGEXP = /.+@(?:softbank\.ne\.jp|disney\.ne\.jp)/
+    MAIL_ADDRESS_REGEXP = /.+@(?:softbank\.ne\.jp|disney\.ne\.jp)/.freeze
     # メールのデフォルトのcharset
     MAIL_CHARSET = 'Shift_JIS'.freeze
     # テキスト部分の content-transfer-encoding

data/lib/jpmobile/mobile/vodafone.rb

@@ -4,9 +4,9 @@ module Jpmobile::Mobile
   # スーパクラスはSoftbank。
   class Vodafone < Softbank
     # 対応するUser-Agentの正規表現
-    USER_AGENT_REGEXP = /^(Vodafone|Vemulator)/
+    USER_AGENT_REGEXP = /^(Vodafone|Vemulator)/.freeze
     # 対応するメールアドレスの正規表現
-    MAIL_ADDRESS_REGEXP = /.+@[dhtcrknsq]\.vodafone\.ne\.jp/
+    MAIL_ADDRESS_REGEXP = /.+@[dhtcrknsq]\.vodafone\.ne\.jp/.freeze
 
     # cookieに対応しているか？
     def supports_cookie?

data/lib/jpmobile/mobile/willcom.rb

@@ -4,9 +4,9 @@ module Jpmobile::Mobile
   # Ddipocketのスーパクラス。
   class Willcom < AbstractMobile
     # 対応するUser-Agentの正規表現
-    USER_AGENT_REGEXP = %r{^Mozilla/3.0\(WILLCOM}
+    USER_AGENT_REGEXP = %r{^Mozilla/3.0\(WILLCOM}.freeze
     # 対応するメールアドレスの正規表現
-    MAIL_ADDRESS_REGEXP = /.+@((.+\.)?pdx\.ne\.jp|willcom\.com)/
+    MAIL_ADDRESS_REGEXP = /.+@((.+\.)?pdx\.ne\.jp|willcom\.com)/.freeze
 
     # 位置情報があれば Position のインスタンスを返す。無ければ +nil+ を返す。
     def position

data/lib/jpmobile/mobile/windows_phone.rb

@@ -4,6 +4,6 @@ module Jpmobile::Mobile
   # ==WindowsPhone
   class WindowsPhone < SmartPhone
     # 対応するUser-Agentの正規表現
-    USER_AGENT_REGEXP = /Windows Phone/
+    USER_AGENT_REGEXP = /Windows Phone/.freeze
   end
 end

data/lib/jpmobile/rack/params_filter.rb

@@ -10,10 +10,11 @@ module Jpmobile
       if (mobile = env['rack.jpmobile']) && mobile.apply_params_filter?
         # パラメータをkey, valueに分解
         # form_params
-        unless env['REQUEST_METHOD'] == 'GET' || env['REQUEST_METHOD'] == 'HEAD'
-          unless env['CONTENT_TYPE'].match?(%r{application/json|application/xml})
-            env['rack.input'] = StringIO.new(parse_query(env['rack.input'].read, mobile))
-          end
+        if env['REQUEST_METHOD'] != 'GET' &&
+           env['REQUEST_METHOD'] != 'HEAD' &&
+           !env['CONTENT_TYPE'].match?(%r{application/json|application/xml})
+
+          env['rack.input'] = StringIO.new(parse_query(env['rack.input'].read, mobile))
         end
 
         # query_params

data/lib/jpmobile/rails.rb

@@ -4,24 +4,26 @@ ActiveSupport.on_load(:action_controller) do
   require 'jpmobile/helpers'
   require 'jpmobile/trans_sid'
   require 'jpmobile/hook_test_request'
-  ActionDispatch::Request.send :prepend, Jpmobile::Encoding
-  ActionDispatch::Request.send :include, Jpmobile::RequestWithMobile
-  ActionController::Base.send :prepend, Jpmobile::FallbackViewSelector
-  ActionController::Base.send :prepend, Jpmobile::TransSidRedirecting
+  ActionDispatch::Request.prepend Jpmobile::Encoding
+  ActionDispatch::Request.include Jpmobile::RequestWithMobile
+  ActionController::Base.prepend Jpmobile::FallbackViewSelector
+  ActionController::Base.prepend Jpmobile::TransSidRedirecting
 end
 
 ActiveSupport.on_load(:action_view) do
   require 'jpmobile/hook_action_view'
+
+  self.prepend Jpmobile::HtmlOptionsWithAcceptCharset
 end
 
 ActiveSupport.on_load(:after_initialize) do
   case Rails.application.config.session_store.to_s
   when 'ActionDispatch::Session::MemCacheStore'
     require 'jpmobile/session/mem_cache_store'
-    ActionDispatch::Session::MemCacheStore.send :prepend, Jpmobile::ParamsOverCookie
+    ActionDispatch::Session::MemCacheStore.prepend Jpmobile::ParamsOverCookie
   when 'ActionDispatch::Session::ActiveRecordStore'
     require 'jpmobile/session/active_record_store'
-    ActionDispatch::Session::AbstractStore.send :prepend, Jpmobile::ParamsOverCookie
+    ActionDispatch::Session::AbstractStore.prepend Jpmobile::ParamsOverCookie
   else
     Rails.application.config.jpmobile.mount_session_store
   end
@@ -31,39 +33,5 @@ ActiveSupport.on_load(:before_configuration) do
   # MobileCarrierのみデフォルトで有効
   config.middleware.insert_after ActionDispatch::Flash, ::Jpmobile::MobileCarrier
 
-  module Rails
-    class Application
-      class Configuration
-        def jpmobile
-          @jpmobile ||= ::Jpmobile.config
-        end
-      end
-    end
-  end
-end
-
-module Jpmobile
-  module ViewSelector
-    def self.included(base)
-      base.class_eval do
-        before_action :register_mobile
-
-        self._view_paths = self._view_paths.dup
-        self.view_paths.unshift(*self.view_paths.map {|resolver| Jpmobile::Resolver.new(resolver.to_path) })
-      end
-    end
-
-    def register_mobile
-      if request.mobile
-        # register mobile
-        self.lookup_context.mobile = request.mobile.variants
-      end
-    end
-
-    def disable_mobile_view!
-      self.lookup_context.mobile = []
-    end
-
-    private :register_mobile, :disable_mobile_view!
-  end
+  Rails::Application::Configuration.include Jpmobile::Configuration::RailsConfiguration
 end