finapps 2.3.6 → 2.3.7
Sign up to get free protection for your applications and to get access to all the features.
- checksums.yaml +4 -4
- data/lib/finapps/rest/orders.rb +2 -2
- data/lib/finapps/version.rb +1 -1
- data/spec/rest/orders_spec.rb +6 -0
- metadata +1 -1
checksums.yaml
CHANGED
@@ -1,7 +1,7 @@
|
|
1
1
|
---
|
2
2
|
SHA1:
|
3
|
-
metadata.gz:
|
4
|
-
data.tar.gz:
|
3
|
+
metadata.gz: fa46e2d5c4400f8118bfa2bfec92330574fd96f0
|
4
|
+
data.tar.gz: f7d566d14145e831f36f9ae7c2d698044902322e
|
5
5
|
SHA512:
|
6
|
-
metadata.gz:
|
7
|
-
data.tar.gz:
|
6
|
+
metadata.gz: ef5a6691ff97b23e0bdb7dfdfd1104110f89ca88bd21a536cb54ed4c2651d212961464a152273dfd4057fd30d0fb4ec6213f6cff572b07bf4161668cfcc2d65f
|
7
|
+
data.tar.gz: 42560e6de9ff8e43146e5f8f9d9a40ff5d816336f6ed14a2fcd6cd321ecc9cff0fdec15c2ce9e764d4614a44ab42ceefd2913ad7b2deb4183e595025710b1c26
|
data/lib/finapps/rest/orders.rb
CHANGED
@@ -55,7 +55,7 @@ module FinApps
|
|
55
55
|
filter = {}
|
56
56
|
filter.merge!(search_query(params[:searchTerm])) if params[:searchTerm]
|
57
57
|
filter.merge!(status_query(params[:status])) if params[:status]
|
58
|
-
filter.merge!(assignment_query(params[:assignment])) if params
|
58
|
+
filter.merge!(assignment_query(params[:assignment])) if params[:assignment] # assignment can be ""
|
59
59
|
filter.merge!(relation_query(params[:relation])) if !params[:searchTerm] && !nil_or_empty?(params[:relation])
|
60
60
|
filter
|
61
61
|
end
|
@@ -91,7 +91,7 @@ module FinApps
|
|
91
91
|
end
|
92
92
|
|
93
93
|
def assignment_query(assignment)
|
94
|
-
{"assignment.operator_id": assignment}
|
94
|
+
{"assignment.operator_id": assignment.empty? ? nil : assignment} # translate "" to null assignment
|
95
95
|
end
|
96
96
|
|
97
97
|
def relation_query(relation)
|
data/lib/finapps/version.rb
CHANGED
data/spec/rest/orders_spec.rb
CHANGED
@@ -105,6 +105,12 @@ RSpec.describe FinApps::REST::Orders do
|
|
105
105
|
'%5D%7D%7D%5D%7D&page=2&requested=25&sort=status'
|
106
106
|
expect(WebMock).to have_requested(:get, url)
|
107
107
|
end
|
108
|
+
it 'builds null assignment query properly when supplied w/ empty string' do
|
109
|
+
FinApps::REST::Orders.new(client).list(assignment: '')
|
110
|
+
|
111
|
+
url = "#{FinAppsCore::REST::Defaults::DEFAULTS[:host]}/v2/orders?filter=%7B%22assignment.operator_id%22:null%7D"
|
112
|
+
expect(WebMock).to have_requested(:get, url)
|
113
|
+
end
|
108
114
|
end
|
109
115
|
end
|
110
116
|
|