add constraints to most_recent

app/models/inter_library_loan_transition.rb

@@ -17,5 +17,5 @@ class InterLibraryLoanTransition < ActiveRecord::Base
 #  inter_library_loan_id :integer
 #  created_at            :datetime
 #  updated_at            :datetime
-#  most_recent           :boolean
+#  most_recent           :boolean          not null
 #

db/migrate/20180107164929_add_constraints_to_most_recent_for_inter_library_loan_transitions.rb

@@ -0,0 +1,13 @@
+class AddConstraintsToMostRecentForInterLibraryLoanTransitions < ActiveRecord::Migration
+  disable_ddl_transaction!
+
+  def up
+    add_index :inter_library_loan_transitions, [:inter_library_loan_id, :most_recent], unique: true, where: "most_recent", name: "index_inter_library_loan_transitions_parent_most_recent" #, algorithm: :concurrently
+    change_column_null :inter_library_loan_transitions, :most_recent, false
+  end
+
+  def down
+    remove_index :inter_library_loan_transitions, name: "index_inter_library_loan_transitions_parent_most_recent"
+    change_column_null :inter_library_loan_transitions, :most_recent, true
+  end
+end

lib/enju_inter_library_loan/version.rb

@@ -1,3 +1,3 @@
 module EnjuInterLibraryLoan
-  VERSION = "0.2.0"
+  VERSION = "0.2.1"
 end

spec/dummy/db/migrate/20180107161311_add_constraints_to_most_recent_for_agent_import_file_transitions.rb

@@ -0,0 +1,13 @@
+class AddConstraintsToMostRecentForAgentImportFileTransitions < ActiveRecord::Migration
+  disable_ddl_transaction!
+
+  def up
+    add_index :agent_import_file_transitions, [:agent_import_file_id, :most_recent], unique: true, where: "most_recent", name: "index_agent_import_file_transitions_parent_most_recent" #, algorithm: :concurrently
+    change_column_null :agent_import_file_transitions, :most_recent, false
+  end
+
+  def down
+    remove_index :agent_import_file_transitions, name: "index_agent_import_file_transitions_parent_most_recent"
+    change_column_null :agent_import_file_transitions, :most_recent, true
+  end
+end

spec/dummy/db/migrate/20180107161331_add_constraints_to_most_recent_for_resource_import_file_transitions.rb

@@ -0,0 +1,13 @@
+class AddConstraintsToMostRecentForResourceImportFileTransitions < ActiveRecord::Migration
+  disable_ddl_transaction!
+
+  def up
+    add_index :resource_import_file_transitions, [:resource_import_file_id, :most_recent], unique: true, where: "most_recent", name: "index_resource_import_file_transitions_parent_most_recent" #, algorithm: :concurrently
+    change_column_null :resource_import_file_transitions, :most_recent, false
+  end
+
+  def down
+    remove_index :resource_import_file_transitions, name: "index_resource_import_file_transitions_parent_most_recent"
+    change_column_null :resource_import_file_transitions, :most_recent, true
+  end
+end

spec/dummy/db/migrate/20180107161347_add_constraints_to_most_recent_for_resource_export_file_transitions.rb

@@ -0,0 +1,13 @@
+class AddConstraintsToMostRecentForResourceExportFileTransitions < ActiveRecord::Migration
+  disable_ddl_transaction!
+
+  def up
+    add_index :resource_export_file_transitions, [:resource_export_file_id, :most_recent], unique: true, where: "most_recent", name: "index_resource_export_file_transitions_parent_most_recent" #, algorithm: :concurrently
+    change_column_null :resource_export_file_transitions, :most_recent, false
+  end
+
+  def down
+    remove_index :resource_export_file_transitions, name: "index_resource_export_file_transitions_parent_most_recent"
+    change_column_null :resource_export_file_transitions, :most_recent, true
+  end
+end

spec/dummy/db/migrate/20180107161410_add_constraints_to_most_recent_for_import_request_transitions.rb

@@ -0,0 +1,13 @@
+class AddConstraintsToMostRecentForImportRequestTransitions < ActiveRecord::Migration
+  disable_ddl_transaction!
+
+  def up
+    add_index :import_request_transitions, [:import_request_id, :most_recent], unique: true, where: "most_recent", name: "index_import_request_transitions_parent_most_recent" #, algorithm: :concurrently
+    change_column_null :import_request_transitions, :most_recent, false
+  end
+
+  def down
+    remove_index :import_request_transitions, name: "index_import_request_transitions_parent_most_recent"
+    change_column_null :import_request_transitions, :most_recent, true
+  end
+end

spec/dummy/db/schema.rb

@@ -11,7 +11,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 20180102162311) do
+ActiveRecord::Schema.define(version: 20180107164929) do
 
   create_table "accepts", force: :cascade do |t|
     t.integer  "basket_id"
@@ -23,6 +23,7 @@
 
   add_index "accepts", ["basket_id"], name: "index_accepts_on_basket_id"
   add_index "accepts", ["item_id"], name: "index_accepts_on_item_id"
+  add_index "accepts", ["librarian_id"], name: "index_accepts_on_librarian_id"
 
   create_table "agent_import_file_transitions", force: :cascade do |t|
     t.string   "to_state"
@@ -31,9 +32,10 @@
     t.integer  "agent_import_file_id"
     t.datetime "created_at"
     t.datetime "updated_at"
-    t.boolean  "most_recent"
+    t.boolean  "most_recent",                         null: false
   end
 
+  add_index "agent_import_file_transitions", ["agent_import_file_id", "most_recent"], name: "index_agent_import_file_transitions_parent_most_recent", unique: true
   add_index "agent_import_file_transitions", ["agent_import_file_id"], name: "index_agent_import_file_transitions_on_agent_import_file_id"
   add_index "agent_import_file_transitions", ["sort_key", "agent_import_file_id"], name: "index_agent_import_file_transitions_on_sort_key_and_file_id", unique: true
 
@@ -449,9 +451,10 @@
     t.integer  "import_request_id"
     t.datetime "created_at"
     t.datetime "updated_at"
-    t.boolean  "most_recent"
+    t.boolean  "most_recent",                      null: false
   end
 
+  add_index "import_request_transitions", ["import_request_id", "most_recent"], name: "index_import_request_transitions_parent_most_recent", unique: true
   add_index "import_request_transitions", ["import_request_id"], name: "index_import_request_transitions_on_import_request_id"
   add_index "import_request_transitions", ["sort_key", "import_request_id"], name: "index_import_request_transitions_on_sort_key_and_request_id", unique: true
 
@@ -474,9 +477,10 @@
     t.integer  "inter_library_loan_id"
     t.datetime "created_at"
     t.datetime "updated_at"
-    t.boolean  "most_recent"
+    t.boolean  "most_recent",                          null: false
   end
 
+  add_index "inter_library_loan_transitions", ["inter_library_loan_id", "most_recent"], name: "index_inter_library_loan_transitions_parent_most_recent", unique: true
   add_index "inter_library_loan_transitions", ["inter_library_loan_id"], name: "index_inter_library_loan_transitions_on_inter_library_loan_id"
   add_index "inter_library_loan_transitions", ["sort_key", "inter_library_loan_id"], name: "index_inter_library_loan_transitions_on_sort_key_and_loan_id", unique: true
 
@@ -585,7 +589,7 @@
     t.text     "note"
     t.integer  "call_number_rows",      default: 1,   null: false
     t.string   "call_number_delimiter", default: "|", null: false
-    t.integer  "library_group_id",      default: 1,   null: false
+    t.integer  "library_group_id",                    null: false
     t.integer  "users_count",           default: 0,   null: false
     t.integer  "position"
     t.integer  "country_id"
@@ -599,7 +603,7 @@
   end
 
   add_index "libraries", ["library_group_id"], name: "index_libraries_on_library_group_id"
-  add_index "libraries", ["name"], name: "index_libraries_on_name", unique: true
+  add_index "libraries", ["name"], name: "index_libraries_on_name"
 
   create_table "library_group_translations", force: :cascade do |t|
     t.integer  "library_group_id", null: false
@@ -632,11 +636,11 @@
     t.boolean  "family_name_first",             default: true
     t.integer  "pub_year_facet_range_interval", default: 10
     t.integer  "user_id"
+    t.boolean  "csv_charset_conversion",        default: false,                    null: false
     t.string   "header_logo_file_name"
     t.string   "header_logo_content_type"
     t.integer  "header_logo_file_size"
     t.datetime "header_logo_updated_at"
-    t.boolean  "csv_charset_conversion",        default: false,                    null: false
     t.text     "header_logo_meta"
   end
 
@@ -766,7 +770,7 @@
     t.datetime "valid_until"
     t.datetime "date_submitted"
     t.datetime "date_accepted"
-    t.datetime "date_caputured"
+    t.datetime "date_captured"
     t.string   "pub_date"
     t.string   "edition_string"
     t.integer  "volume_number"
@@ -1048,9 +1052,10 @@
     t.integer  "resource_export_file_id"
     t.datetime "created_at"
     t.datetime "updated_at"
-    t.boolean  "most_recent"
+    t.boolean  "most_recent",                            null: false
   end
 
+  add_index "resource_export_file_transitions", ["resource_export_file_id", "most_recent"], name: "index_resource_export_file_transitions_parent_most_recent", unique: true
   add_index "resource_export_file_transitions", ["resource_export_file_id"], name: "index_resource_export_file_transitions_on_file_id"
   add_index "resource_export_file_transitions", ["sort_key", "resource_export_file_id"], name: "index_resource_export_file_transitions_on_sort_key_and_file_id", unique: true
 
@@ -1072,9 +1077,10 @@
     t.integer  "resource_import_file_id"
     t.datetime "created_at"
     t.datetime "updated_at"
-    t.boolean  "most_recent"
+    t.boolean  "most_recent",                            null: false
   end
 
+  add_index "resource_import_file_transitions", ["resource_import_file_id", "most_recent"], name: "index_resource_import_file_transitions_parent_most_recent", unique: true
   add_index "resource_import_file_transitions", ["resource_import_file_id"], name: "index_resource_import_file_transitions_on_file_id"
   add_index "resource_import_file_transitions", ["sort_key", "resource_import_file_id"], name: "index_resource_import_file_transitions_on_sort_key_and_file_id", unique: true
 
@@ -1184,7 +1190,7 @@
     t.string   "name",                         null: false
     t.text     "display_name"
     t.text     "note"
-    t.integer  "library_id",   default: 1,     null: false
+    t.integer  "library_id",                   null: false
     t.integer  "items_count",  default: 0,     null: false
     t.integer  "position"
     t.datetime "created_at"
@@ -1267,6 +1273,7 @@
 
   add_index "user_export_file_transitions", ["sort_key", "user_export_file_id"], name: "index_user_export_file_transitions_on_sort_key_and_file_id", unique: true
   add_index "user_export_file_transitions", ["user_export_file_id"], name: "index_user_export_file_transitions_on_file_id"
+  add_index "user_export_file_transitions", ["user_export_file_id"], name: "index_user_export_file_transitions_on_user_export_file_id"
 
   create_table "user_export_files", force: :cascade do |t|
     t.integer  "user_id"
@@ -1279,6 +1286,8 @@
     t.datetime "updated_at"
   end
 
+  add_index "user_export_files", ["user_id"], name: "index_user_export_files_on_user_id"
+
   create_table "user_group_has_checkout_types", force: :cascade do |t|
     t.integer  "user_group_id",                                   null: false
     t.integer  "checkout_type_id",                                null: false
@@ -1355,6 +1364,8 @@
     t.integer  "default_user_group_id"
   end
 
+  add_index "user_import_files", ["user_id"], name: "index_user_import_files_on_user_id"
+
   create_table "user_import_results", force: :cascade do |t|
     t.integer  "user_import_file_id"
     t.integer  "user_id"
@@ -1364,6 +1375,9 @@
     t.text     "error_message"
   end
 
+  add_index "user_import_results", ["user_id"], name: "index_user_import_results_on_user_id"
+  add_index "user_import_results", ["user_import_file_id"], name: "index_user_import_results_on_user_import_file_id"
+
   create_table "user_reserve_stat_transitions", force: :cascade do |t|
     t.string   "to_state"
     t.text     "metadata",             default: "{}"
@@ -1440,5 +1454,6 @@
 
   add_index "withdraws", ["basket_id"], name: "index_withdraws_on_basket_id"
   add_index "withdraws", ["item_id"], name: "index_withdraws_on_item_id"
+  add_index "withdraws", ["librarian_id"], name: "index_withdraws_on_librarian_id"
 
 end

spec/fixtures/inter_library_loan_transitions.yml

@@ -45,5 +45,5 @@ inter_library_loan_transition_00007:
 #  inter_library_loan_id :integer
 #  created_at            :datetime
 #  updated_at            :datetime
-#  most_recent           :boolean
+#  most_recent           :boolean          not null
 #