spree_cm_commissioner 2.4.0 → 2.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5f63b4eb2a25d5dff782d61eb3df12b639c261a418213940ed12d8b9b483b9a1
-  data.tar.gz: 6682f9d70eb92effa0345aee38da3d483c8bf555a74da8c88fc52ddb7553a2e5
+  metadata.gz: 4e32e4116a234b679abcfe3a9d7005bd61890d80936839c32330ecf2b421dadf
+  data.tar.gz: a7759f377e09d49281150257a496e44af61e4cabb67a8bec0bacb20a73ea439d
 SHA512:
-  metadata.gz: 7abea09634def1e05f839d075bb5fcd1cdfac7bc13ed0392ac6c5aa60e5f9aef3a54600d69afa63778aca0ddf62e09ffc1f5c0200dd88f728b7a1bcd0272faea
-  data.tar.gz: f102b65bf2526916e47c728b0e50eebe7ce188b96bc0f206d92be0434499a734fe0c434f5d364554c20d5bcf0cec20f4b2bdb945d321c213ddb03c1678d9f135
+  metadata.gz: 5119cedb570e77e3417bf723ea84926231e4d72000686312a194c1cfb4764061b5df9801923a5db629d8b8adccfd8a70378a238a0c099af46abe8f5c6a2da1d0
+  data.tar.gz: adeee4909603c0294be64345ea3de5b310e2dcaa174669137794b56dd23b800ddd9a1f429da636df23953d80154a3b96835cb19a81274fc0908edaae1ebd13d9

data/Gemfile.lock

@@ -34,7 +34,7 @@ GIT
 PATH
   remote: .
   specs:
-    spree_cm_commissioner (2.4.0)
+    spree_cm_commissioner (2.4.1)
       activerecord-multi-tenant
       activerecord_json_validator (~> 2.1, >= 2.1.3)
       aws-sdk-cloudfront

data/app/controllers/spree/api/v2/storefront/route_places_controller.rb

@@ -1,24 +1,42 @@
-# API endpoint for searching places that are used in routes by keyword.
+# API endpoint for searching places connected to a specific place via routes.
 #
 # GET /api/v2/storefront/route_places
 #
-# Searches for places (origins or destinations) that are used in existing routes,
-# filtered by keyword search on place names.
+# Finds places (origins or destinations) that are connected to a given place through existing routes.
+# Optionally filters results by keyword. Supports two modes:
+#
+# Usage 1: Connected places (requires place_id)
+#   - Finds places connected to a specific place via routes
+#   - Returns origins/destinations that have routes with the specified place
+#
+# Usage 2: Keyword search (requires query, place_id optional)
+#   - Searches all route places by keyword
+#   - Returns all origins/destinations matching the keyword
 #
 # Query Parameters:
-#   - query: [String] Required. Keyword to search in place names (case-insensitive)
 #   - place_type: [String] Required. Type of route place: 'origin' or 'destination'
+#     * 'origin': returns origins that have routes TO the specified place (if place_id provided)
+#     * 'destination': returns destinations that have routes FROM the specified place (if place_id provided)
+#   - place_id: [Integer] Optional. The place ID to find connected places for
+#   - query: [String] Optional. Keyword to filter place names (case-insensitive)
 #   - include: Optional comma-separated list (e.g., 'vendors,nearby_places')
 #
 # Response: Collection of places serialized with PlaceSerializer
 #
 # Behavior:
-#   - Returns empty collection if query is blank or place_type is invalid
-#   - UI should only call this endpoint when user has input a search term
-#   - For initial/empty state, use GET /api/v2/storefront/popular_route_places instead
+#   - Returns empty collection if place_type is invalid
+#   - If place_id provided: returns places connected to that place
+#   - If place_id blank: returns all origins/destinations
+#   - Filters results by keyword if provided
+#
+# @example Mode 1: Find destinations connected to origin place ID 123
+#   GET /api/v2/storefront/route_places?place_id=123&place_type=destination
+#
+# @example Mode 2: Search all destination places by keyword
+#   GET /api/v2/storefront/route_places?place_type=destination&query=Phnom
 #
-# @example
-#   GET /api/v2/storefront/route_places?query=Phnom&place_type=origin
+# @example Combined: Find origins connected to place 456, filtered by keyword
+#   GET /api/v2/storefront/route_places?place_id=456&place_type=origin&query=Siem
 module Spree
   module Api
     module V2
@@ -28,7 +46,11 @@ module Spree
 
           # override
           def collection
-            @collection ||= collection_finder.new(keyword: params[:query], place_type: params[:place_type]).execute
+            @collection ||= collection_finder.new(
+              place_type: params[:place_type],
+              place_id: params[:place_id],
+              keyword: params[:query]
+            ).execute
           end
 
           # override

data/app/finders/spree_cm_commissioner/places/find_with_route.rb

@@ -1,42 +1,49 @@
-# Finds places that are used in routes (as origin or destination) by keyword search.
+# Finds places connected via routes with optional filtering.
 #
-# @param keyword [String] Search term to match against place names (case-insensitive)
-# @param place_type [String] Type of route place: 'origin' or 'destination'
+# @param place_type [String] Required. 'origin' or 'destination'
+# @param place_id [Integer] Optional. Filter by connected place ID
+# @param keyword [String] Optional. Filter by place name (case-insensitive)
 #
-# @return [ActiveRecord::Relation<SpreeCmCommissioner::Place>] Places matching the keyword
-#   that are used as origins or destinations in existing routes. Returns empty relation if
-#   keyword is blank or place_type is invalid.
+# @return [ActiveRecord::Relation<SpreeCmCommissioner::Place>]
 #
-# @note For default/empty state display, use SpreeCmCommissioner::Routes::FindPopular instead
-#   to show popular routes to users before they enter a search term.
+# Modes:
+#   - place_id only: returns connected places (origins TO or destinations FROM given place)
+#   - keyword only: filters by name
+#   - both: connected places filtered by name
+#   - neither: all origins/destinations in routes
 #
-# @example
-#   finder = SpreeCmCommissioner::Places::FindWithRoute.new(keyword: 'Phnom', place_type: 'origin')
-#   finder.execute # => Returns places with "Phnom" in name that are used as route origins
+# @example Origins with routes to place 123
+#   FindWithRoute.new(place_type: 'origin', place_id: 123).execute
+#
+# @example Destinations filtered by keyword
+#   FindWithRoute.new(place_type: 'destination', keyword: 'Phnom').execute
 module SpreeCmCommissioner
   module Places
     class FindWithRoute
-      attr_reader :keyword, :place_type
+      attr_reader :place_type, :place_id, :keyword
 
-      def initialize(keyword:, place_type:)
-        @keyword = keyword
+      def initialize(place_type:, place_id: nil, keyword: nil)
         @place_type = place_type
+        @place_id = place_id
+        @keyword = keyword
       end
 
       def execute
-        return SpreeCmCommissioner::Place.none if keyword.blank? || place_type.blank?
+        return SpreeCmCommissioner::Place.none if place_type.blank?
 
-        scope.where('cm_places.name ILIKE ?', "%#{keyword}%")
-      end
+        result = if place_type == 'origin'
+                   scope = SpreeCmCommissioner::Place.with_routes_as_origin
+                   place_id.present? ? scope.where(cm_routes: { destination_place_id: place_id }) : scope
+                 else
+                   scope = SpreeCmCommissioner::Place.with_routes_as_destination
+                   place_id.present? ? scope.where(cm_routes: { origin_place_id: place_id }) : scope
+                 end
 
-      private
+        return SpreeCmCommissioner::Place.none if place_id.present? && !SpreeCmCommissioner::Place.exists?(place_id)
 
-      def scope
-        if place_type == 'origin'
-          SpreeCmCommissioner::Place.with_routes_as_origin
-        else
-          SpreeCmCommissioner::Place.with_routes_as_destination
-        end
+        # Apply keyword filter if provided
+        result = result.where('cm_places.name ILIKE ?', "%#{keyword}%") if keyword.present?
+        result
       end
     end
   end

data/app/request_schemas/spree_cm_commissioner/route_places_request_schema.rb

@@ -2,7 +2,8 @@ module SpreeCmCommissioner
   class RoutePlacesRequestSchema < ApplicationRequestSchema
     params do
       required(:place_type).filled(:string)
-      required(:query).filled(:string)
+      optional(:query).maybe(:string)
+      optional(:place_id).maybe(:integer)
     end
 
     rule(:place_type) do

data/lib/spree_cm_commissioner/version.rb

@@ -1,5 +1,5 @@
 module SpreeCmCommissioner
-  VERSION = '2.4.0'.freeze
+  VERSION = '2.4.1'.freeze
 
   module_function
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: spree_cm_commissioner
 version: !ruby/object:Gem::Version
-  version: 2.4.0
+  version: 2.4.1
 platform: ruby
 authors:
 - You