activerecord-mysql2rgeo-adapter 2.0.3 → 5.2.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 347e7b8819994c6d6759e88a5ab6af2b580650bd
-  data.tar.gz: 8e1aba30818aae96723909abb47da9f304cb7896
+SHA256:
+  metadata.gz: 8be8c0bd91551b3593b6be777f8a05243fc136f79e01234678550a09233bb607
+  data.tar.gz: ce6f2c3b063af63d66a8779f007c8367639c8d3f8dea4225680477f0a48ff5eb
 SHA512:
-  metadata.gz: c91c8e58ceeef1a4d0482969d52df5abd15008b97580804ef7fde65a4dd6377b8d91f2adc289957d5ec5211ecc5c15862440412f6851d8efa7494c8ad8fd49ba
-  data.tar.gz: 9f772f1d2501e46e9cf7a0982ef73fa06809b17aad795e2ecb6e5307a3f538f1f2e56b904216ca2d50fb9f665e26d702716f0b8d3b11033a270d4fb75996f1b1
+  metadata.gz: 0aa088c4492c15cdab24145b41c3ba37dea76e2b298a03332445698f0b278f2be7c5ed506249d8b4bfe5f214c2d1bd8566eec9e9dabcbcf6a093d3928125e198
+  data.tar.gz: 4cdb993e6a3893cfaa57ad38d21618cbce0ea303d05914586ccf488e2907bab1c2a4aaa2be2bbee0eb9819850173c1d437592fca8c73c92f806f26e387062c83

data/lib/active_record/connection_adapters/mysql2rgeo/arel_tosql.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Arel  # :nodoc:
   module Visitors  # :nodoc:
     # Different super-class under JRuby JDBC adapter.
@@ -5,7 +7,7 @@ module Arel  # :nodoc:
                         ::ArJdbc::MySQL::BindSubstitution
                       else
                         MySQL
-                        end
+                      end
 
     class Mysql2Rgeo < MySQLSuperclass  # :nodoc:
       include RGeo::ActiveRecord::SpatialToSql

data/lib/active_record/connection_adapters/mysql2rgeo/column_methods.rb

@@ -1,51 +1,54 @@
+# frozen_string_literal: true
+
 module ActiveRecord
   module ConnectionAdapters
     module Mysql2Rgeo
       module ColumnMethods
         def spatial(name, options = {})
-          raise "You must set a type. For example: 't.spatial :object1, limit: { type: 'point' }'" if options[:limit].blank? || options[:limit][:type].blank?
-          column(name, options[:limit][:type], options)
-        end
+          raise "You must set a type. For example: 't.spatial type: :point'" unless options[:type]
 
-        def geometry(*args, multi: false, **options)
-          multi ? multi_geometry(*args, **options) : args.each { |name| column(name, :geometry, options) }
+          column(name, options[:type], options)
         end
 
-        def geometrycollection(*args, **options)
-          args.each { |name| column(name, :geometrycollection, options) }
+        def geometry(name, options = {})
+          column(name, :geometry, options)
         end
 
-        def point(*args, multi: false, **options)
-          multi ? multi_point(*args, **options) : args.each { |name| column(name, :point, options) }
+        def geometry_collection(name, options = {})
+          column(name, :geometrycollection, options)
         end
+        alias geometrycollection geometry_collection
 
-        def multipoint(*args, **options)
-          args.each { |name| column(name, :multipoint, options) }
+        def line_string(name, options = {})
+          column(name, :linestring, options)
         end
+        alias linestring line_string
 
-        def linestring(*args, multi: false, **options)
-          multi ? multi_linestring(*args, **options) : args.each { |name| column(name, :linestring, options) }
+        def multi_line_string(name, options = {})
+          column(name, :multilinestring, options)
         end
+        alias multilinestring multi_line_string
 
-        def multilinestring(*args, **options)
-          args.each { |name| column(name, :multilinestring, options) }
+        def multi_point(name, options = {})
+          column(name, :multipoint, options)
         end
+        alias multipoint multi_point
 
-        def polygon(*args, multi: false, **options)
-          multi ? multipolygon(*args, **options) : args.each { |name| column(name, :polygon, options) }
+        def multi_polygon(name, options = {})
+          column(name, :multipolygon, options)
         end
+        alias multipolygon multi_polygon
 
-        def multipolygon(*args, **options)
-          args.each { |name| column(name, :multipolygon, options) }
+        def point(name, options = {})
+          column(name, :point, options)
         end
 
-        alias multi_point multipoint
-        alias multi_geometry geometrycollection
-        alias multi_linestring multilinestring
-        alias multi_polygon multipolygon
+        def polygon(name, options = {})
+          column(name, :polygon, options)
+        end
       end
-
-      MySQL::Table.send(:include, ColumnMethods)
     end
+
+    MySQL::Table.include Mysql2Rgeo::ColumnMethods
   end
 end

data/lib/active_record/connection_adapters/mysql2rgeo/create_connection.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 if RUBY_ENGINE == "jruby"
   require "active_record/connection_adapters/jdbcmysql_adapter"
 else
@@ -13,7 +15,7 @@ module ActiveRecord # :nodoc:
         mysql2_connection(config)
       end
 
-      alias jdbcmysql2rgeo_connection mysql2rgeo_connection
+      alias_method :jdbcmysql2rgeo_connection, :mysql2rgeo_connection
 
     else
 
@@ -29,7 +31,7 @@ module ActiveRecord # :nodoc:
 
         if Mysql2::Client.const_defined? :FOUND_ROWS
           if config[:flags].is_a? Array
-            config[:flags].push "FOUND_ROWS".freeze
+            config[:flags].push "FOUND_ROWS"
           else
             config[:flags] |= Mysql2::Client::FOUND_ROWS
           end

data/lib/active_record/connection_adapters/mysql2rgeo/schema_statements.rb

@@ -1,16 +1,21 @@
+# frozen_string_literal: true
+
 module ActiveRecord
   module ConnectionAdapters
     module Mysql2Rgeo
       module SchemaStatements
         # override
-        def new_column(*args)
-          SpatialColumn.new(*args)
+        def indexes(table_name) #:nodoc:
+          indexes = super
+          # HACK(aleks, 06/15/18): MySQL 5 does not support prefix lengths for spatial indexes
+          # https://dev.mysql.com/doc/refman/5.6/en/create-index.html
+          indexes.select { |idx| idx.type == :spatial }.each { |idx| idx.is_a?(Struct) ? idx.lengths = {} : idx.instance_variable_set(:@lengths, {}) }
+          indexes
         end
 
         def type_to_sql(type, limit: nil, precision: nil, scale: nil, unsigned: nil, **) # :nodoc:
           if (info = RGeo::ActiveRecord.geometric_type_from_name(type.to_s.delete("_")))
             type = limit[:type] || type if limit.is_a?(::Hash)
-            type = :geometry if type.eql? :spatial
             type = type.to_s.delete("_").upcase
           end
           super
@@ -19,32 +24,28 @@ module ActiveRecord
         # override
         def native_database_types
           # Add spatial types
+          # Reference: https://dev.mysql.com/doc/refman/5.6/en/spatial-type-overview.html
           super.merge(
-            geometry: { name: "geometry" },
-            point: { name: "point" },
-            linestring: { name: "linestring" },
-            polygon: { name: "polygon" },
-            multi_geometry: { name: "geometrycollection" },
-            multi_point: { name: "multipoint" },
-            multi_linestring: { name: "multilinestring" },
-            multi_polygon: { name: "multipolygon" },
-            spatial: { name: "geometry", limit: { type: :point } }
+            geometry:            { name: "geometry" },
+            geometrycollection:  { name: "geometrycollection" },
+            linestring:          { name: "linestring" },
+            multi_line_string:   { name: "multilinestring" },
+            multi_point:         { name: "multipoint" },
+            multi_polygon:       { name: "multipolygon" },
+            spatial:             { name: "geometry" },
+            point:               { name: "point" },
+            polygon:             { name: "polygon" }
           )
         end
 
-        # override
-        def create_table_definition(*args)
-          Mysql2Rgeo::TableDefinition.new(*args)
-        end
-
-        def initialize_type_map(m)
+        def initialize_type_map(m = type_map)
           super
           %w(
             geometry
+            geometrycollection
             point
             linestring
             polygon
-            geometrycollection
             multipoint
             multilinestring
             multipolygon
@@ -52,6 +53,34 @@ module ActiveRecord
             m.register_type(geo_type, Type::Spatial.new(geo_type))
           end
         end
+
+        private
+
+        # override
+        def create_table_definition(*args)
+          Mysql2Rgeo::TableDefinition.new(*args)
+        end
+
+        # override
+        def new_column_from_field(table_name, field)
+          type_metadata = fetch_type_metadata(field[:Type], field[:Extra])
+          if type_metadata.type == :datetime && /\ACURRENT_TIMESTAMP(?:\([0-6]?\))?\z/i.match?(field[:Default])
+            default, default_function = nil, field[:Default]
+          else
+            default, default_function = field[:Default], nil
+          end
+
+          SpatialColumn.new(
+              field[:Field],
+              default,
+              type_metadata,
+              field[:Null] == "YES",
+              table_name,
+              default_function,
+              field[:Collation],
+              comment: field[:Comment].presence
+          )
+        end
       end
     end
   end

data/lib/active_record/connection_adapters/mysql2rgeo/spatial_column.rb

@@ -11,8 +11,7 @@ module ActiveRecord # :nodoc:
             # @geometric_type = geo_type_from_sql_type(sql_type)
             build_from_sql_type(sql_type_metadata.sql_type)
           end
-          super(name, default, sql_type_metadata, null, table_name, default_function, collation)
-          @comment = comment
+          super(name, default, sql_type_metadata, null, table_name, default_function, collation, comment: comment)
           if spatial?
             if @srid
               @limit = { type: @geometric_type.type_name.underscore }
@@ -35,9 +34,9 @@ module ActiveRecord # :nodoc:
           false
         end
 
+        alias :geographic? :geographic
         alias :has_z? :has_z
         alias :has_m? :has_m
-        alias :geographic? :geographic
 
         def limit
           if spatial?
@@ -66,7 +65,7 @@ module ActiveRecord # :nodoc:
         end
 
         def build_from_sql_type(sql_type)
-          geo_type, @srid, @has_z, @has_m = Type::Spatial.parse_sql_type(sql_type)
+          geo_type, @srid = Type::Spatial.parse_sql_type(sql_type)
           set_geometric_type_from_name(geo_type)
         end
       end

data/lib/active_record/connection_adapters/mysql2rgeo/spatial_table_definition.rb

@@ -12,9 +12,6 @@ module ActiveRecord  # :nodoc:
             end
 
             geo_type = ColumnDefinitionUtils.geo_type(options[:type] || type || info[:type])
-            base_type = info[:type] || :geometry
-
-            # puts name.dup << " - " << type.to_s << " - " << options.to_s << " :: " << geo_type.to_s << " - " << base_type.to_s
 
             options[:spatial_type] = geo_type
             column = super(name, geo_type.downcase.to_sym, options)
@@ -26,8 +23,6 @@ module ActiveRecord  # :nodoc:
         end
       end
 
-      SpatialIndexDefinition = Struct.new(*IndexDefinition.members, :spatial)
-
       module ColumnDefinitionUtils
         class << self
           def geo_type(type = "GEOMETRY")

data/lib/active_record/connection_adapters/mysql2rgeo/version.rb

@@ -1,7 +1,9 @@
+# frozen_string_literal: true
+
 module ActiveRecord
   module ConnectionAdapters
     module Mysql2Rgeo
-      VERSION = "2.0.3".freeze
+      VERSION = "5.2.5"
     end
   end
-end
+end

data/lib/active_record/connection_adapters/mysql2rgeo_adapter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # The activerecord-mysql2rgeo-adapter gem installs the *mysql2rgeo*
 # connection adapter into ActiveRecord.
 
@@ -22,6 +24,7 @@ require "active_record/connection_adapters/mysql2rgeo/spatial_expressions"
 require "active_record/connection_adapters/mysql2rgeo/arel_tosql"
 require "active_record/type/spatial"
 require "active_record/connection_adapters/mysql2rgeo/create_connection"
+require "active_record/tasks/database_tasks"
 
 # :startdoc:
 
@@ -32,16 +35,16 @@ module ActiveRecord
 
       SPATIAL_COLUMN_OPTIONS =
         {
-          geometry: {},
-          geometrycollection: {},
-          linestring: {},
-          multilinestring: {},
-          multipoint: {},
-          multipolygon: {},
-          spatial: { type: "geometry" },
-          point: {},
-          polygon: {}
-        }.freeze
+          geometry:            {},
+          geometrycollection:  {},
+          linestring:          {},
+          multilinestring:     {},
+          multipoint:          {},
+          multipolygon:        {},
+          spatial:             { type: "geometry" },
+          point:               {},
+          polygon:             {}
+        }
 
       # http://postgis.17.x6.nabble.com/Default-SRID-td5001115.html
       DEFAULT_SRID = 0
@@ -50,11 +53,10 @@ module ActiveRecord
         super
 
         @visitor = Arel::Visitors::Mysql2Rgeo.new(self)
-        @visitor.extend(DetermineIfPreparableVisitor) if self.class.type_cast_config_to_boolean(config.fetch(:prepared_statements) { true })
       end
 
       def adapter_name
-        "Mysql2Rgeo".freeze
+        "Mysql2Rgeo"
       end
 
       def self.spatial_column_options(key)
@@ -69,45 +71,10 @@ module ActiveRecord
         !mariadb? && version >= "5.7.6"
       end
 
-      # Returns an array of indexes for the given table.
-      def indexes(table_name, name = nil) #:nodoc:
-        if name
-          ActiveSupport::Deprecation.warn(<<-MSG.squish)
-            Passing name to #indexes is deprecated without replacement.
-          MSG
-        end
-
-        indexes = []
-        current_index = nil
-        execute_and_free("SHOW KEYS FROM #{quote_table_name(table_name)}", "SCHEMA") do |result|
-          each_hash(result) do |row|
-            if current_index != row[:Key_name]
-              next if row[:Key_name] == "PRIMARY" # skip the primary key
-              current_index = row[:Key_name]
-
-              mysql_index_type = row[:Index_type].downcase.to_sym
-              index_type = INDEX_TYPES.include?(mysql_index_type) ? mysql_index_type : nil
-              index_using = INDEX_USINGS.include?(mysql_index_type) ? mysql_index_type : nil
-              options = [row[:Table], row[:Key_name], row[:Non_unique].to_i == 0, [], {}, nil, nil, index_type, index_using, row[:Index_comment].presence]
-              indexes << if mysql_index_type == :spatial
-                           options.push(true)
-                           Mysql2Rgeo::SpatialIndexDefinition.new(*options)
-                         else
-                           IndexDefinition.new(row[:Table], row[:Key_name], row[:Non_unique].to_i == 0, [], {}, nil, nil, index_type, index_using, row[:Index_comment].presence)
-                         end
-            end
-
-            indexes.last.columns << row[:Column_name]
-            indexes.last.lengths.merge!(row[:Column_name] => row[:Sub_part].to_i) if row[:Sub_part] && mysql_index_type != :spatial
-          end
-        end
-
-        indexes
-      end
-
       def quote(value)
-        if RGeo::Feature::Geometry.check_type(value)
-          "ST_GeomFromWKB(0x#{RGeo::WKRep::WKBGenerator.new(hex_format: true, little_endian: true).generate(value)},#{value.srid})"
+        dbval = value.try(:value_for_database) || value
+        if RGeo::Feature::Geometry.check_type(dbval)
+          "ST_GeomFromWKB(0x#{RGeo::WKRep::WKBGenerator.new(hex_format: true, little_endian: true).generate(dbval)},#{dbval.srid})"
         else
           super
         end

data/lib/active_record/type/spatial.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ActiveRecord
   module Type
     class Spatial < Value # :nodoc:
@@ -14,66 +16,61 @@ module ActiveRecord
 
       # sql_type: geometry, geometry(Point), geometry(Point,4326), ...
       #
-      # returns [geo_type, srid, has_z, has_m]
+      # returns [geo_type, srid]
       #   geo_type: geography, geometry, point, line_string, polygon, ...
       #   srid:     1234
-      #   has_z:    false
-      #   has_m:    false
       def self.parse_sql_type(sql_type)
-        geo_type, srid, has_z, has_m = nil, 0, false, false
+        geo_type, srid = nil, 0, false, false
 
         if sql_type =~ /(geography|geometry)\((.*)\)$/i
+          # geometry(Point)
           # geometry(Point,4326)
           params = Regexp.last_match(2).split(",")
-          if params.size > 1
-            if params.first =~ /([a-z]+[^zm])(z?)(m?)/i
-              has_z = !Regexp.last_match(2).empty?
-              has_m = !Regexp.last_match(3).empty?
-              geo_type = Regexp.last_match(1)
-            end
-            if params.last =~ /(\d+)/
-              srid = Regexp.last_match(1).to_i
-            end
-          else
-            # geometry(Point)
-            geo_type = params[0]
+          if params.first =~ /([a-z]+[^zm])(z?)(m?)/i
+            geo_type = Regexp.last_match(1)
+          end
+          if params.last =~ /(\d+)/
+            srid = Regexp.last_match(1).to_i
           end
         else
           # geometry
+          # otherType(a,b)
           geo_type = sql_type
         end
-        [geo_type, srid, has_z, has_m]
+        [geo_type, srid]
       end
 
-      def klass
-        type == :geometry ? RGeo::Feature::Geometry : super
+      def spatial_factory
+        @spatial_factory ||=
+          RGeo::ActiveRecord::SpatialFactoryStore.instance.factory(
+            geo_type: @geo_type,
+            sql_type: @sql_type,
+            srid: @srid
+          )
       end
 
-      def type
-        :geometry
+      def klass
+        type == :geometry ? RGeo::Feature::Geometry : super
       end
 
       def spatial?
         true
       end
 
-      def spatial_factory
-        @spatial_factory ||=
-          RGeo::ActiveRecord::SpatialFactoryStore.instance.factory(
-            geo_type: @geo_type,
-            sql_type: @sql_type,
-            srid: @srid
-          )
+      def type
+        :geometry
       end
 
       # support setting an RGeo object or a WKT string
       def serialize(value)
         return if value.nil?
+
         geo_value = cast_value(value)
 
         # TODO - only valid types should be allowed
         # e.g. linestring is not valid for point column
         raise "maybe should raise" unless RGeo::Feature::Geometry.check_type(geo_value)
+
         geo_value
       end
 
@@ -81,35 +78,24 @@ module ActiveRecord
 
       def cast_value(value)
         return if value.nil?
-        case value
-        when ::RGeo::Feature::Geometry
-          value
-          # RGeo::Feature.cast(value, spatial_factory) rescue nil
-        when ::String
-          marker = value[4, 1]
-          if marker == "\x00" || marker == "\x01"
-            srid = value[0, 4].unpack(marker == "\x01" ? "V" : "N").first
-            begin
-              RGeo::WKRep::WKBParser.new(spatial_factory, support_ewkb: true, default_srid: srid).parse(value[4..-1])
-            rescue
-              nil
-            end
-          elsif value[0, 10] =~ /[0-9a-fA-F]{8}0[01]/
-            srid = value[0, 8].to_i(16)
-            srid = [srid].pack("V").unpack("N").first if value[9, 1] == "1"
-            begin
-              RGeo::WKRep::WKBParser.new(spatial_factory, support_ewkb: true, default_srid: srid).parse(value[8..-1])
-            rescue
-              nil
-            end
-          else
-            begin
-              RGeo::WKRep::WKTParser.new(spatial_factory, support_ewkt: true, default_srid: @srid).parse(value)
-            rescue
-              nil
-            end
-          end
+
+        ::String === value ? parse_wkt(value) : value
+      end
+
+      def parse_wkt(string)
+        marker = string[4, 1]
+        if marker == "\x00" || marker == "\x01"
+          srid = string[0, 4].unpack(marker == "\x01" ? "V" : "N").first
+          RGeo::WKRep::WKBParser.new(spatial_factory, support_ewkb: true, default_srid: srid).parse(string[4..-1])
+        elsif string[0, 10] =~ /[0-9a-fA-F]{8}0[01]/
+          srid = string[0, 8].to_i(16)
+          srid = [srid].pack("V").unpack("N").first if string[9, 1] == "1"
+          RGeo::WKRep::WKBParser.new(spatial_factory, support_ewkb: true, default_srid: srid).parse(string[8..-1])
+        else
+          RGeo::WKRep::WKTParser.new(spatial_factory, support_ewkt: true, default_srid: @srid).parse(string)
         end
+      rescue RGeo::Error::ParseError, RGeo::Error::InvalidGeometry
+        nil
       end
     end
   end

data/lib/activerecord-mysql2rgeo-adapter.rb

@@ -1 +1,3 @@
+# frozen_string_literal: true
+
 require "active_record/connection_adapters/mysql2rgeo_adapter"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: activerecord-mysql2rgeo-adapter
 version: !ruby/object:Gem::Version
-  version: 2.0.3
+  version: 5.2.5
 platform: ruby
 authors:
 - Yongdae Hwang
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-12-12 00:00:00.000000000 Z
+date: 2021-04-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -30,28 +30,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '5.1'
+        version: '6.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '5.1'
+        version: '6.0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '12.0'
+        version: '13.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '12.0'
+        version: '13.0'
 - !ruby/object:Gem::Dependency
   name: minitest
   requirement: !ruby/object:Gem::Requirement
@@ -134,7 +134,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.13
+rubygems_version: 2.7.6
 signing_key: 
 specification_version: 4
 summary: ActiveRecord adapter for MySQL, based on RGeo.