Feature/database migrations

app/models/accommodation.rb

@@ -1,5 +1,5 @@
 class Accommodation < ApplicationRecord
-  belongs_to :details
+  belongs_to :accomodation_details
 
   validates :name, presence: true, length: { minimum: 3, maximum: 255 }
   validates :image_url, presence: true, format: { with: %r{\Ahttps?://\S+\z}, message: 'Invalid URL format' }

app/models/accommodation_detail.rb

@@ -1,6 +1,4 @@
 class AccommodationDetail < ApplicationRecord
-  belongs_to :accommodation
-
   validates :accommodation, presence: true
   validates :accommodation_type, presence: true
   validates :bedrooms, presence: true, numericality: { only_integer: true, greater_than_or_equal_to: 1 }

app/models/reservation.rb

@@ -2,4 +2,15 @@ class Reservation < ApplicationRecord
   belongs_to :user
   belongs_to :accommodation
   belongs_to :city
+
+  validates_presence_of :user, :accommodation, :schedule_date, :city
+  validate :schedule_date_in_future
+
+  private
+
+  def schedule_date_in_future
+    return unless schedule_date.present? && schedule_date <= Date.today
+
+    errors.add(:schedule_date, 'must be in the future')
+  end
 end

db/migrate/20231026162755_create_accommodation_details.rb → db/migrate/20231026154555_create_accommodation_details.rb

@@ -1,7 +1,6 @@
 class CreateAccommodationDetails < ActiveRecord::Migration[7.1]
   def change
     create_table :accommodation_details do |t|
-      t.references :accommodation, null: false, foreign_key: true
       t.string :acommodation_type
       t.integer :bedrooms
       t.integer :beds

db/migrate/20231026162727_create_accommodations.rb → db/migrate/20231026154727_create_accommodations.rb

@@ -2,7 +2,7 @@ class CreateAccommodations < ActiveRecord::Migration[7.1]
   def change
     create_table :accommodations do |t|
       t.string :name
-      t.references :details, null: false, foreign_key: true
+      t.references :accommodation_details, null: false, foreign_key: true
       t.string :image_url
 
       t.timestamps

db/migrate/20231026155958_create_reservations.rb

@@ -7,18 +7,6 @@ def change
       t.references :city, null: false, foreign_key: true
 
       t.timestamps
-
-      # Validations
-      validates_presence_of :user, :accommodation, :schedule_date, :city
-      validate :schedule_date_in_future
-
-      private
-
-      def schedule_date_in_future
-        if schedule_date.present? && schedule_date <= Date.today
-          errors.add(:schedule_date, "must be in the future")
-        end
-      end
     end
   end
 end