Changed created_before to published_before

Author: Dantemss

app/controllers/api/v1/exercises_controller.rb

@@ -45,8 +45,8 @@ class ExercisesController < OpenStax::Api::V1::ApiController
       * `number` &ndash; Matches the exercise number exactly.
       * `version` &ndash; Matches the exercise version exactly.
       * `id` &ndash; Matches the exercise ID exactly.
-      * `created_before` &ndash; Matches exercises created before the given date.
-                                 Enclose date in quotes to avoid parsing errors.
+      * `published_before` &ndash; Matches exercises published before the given date.
+                                   Enclose date in quotes to avoid parsing errors.
 
       You can also add search terms without prefixes, separated by spaces.
       These terms will be searched for in all of the prefix categories.

app/routines/search_exercises.rb

@@ -10,7 +10,9 @@ class SearchExercises
     'number' => Publication.arel_table[:number],
     'version' => Publication.arel_table[:version],
     'title' => :title,
-    'created_at' => :created_at
+    'created_at' => :created_at,
+    'updated_at' => :updated_at,
+    'published_at' => Publication.arel_table[:published_at]
   }
 
   protected
@@ -189,16 +191,17 @@ def exec(params = {}, options = {})
         end
       end
 
-      with.keyword :created_before do |created_befores|
-        min_created_before = created_befores.flatten.collect do |str|
+      with.keyword :published_before do |published_befores|
+        min_published_before = published_befores.flatten.collect do |str|
           DateTime.parse(str) rescue nil
         end.compact.min
-        next @items = @items.none if min_created_before.nil?
+        next @items = @items.none if min_published_before.nil?
 
-        @items = @items.where{created_at < min_created_before}
+        @items = @items.where{publication.published_at < min_published_before}
 
-        # Latest now refers to results that happened before min_created_before
-        latest_scope = latest_scope.where{created_at < min_created_before} unless latest_scope.nil?
+        # Latest now refers to results that happened before min_published_before
+        latest_scope = latest_scope.where{publication.published_at < min_published_before} \
+          unless latest_scope.nil?
       end
     end
 

db/migrate/20150731182709_add_created_at_index_to_exercises.rb

@@ -11,7 +11,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 20150731182709) do
+ActiveRecord::Schema.define(version: 20150225002247) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
@@ -192,7 +192,6 @@
     t.datetime "updated_at", null: false
   end
 
-  add_index "exercises", ["created_at"], name: "index_exercises_on_created_at", using: :btree
   add_index "exercises", ["title"], name: "index_exercises_on_title", using: :btree
 
   create_table "fine_print_contracts", force: :cascade do |t|

db/schema.rb

@@ -143,7 +143,7 @@
       expect(outputs.items).to eq [new_exercise_2]
     end
 
-    it 'changes the definition of "latest" if created_before is specified' do
+    it 'changes the definition of "latest" if published_before is specified' do
       new_exercise = Exercise.new
       Api::V1::ExerciseRepresenter.new(new_exercise).from_json({
         tags: ['tag2', 'tag3'],
@@ -179,7 +179,7 @@
       expect(outputs.items).to eq []
 
       result = SearchExercises.call(
-        q: "tag:tAg1 created_before:\"#{new_exercise.created_at.as_json}\""
+        q: "tag:tAg1 published_before:\"#{new_exercise.published_at.as_json}\""
       )
       expect(result.errors).to be_empty