Browse Source

Fix controller and don't display the previous day if location availability start afterwards

pull/1007/head
Kat 4 years ago
parent
commit
53b706c32d
  1. 2
      app/controllers/locations_controller.rb
  2. 3
      app/helpers/locations_helper.rb

2
app/controllers/locations_controller.rb

@ -62,7 +62,7 @@ class LocationsController < ApplicationController
flash[:notice] = reactivate_success_notice flash[:notice] = reactivate_success_notice
redirect_to scheme_location_path(@scheme, @location) redirect_to scheme_location_path(@scheme, @location)
else else
render "toggle_active", locals: { action: "deactivate" }, status: :unprocessable_entity render "toggle_active", locals: { action: "reactivate" }, status: :unprocessable_entity
end end
end end

3
app/helpers/locations_helper.rb

@ -48,7 +48,8 @@ module LocationsHelper
sorted_deactivation_periods = location.location_deactivation_periods.sort_by(&:deactivation_date) sorted_deactivation_periods = location.location_deactivation_periods.sort_by(&:deactivation_date)
deactivation_open = false deactivation_open = false
sorted_deactivation_periods.each do |deactivation| sorted_deactivation_periods.each do |deactivation|
availability << " to #{(deactivation.deactivation_date - 1.day).to_formatted_s(:govuk_date)}\nDeactivated on #{deactivation.deactivation_date.to_formatted_s(:govuk_date)}" unless deactivation_open from = deactivation.deactivation_date == location.available_from ? deactivation.deactivation_date : deactivation.deactivation_date - 1.day
availability << " to #{from.to_formatted_s(:govuk_date)}\nDeactivated on #{deactivation.deactivation_date.to_formatted_s(:govuk_date)}" unless deactivation_open
if deactivation.reactivation_date.present? if deactivation.reactivation_date.present?
availability << "\nActive from #{deactivation.reactivation_date.to_formatted_s(:govuk_date)}" availability << "\nActive from #{deactivation.reactivation_date.to_formatted_s(:govuk_date)}"
deactivation_open = false deactivation_open = false

Loading…
Cancel
Save