app-info 2.5.4 → 2.6.3

data/lib/app_info/apk.rb

@@ -3,7 +3,6 @@
 require 'ruby_apk'
 require 'image_size'
 require 'forwardable'
-require 'app_info/util'
 
 module AppInfo
   # Parse APK file
@@ -24,8 +23,8 @@ module AppInfo
       @file = file
     end
 
-    def size(humanable = false)
-      AppInfo::Util.file_size(@file, humanable)
+    def size(human_size: false)
+      AppInfo::Util.file_size(@file, human_size)
     end
 
     def os
@@ -77,6 +76,7 @@ module AppInfo
     def min_sdk_version
       manifest.min_sdk_ver
     end
+    alias min_os_version min_sdk_version
 
     def target_sdk_version
       manifest.doc
@@ -110,29 +110,23 @@ module AppInfo
     end
 
     def apk
-      Zip.warn_invalid_date = false # fix invaild date format warnings
-
       @apk ||= ::Android::Apk.new(@file)
     end
 
     def icons
-      unless @icons
-        @icons = apk.icon.each_with_object([]) do |(path, data), obj|
-          icon_name = File.basename(path)
-          icon_path = File.join(contents, File.dirname(path))
-          icon_file = File.join(icon_path, icon_name)
-          FileUtils.mkdir_p icon_path
-          File.open(icon_file, 'wb') { |f| f.write(data) }
-
-          obj << {
-            name: icon_name,
-            file: icon_file,
-            dimensions: ImageSize.path(icon_file).size
-          }
-        end
+      @icons ||= apk.icon.each_with_object([]) do |(path, data), obj|
+        icon_name = File.basename(path)
+        icon_path = File.join(contents, File.dirname(path))
+        icon_file = File.join(icon_path, icon_name)
+        FileUtils.mkdir_p icon_path
+        File.write(icon_file, data, encoding: Encoding::BINARY)
+
+        obj << {
+          name: icon_name,
+          file: icon_file,
+          dimensions: ImageSize.path(icon_file).size
+        }
       end
-
-      @icons
     end
 
     def clear!
@@ -164,6 +158,7 @@ module AppInfo
     # Android Certificate
     class Certificate
       attr_reader :path, :certificate
+
       def initialize(path, certificate)
         @path = path
         @certificate = certificate
@@ -173,6 +168,7 @@ module AppInfo
     # Android Sign
     class Sign
       attr_reader :path, :sign
+
       def initialize(path, sign)
         @path = path
         @sign = sign

data/lib/app_info/core_ext/object/try.rb

@@ -4,7 +4,71 @@
 # Copy from https://github.com/rails/rails/blob/master/activesupport/lib/active_support/core_ext/object/try.rb
 
 module AppInfo
-  module Tryable #:nodoc:
+  module Tryable # :nodoc:
+    ##
+    # :method: try
+    #
+    # :call-seq:
+    #   try(*a, &b)
+    #
+    # Invokes the public method whose name goes as first argument just like
+    # +public_send+ does, except that if the receiver does not respond to it the
+    # call returns +nil+ rather than raising an exception.
+    #
+    # This method is defined to be able to write
+    #
+    #   @person.try(:name)
+    #
+    # instead of
+    #
+    #   @person.name if @person
+    #
+    # +try+ calls can be chained:
+    #
+    #   @person.try(:spouse).try(:name)
+    #
+    # instead of
+    #
+    #   @person.spouse.name if @person && @person.spouse
+    #
+    # +try+ will also return +nil+ if the receiver does not respond to the method:
+    #
+    #   @person.try(:non_existing_method) # => nil
+    #
+    # instead of
+    #
+    #   @person.non_existing_method if @person.respond_to?(:non_existing_method) # => nil
+    #
+    # +try+ returns +nil+ when called on +nil+ regardless of whether it responds
+    # to the method:
+    #
+    #   nil.try(:to_i) # => nil, rather than 0
+    #
+    # Arguments and blocks are forwarded to the method if invoked:
+    #
+    #   @posts.try(:each_slice, 2) do |a, b|
+    #     ...
+    #   end
+    #
+    # The number of arguments in the signature must match. If the object responds
+    # to the method the call is attempted and +ArgumentError+ is still raised
+    # in case of argument mismatch.
+    #
+    # If +try+ is called without arguments it yields the receiver to a given
+    # block unless it is +nil+:
+    #
+    #   @person.try do |p|
+    #     ...
+    #   end
+    #
+    # You can also call try with a block without accepting an argument, and the block
+    # will be instance_eval'ed instead:
+    #
+    #   @person.try { upcase.truncate(50) }
+    #
+    # Please also note that +try+ is defined on +Object+. Therefore, it won't work
+    # with instances of classes that do not have +Object+ among their ancestors,
+    # like direct subclasses of +BasicObject+.
     def try(method_name = nil, *args, &block)
       if method_name.nil? && block_given?
         if block.arity.zero?
@@ -17,6 +81,18 @@ module AppInfo
       end
     end
 
+    ##
+    # :method: try!
+    #
+    # :call-seq:
+    #   try!(*a, &b)
+    #
+    # Same as #try, but raises a +NoMethodError+ exception if the receiver is
+    # not +nil+ and does not implement the tried method.
+    #
+    #   "a".try!(:upcase) # => "A"
+    #   nil.try!(:upcase) # => nil
+    #   123.try!(:upcase) # => NoMethodError: undefined method `upcase' for 123:Integer
     def try!(method_name = nil, *args, &block)
       if method_name.nil? && block_given?
         if block.arity.zero?
@@ -33,82 +109,4 @@ end
 
 class Object
   include AppInfo::Tryable
-
-  ##
-  # :method: try
-  #
-  # :call-seq:
-  #   try(*a, &b)
-  #
-  # Invokes the public method whose name goes as first argument just like
-  # +public_send+ does, except that if the receiver does not respond to it the
-  # call returns +nil+ rather than raising an exception.
-  #
-  # This method is defined to be able to write
-  #
-  #   @person.try(:name)
-  #
-  # instead of
-  #
-  #   @person.name if @person
-  #
-  # +try+ calls can be chained:
-  #
-  #   @person.try(:spouse).try(:name)
-  #
-  # instead of
-  #
-  #   @person.spouse.name if @person && @person.spouse
-  #
-  # +try+ will also return +nil+ if the receiver does not respond to the method:
-  #
-  #   @person.try(:non_existing_method) # => nil
-  #
-  # instead of
-  #
-  #   @person.non_existing_method if @person.respond_to?(:non_existing_method) # => nil
-  #
-  # +try+ returns +nil+ when called on +nil+ regardless of whether it responds
-  # to the method:
-  #
-  #   nil.try(:to_i) # => nil, rather than 0
-  #
-  # Arguments and blocks are forwarded to the method if invoked:
-  #
-  #   @posts.try(:each_slice, 2) do |a, b|
-  #     ...
-  #   end
-  #
-  # The number of arguments in the signature must match. If the object responds
-  # to the method the call is attempted and +ArgumentError+ is still raised
-  # in case of argument mismatch.
-  #
-  # If +try+ is called without arguments it yields the receiver to a given
-  # block unless it is +nil+:
-  #
-  #   @person.try do |p|
-  #     ...
-  #   end
-  #
-  # You can also call try with a block without accepting an argument, and the block
-  # will be instance_eval'ed instead:
-  #
-  #   @person.try { upcase.truncate(50) }
-  #
-  # Please also note that +try+ is defined on +Object+. Therefore, it won't work
-  # with instances of classes that do not have +Object+ among their ancestors,
-  # like direct subclasses of +BasicObject+.
-
-  ##
-  # :method: try!
-  #
-  # :call-seq:
-  #   try!(*a, &b)
-  #
-  # Same as #try, but raises a +NoMethodError+ exception if the receiver is
-  # not +nil+ and does not implement the tried method.
-  #
-  #   "a".try!(:upcase) # => "A"
-  #   nil.try!(:upcase) # => nil
-  #   123.try!(:upcase) # => NoMethodError: undefined method `upcase' for 123:Integer
 end

data/lib/app_info/dsym.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 require 'macho'
-require 'app_info/util'
 
 module AppInfo
   # DSYM parser
@@ -130,8 +129,8 @@ module AppInfo
         @file.filetype
       end
 
-      def size(humanable = false)
-        return Util.size_to_humanable(@size) if humanable
+      def size(human_size: false)
+        return Util.size_to_human_size(@size) if human_size
 
         @size
       end
@@ -152,7 +151,7 @@ module AppInfo
           cpu_name: cpu_name,
           cpu_type: cpu_type,
           size: size,
-          humanable_size: size(true)
+          human_size: size(human_size: true)
         }
       end
     end

data/lib/app_info/info_plist.rb

@@ -0,0 +1,143 @@
+# frozen_string_literal: true
+
+require 'forwardable'
+require 'cfpropertylist'
+require 'app_info/png_uncrush'
+
+module AppInfo
+  # iOS Info.plist parser
+  class InfoPlist
+    extend Forwardable
+
+    def initialize(file)
+      @file = file
+    end
+
+    def version
+      release_version || build_version
+    end
+
+    def build_version
+      info.try(:[], 'CFBundleVersion')
+    end
+
+    def release_version
+      info.try(:[], 'CFBundleShortVersionString')
+    end
+
+    def identifier
+      info.try(:[], 'CFBundleIdentifier')
+    end
+    alias bundle_id identifier
+
+    def name
+      display_name || bundle_name
+    end
+
+    def display_name
+      info.try(:[], 'CFBundleDisplayName')
+    end
+
+    def bundle_name
+      info.try(:[], 'CFBundleName')
+    end
+
+    def min_os_version
+      min_sdk_version || min_system_version
+    end
+
+    #
+    # Extract the Minimum OS Version from the Info.plist (iOS Only)
+    #
+    def min_sdk_version
+      info.try(:[], 'MinimumOSVersion')
+    end
+
+    #
+    # Extract the Minimum OS Version from the Info.plist (macOS Only)
+    #
+    def min_system_version
+      info.try(:[], 'LSMinimumSystemVersion')
+    end
+
+    def icons
+      @icons ||= ICON_KEYS[device_type]
+    end
+
+    def device_type
+      device_family = info.try(:[], 'UIDeviceFamily')
+      if device_family == [1]
+        AppInfo::Device::IPHONE
+      elsif device_family == [2]
+        AppInfo::Device::IPAD
+      elsif device_family == [1, 2]
+        AppInfo::Device::UNIVERSAL
+      elsif !info.try(:[], 'DTSDKName').nil? || !info.try(:[], 'DTPlatformName').nil?
+        AppInfo::Device::MACOS
+      end
+    end
+
+    def iphone?
+      device_type == Device::IPHONE
+    end
+
+    def ipad?
+      device_type == Device::IPAD
+    end
+
+    def universal?
+      device_type == Device::UNIVERSAL
+    end
+
+    def macos?
+      device_type == Device::MACOS
+    end
+
+    def device_family
+      info.try(:[], 'UIDeviceFamily') || []
+    end
+
+    def release_type
+      if stored?
+        'Store'
+      else
+        build_type
+      end
+    end
+
+    def [](key)
+      info.try(:[], key.to_s)
+    end
+
+    def_delegators :info, :to_h
+
+    def method_missing(method_name, *args, &block)
+      info.try(:[], Util.format_key(method_name)) ||
+        info.send(method_name) ||
+        super
+    end
+
+    def respond_to_missing?(method_name, *args)
+      info.key?(Util.format_key(method_name)) ||
+        info.respond_to?(method_name) ||
+        super
+    end
+
+    private
+
+    def info
+      return unless File.file?(@file)
+
+      @info ||= CFPropertyList.native_types(CFPropertyList::List.new(file: @file).value)
+    end
+
+    def app_path
+      @app_path ||= case device_type
+                    when Device::MACOS
+                      File.dirname(@file)
+                    else
+                      File.expand_path('../', @file)
+                    end
+    end
+  end
+end

data/lib/app_info/ipa/framework.rb

@@ -3,7 +3,7 @@
 require 'forwardable'
 
 module AppInfo
-  # iOS Plugin parser
+  # iOS Framework parser
   class Framework
     extend Forwardable
 
@@ -11,7 +11,7 @@ module AppInfo
       files = Dir.glob(File.join(path, name.to_s, '*'))
       return [] if files.empty?
 
-      files.each_with_object([]) do |file, obj|
+      files.sort.each_with_object([]) do |file, obj|
         obj << new(file)
       end
     end
@@ -41,7 +41,7 @@ module AppInfo
     end
 
     def info
-      @info ||= InfoPlist.new(file)
+      @info ||= InfoPlist.new(File.join(file, 'Info.plist'))
     end
 
     def to_s