miscellany 0.1.22 → 0.1.24

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cbc281a50690bfaa8e5745c43786179654ffd59a87a5df621d1f7423f8a1cb6c
-  data.tar.gz: 105181f0df9279f89e85ba59a35ddda7d390d2dc86068b8a5d2e28ee5cc4887e
+  metadata.gz: 6de3e8ac55956c58bff00af5ffae6ce8972b142be6a5aad47096c4776658ba4e
+  data.tar.gz: 4668195d6a891348e495fff7ecf187415eef6f2588d14e33be556f04f7358042
 SHA512:
-  metadata.gz: f393bac71597cef4b0879b0a8d9a21003d5e36355887706951880f4cac5ea769ffc8088f5b4d523ea2cdbe7c0b2552d56a4a759db0a9ae4acf69134a2e4e15b8
-  data.tar.gz: a9265249c223ebd121fe8189aacfd462376250b3fb13726ea1878e2b2931317604cec856d6db7f457ecefb77e645bc178a07d45e7a3b95b7347cb8e9eda9515c
+  metadata.gz: 2cec7d17686d2d9faf88dcfc20ac2ca5fba8f10ca7c1e021a7e0b67af4f6506b8d0a03444437ac1aae23eaa39e3a116cbb9e9ef62b649ec73faaf3b45e155772
+  data.tar.gz: 7a0c8b98668ad66aebd4b51e3d41aa969a13e3138992021f36bf95f04be3e1db289f95fb43e95f04e8b8876c2f5e4652a9e1609b1dfbd907595f21f7d9513d4d

data/lib/miscellany/active_record/arbitrary_prefetch.rb

@@ -203,12 +203,12 @@ module Miscellany
 
       base_module ||= mod
 
-      if mod.name.ends_with? "Patch"
+      if mod.name.end_with? "Patch"
         base_full_name = base_module.to_s
         mod_full_name = mod.to_s
         mod_rel_name = mod_full_name.sub(base_full_name, '')
         mod_rel_bits = mod_rel_name.split('::').select(&:present?).map do |bit|
-          bit.ends_with?('Patch') ? bit[0..-6] : bit
+          bit.end_with?('Patch') ? bit[0..-6] : bit
         end
         final_mod_name = [install_base, *mod_rel_bits].select(&:present?).join("::")
         install_mod = final_mod_name.constantize

data/lib/miscellany/controller/sliced_response.rb

@@ -202,6 +202,18 @@ module Miscellany
             limit -= offset unless limit.nil?
 
             query = items
+
+            applied_sorts.each do |sort|
+              # TODO Throw an error if joins: is present on a non-AR::Relation?
+              if sort[:joins].is_a?(ActiveRecord::Relation)
+                query = query.merge(sort[:joins])
+              elsif sort[:joins].is_a?(Proc)
+                query = sort[:joins].call(query)
+              elsif sort[:joins].present?
+                query = query.joins(sort[:joins])
+              end
+            end
+
             sort_clause = sort_sql
             query = query.order(Arel.sql(sort_clause)) if sort_clause.present?
 
@@ -219,13 +231,19 @@ module Miscellany
         [slice_start, slice_end == -1 ? nil : slice_end]
       end
 
-      def sort_sql
-        sorts = [ *Array(self.sort), *options[:sort_parser]&.default_sorts]
-        sorts.compact!
+      def applied_sorts
+        @applied_sorts ||= begin
+          sorts = [ *Array(self.sort), *options[:sort_parser]&.default_sorts]
+          sorts.compact!
+          sorts = Miscellany::SortLang.distinct_sorts(sorts)
+          sorts
+        end
+      end
 
-        return nil unless sorts.present?
+      def sort_sql
+        return nil unless applied_sorts.present?
 
-        Miscellany::SortLang.sqlize(sorts)
+        Miscellany::SortLang.sqlize(applied_sorts)
       end
     end
   end

data/lib/miscellany/sort_lang.rb

@@ -22,11 +22,11 @@ module Miscellany
       sort.compact
     end
 
-    def self.sqlize(sorts)
+    def self.distinct_sorts(sorts)
       seen_sorts = Set.new
 
       # Only include each sort key/"column" once
-      sorts = sorts.select do |sort|
+      sorts.select do |sort|
         sid = sort[:key] || sort[:column]
         next true unless sid.present?
 
@@ -37,7 +37,9 @@ module Miscellany
           true
         end
       end
+    end
 
+    def self.sqlize(sorts)
       sorts.map do |sort|
         order = sort[:order] || 'ASC'
         if sort[:column].is_a?(Proc)

data/lib/miscellany/version.rb

@@ -1,3 +1,3 @@
 module Miscellany
-  VERSION = "0.1.22".freeze
+  VERSION = "0.1.24".freeze
 end

data/spec/miscellany/sort_lang_spec.rb

@@ -30,14 +30,17 @@ describe Miscellany::SortLang do
     end
   end
 
-  describe ".sqlize" do
+  describe ".distinct_sorts" do
     it "only includes each sort key once" do
-      expect(Miscellany::SortLang.sqlize([
+      distinct = Miscellany::SortLang.distinct_sorts([
         { column: "created_at", order: "ASC" },
         { column: "created_at", order: "DESC" },
-      ])).to eql "created_at ASC NULLS FIRST"
+      ])
+      expect(Miscellany::SortLang.sqlize(distinct)).to eql "created_at ASC NULLS FIRST"
     end
+  end
 
+  describe ".sqlize" do
     describe "nulls: option" do
       it "accepts a nulls: option" do
         expect(Miscellany::SortLang.sqlize([

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: miscellany
 version: !ruby/object:Gem::Version
-  version: 0.1.22
+  version: 0.1.24
 platform: ruby
 authors:
 - Ethan Knapp
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-10-20 00:00:00.000000000 Z
+date: 2024-08-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails