positioning 0.1.2 → 0.1.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 42c7d9b5b08bb03060627230a1b44fe9d243b57779541de8def15a0f49f57011
-  data.tar.gz: 66a36e6cc2f019a34c4e1c0d92700be81062556c01da38de8fed632f12bde607
+  metadata.gz: 2863317333ca4e36b7c8aecb178fb38f4489fc4dd76eb771c053769eb82cfbe2
+  data.tar.gz: 13ad7808c1beb5ffaff7d97849b48e194915eeb043e541254843bb22ecfc167d
 SHA512:
-  metadata.gz: 50b3a860281e94d7657b82265e639f038d0004bbfde9d3cf19ca38aeab77814a92d31f5660e6f802c4c72c8931e42c960313413660dd48b0cbc81ca3c899ca30
-  data.tar.gz: 71718c58f8ada39e8b089cc2a2de1da96b685099461b5b397fd40c766549e60279e7b5e6c7d3a61081d5e435b18d3d24e260d985bbda5f2916669f3bdd8f1321
+  metadata.gz: dfcf0d2652e7c9e7ac65e5a1c885edae966ec0d7bc7119b3691c13b2c34c8ad434c6a7e8590e0ff76e550000cddd8cc3b2b2daf72b0626803d988ed21d6e8d7c
+  data.tar.gz: bcdd1517650a8327eaa63ebed37bdcf2cc236beaad383eed23c3e063dfca1031c00714acf99542e8dca0d7320a53fcba6181da24f1e60fe0e4c706e870dd23ae

data/CHANGELOG.md

@@ -1,5 +1,15 @@
 ## [Unreleased]
 
+## [0.1.4] - 2024-03-04
+
+- Fix bug relating to relative position hash coming from Rails being a Hash With Indifferent Access
+
+## [0.1.3] - 2024-03-04
+
+- Internal refactoring of Mechanisms for clarity
+- Additional unit testing of Mechanisms
+- Added additional Ruby and Rails versions to the Github Actions matrix
+
 ## [0.1.2] - 2024-02-29
 
 - Fix a bug related to the scope changing with an explicitly set position value that is the same as the original position.

data/Gemfile

@@ -8,3 +8,8 @@ gem "rake", "~> 13.0"
 gem "minitest", "~> 5.0"
 
 gem "standard", "~> 1.3"
+
+if ENV["RAILS_VERSION"]
+  gem "activerecord", ENV["RAILS_VERSION"]
+  gem "activesupport", ENV["RAILS_VERSION"]
+end

data/lib/positioning/mechanisms.rb

@@ -20,23 +20,14 @@ module Positioning
     end
 
     def update_position
-      # If we're changing scope but not explicitly setting the position then we set the position
-      # to nil so that the item gets placed at the end of the list.
-      self.position = nil if positioning_scope_changed? && !position_changed?
+      clear_position if positioning_scope_changed? && !position_changed?
 
       solidify_position
 
-      # The update strategy is to temporarily set our position to 0, then shift everything out of the way of
-      # our new desired position before finalising it.
       if positioning_scope_changed? || position_changed?
-        record_scope = base_class.where("#{primary_key_column}": primary_key)
-
-        position_was = record_scope.pick(@column)
-        record_scope.update_all "#{@column}": 0
+        move_out_of_the_way
 
         if positioning_scope_changed?
-          positioning_scope_was = base_class.where record_scope.first.slice(*positioning_columns)
-
           contract(positioning_scope_was, position_was..)
           expand(positioning_scope, position..)
         elsif position_was > position
@@ -65,6 +56,10 @@ module Positioning
       @positioned.send primary_key_column
     end
 
+    def record_scope
+      base_class.where("#{primary_key_column}": primary_key)
+    end
+
     def position
       @positioned.send @column
     end
@@ -73,10 +68,23 @@ module Positioning
       @positioned.send :"#{@column}=", position
     end
 
+    def clear_position
+      self.position = nil
+    end
+
     def position_changed?
       @positioned.send :"#{@column}_changed?"
     end
 
+    def position_was
+      @position_was ||= record_scope.pick(@column)
+    end
+
+    def move_out_of_the_way
+      position_was # Memoize the original position before changing it
+      record_scope.update_all "#{@column}": 0
+    end
+
     def expand(scope, range)
       scope.where("#{@column}": range).update_all "#{@column} = #{@column} * -1"
       scope.where("#{@column}": ..-1).update_all "#{@column} = #{@column} * -1 + 1"
@@ -90,7 +98,7 @@ module Positioning
     def solidify_position
       position_before_type_cast = @positioned.read_attribute_before_type_cast @column
       position_before_type_cast.to_sym if position_before_type_cast.is_a? String
-      position_before_type_cast.symbolize_keys! if position_before_type_cast.is_a? Hash
+      position_before_type_cast = position_before_type_cast.symbolize_keys if position_before_type_cast.is_a? Hash
 
       case position_before_type_cast
       when Integer
@@ -118,8 +126,6 @@ module Positioning
           raise Error.new, "relative `#{@column}` record must be in the same scope"
         end
 
-        position_was = base_class.where("#{primary_key_column}": primary_key).pick(@column)
-
         solidified_position = relative_record_scope.pick(@column)
         solidified_position += 1 if relative_position == :after
         solidified_position -= 1 if in_positioning_scope? && position_was < solidified_position
@@ -143,11 +149,15 @@ module Positioning
     end
 
     def positioning_scope
-      base_class.where(@positioned.slice(*positioning_columns)).order(@column)
+      base_class.where @positioned.slice(*positioning_columns)
+    end
+
+    def positioning_scope_was
+      base_class.where record_scope.first.slice(*positioning_columns)
     end
 
     def in_positioning_scope?
-      @positioned.persisted? && positioning_scope.exists?(primary_key)
+      @positioned.persisted? && positioning_scope.where("#{primary_key_column}": primary_key).exists?
     end
 
     def positioning_scope_changed?

data/lib/positioning/version.rb

@@ -1,3 +1,3 @@
 module Positioning
-  VERSION = "0.1.2"
+  VERSION = "0.1.4"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: positioning
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.4
 platform: ruby
 authors:
 - Brendon Muir
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2024-02-29 00:00:00.000000000 Z
+date: 2024-03-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -118,7 +118,6 @@ files:
 - ".standard.yml"
 - CHANGELOG.md
 - Gemfile
-- Gemfile.lock
 - LICENSE.txt
 - README.md
 - Rakefile

data/Gemfile.lock

@@ -1,109 +0,0 @@
-PATH
-  remote: .
-  specs:
-    positioning (0.1.2)
-      activerecord (>= 6.1)
-      activesupport (>= 6.1)
-
-GEM
-  remote: https://rubygems.org/
-  specs:
-    activemodel (7.1.3.2)
-      activesupport (= 7.1.3.2)
-    activerecord (7.1.3.2)
-      activemodel (= 7.1.3.2)
-      activesupport (= 7.1.3.2)
-      timeout (>= 0.4.0)
-    activesupport (7.1.3.2)
-      base64
-      bigdecimal
-      concurrent-ruby (~> 1.0, >= 1.0.2)
-      connection_pool (>= 2.2.5)
-      drb
-      i18n (>= 1.6, < 2)
-      minitest (>= 5.1)
-      mutex_m
-      tzinfo (~> 2.0)
-    ast (2.4.2)
-    base64 (0.2.0)
-    bigdecimal (3.1.6)
-    concurrent-ruby (1.2.3)
-    connection_pool (2.4.1)
-    drb (2.2.0)
-      ruby2_keywords
-    i18n (1.14.1)
-      concurrent-ruby (~> 1.0)
-    json (2.7.1)
-    language_server-protocol (3.17.0.3)
-    lint_roller (1.1.0)
-    minitest (5.22.1)
-    minitest-hooks (1.5.1)
-      minitest (> 5.3)
-    mocha (2.1.0)
-      ruby2_keywords (>= 0.0.5)
-    mutex_m (0.2.0)
-    mysql2 (0.5.6)
-    parallel (1.24.0)
-    parser (3.3.0.5)
-      ast (~> 2.4.1)
-      racc
-    pg (1.5.5)
-    racc (1.7.3)
-    rainbow (3.1.1)
-    rake (13.1.0)
-    regexp_parser (2.9.0)
-    rexml (3.2.6)
-    rubocop (1.59.0)
-      json (~> 2.3)
-      language_server-protocol (>= 3.17.0)
-      parallel (~> 1.10)
-      parser (>= 3.2.2.4)
-      rainbow (>= 2.2.2, < 4.0)
-      regexp_parser (>= 1.8, < 3.0)
-      rexml (>= 3.2.5, < 4.0)
-      rubocop-ast (>= 1.30.0, < 2.0)
-      ruby-progressbar (~> 1.7)
-      unicode-display_width (>= 2.4.0, < 3.0)
-    rubocop-ast (1.30.0)
-      parser (>= 3.2.1.0)
-    rubocop-performance (1.20.2)
-      rubocop (>= 1.48.1, < 2.0)
-      rubocop-ast (>= 1.30.0, < 2.0)
-    ruby-progressbar (1.13.0)
-    ruby2_keywords (0.0.5)
-    sqlite3 (1.7.2-arm64-darwin)
-    sqlite3 (1.7.2-x86_64-linux)
-    standard (1.33.0)
-      language_server-protocol (~> 3.17.0.2)
-      lint_roller (~> 1.0)
-      rubocop (~> 1.59.0)
-      standard-custom (~> 1.0.0)
-      standard-performance (~> 1.3)
-    standard-custom (1.0.2)
-      lint_roller (~> 1.0)
-      rubocop (~> 1.50)
-    standard-performance (1.3.1)
-      lint_roller (~> 1.1)
-      rubocop-performance (~> 1.20.2)
-    timeout (0.4.1)
-    tzinfo (2.0.6)
-      concurrent-ruby (~> 1.0)
-    unicode-display_width (2.5.0)
-
-PLATFORMS
-  arm64-darwin-21
-  x86_64-linux
-
-DEPENDENCIES
-  minitest (~> 5.0)
-  minitest-hooks (~> 1.5.1)
-  mocha (~> 2.1.0)
-  mysql2 (~> 0.5.6)
-  pg (~> 1.5.5)
-  positioning!
-  rake (~> 13.0)
-  sqlite3 (~> 1.7.2)
-  standard (~> 1.3)
-
-BUNDLED WITH
-   2.3.8