Skip to content

Commit 2a2beb4

Browse files
committed
moved qualifying_fields_for_newsfeed to contstant
1 parent aea4729 commit 2a2beb4

File tree

2 files changed

+8
-14
lines changed

2 files changed

+8
-14
lines changed

lib/spark_api/models/saved_search.rb

Lines changed: 6 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,11 @@ class SavedSearch < Base
88

99
attr_accessor :newsfeeds
1010

11+
# Newsfeed restriction criteria for saved searches:
12+
# http://alpha.sparkplatform.com/docs/api_services/newsfeed/restrictions#criteria
13+
QUALIFYING_FIELDS_FOR_NEWSFEED = %w(BathsTotal BedsTotal City CountyOrParish ListPrice Location MlsStatus
14+
PostalCode PropertyType RoomsTotal State)
15+
1116
self.element_name="savedsearches"
1217

1318
def initialize(attributes={})
@@ -74,24 +79,13 @@ def provided_search?
7479
false
7580
end
7681

77-
def self.qualifying_fields_for_newsfeed
78-
# Newsfeed restriction criteria for saved searches:
79-
# http://alpha.sparkplatform.com/docs/api_services/newsfeed/restrictions#criteria
80-
%w(BathsTotal BedsTotal City CountyOrParish ListPrice Location MlsStatus PostalCode PropertyType
81-
RoomsTotal State)
82-
end
83-
84-
def qualifying_fields_for_newsfeed
85-
self.class.qualifying_fields_for_newsfeed
86-
end
87-
8882
def can_have_newsfeed?
8983

9084
return true if has_active_newsfeed? || has_inactive_newsfeed?
9185

9286
number_of_filters = 0
9387

94-
self.class.qualifying_fields_for_newsfeed.each do |field|
88+
QUALIFYING_FIELDS_FOR_NEWSFEED.each do |field|
9589
number_of_filters += 1 if self.Filter.include? field
9690
end
9791

spec/unit/spark_api/models/saved_search_spec.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -238,9 +238,9 @@
238238
end
239239
end
240240

241-
describe "qualifying_fields_for_newsfeed" do
241+
describe "QUALIFYING_FIELDS_FOR_NEWSFEED" do
242242
it "should return an array" do
243-
subject.class.qualifying_fields_for_newsfeed.should be_an(Array)
243+
subject.class::QUALIFYING_FIELDS_FOR_NEWSFEED.should be_an(Array)
244244
end
245245
end
246246

0 commit comments

Comments
 (0)