| Age | Commit message (Collapse) | Author |
|---|
|
|
|
|
|
Referential#stop_areas has_many through. Refs #5824
|
|
|
|
Ajout de snapshots dans les specs
|
|
|
|
|
|
Make `#checksum_replace_nil_or_empty_values` more readable.
Refs #5416
|
|
Add Johan's suggestion for simplifying this line. This allows us to get
rid of one of the `#map` calls.
Refs #5416
|
|
Replace `ParentImportNotifier` with the new generalised
`ParentNotifier`. This will allow us to use the same service for parent
notification of both imports and compliance check sets.
Delete the `ParentImportNotifier` class and spec as these are now
superseded by `ParentNotifier`.
Refs #4758
|
|
This is a takeoff on `ParentImportNotifier`. I just copied over the
class and spec and generalised the service to work with `Import`-style
interfaces. We can now use the same notifier service class for both
imports and compliance check sets.
Refs #4758
|
|
Copy over the `Import.abort_old` method to this class. For now I haven't
bothered with moving it to a consolidated module. We'll see, maybe
that's a better approach. For now I'm going with a relaxed copy-paste
solution until we decide that the increased complexity of the module is
worth the deduplication.
We need this method to be able to abort stale imports. We'll be doing
this, similarly to imports, in a regular cron job.
Refs #4758
|
|
|
|
menu link
|
|
Refs #5416
|
|
Refs #5416
|
|
|
|
|
|
|
|
This determines if the StopArea is commercial or not
The useless fields are hidden in the form for the non-commercials ones
|
|
|
|
|
|
This determines if the StopArea is commercial or not
The useless fields are hidden in the form for the non-commercials ones
|
|
|
|
|
|
Note: Did not fix the missing workgroup in the calendar mailer
|
|
|
|
|
|
scoping
|
|
|
|
This reverts commit c2bc391ee91cac70e726d188be97dd2323df0df2.
Thanks to 76abdbd66c16e6e1233685a5fa28a42cba5580d9, this temporary fix
can be reverted. We don't actually need `kind` selected here, it was
only added to appease the auto-initialisation of `kind` in `StopArea`.
Now that we don't set a default value for `kind` on initialisation, we
can remove it from this `select`.
|
|
We were getting this error on `ReferentialVehicleJourneys#index`:
Feb 1 17:48:32 chouette-dev rails/chouette[18900]: Started GET "/referentials/9/vehicle_journeys" for 172.28.10.14 at 2018-02-01 17:48:32 +0100
Feb 1 17:48:32 chouette-dev rails/chouette[18900]: Processing by ReferentialVehicleJourneysController#index as HTML
Feb 1 17:48:32 chouette-dev rails/chouette[18900]: Parameters: {"referential_id"=>"9"}
Feb 1 17:48:32 chouette-dev rails/chouette[18900]: locale set to :fr
Feb 1 17:48:32 chouette-dev rails/chouette[18900]: Rendered referential_vehicle_journeys/_filters.html.slim (37.0ms)
Feb 1 17:48:32 chouette-dev rails/chouette[18900]: Rendered referential_vehicle_journeys/index.html.slim within layouts/application (54.3ms)
Feb 1 17:48:32 chouette-dev rails/chouette[18900]: Completed 500 Internal Server Error in 73ms (ActiveRecord: 10.4ms)
Feb 1 17:48:32 chouette-dev rails/chouette[18900]: ActionView::Template::Error (missing attribute: kind):#012 23: = f.input :published_journey_name_lteq, label: false, wrapper_html: { class: 'w45'}#012 24: .form-group.togglable#012 25: = f.label Chouette::StopArea.model_name.human.pluralize, required: false, class: 'control-label'#012 26: = f.input :stop_area_ids, collection: @all_stop_areas.select(:id, :name).order(name: :asc), checked: params[:q] && params[:q][:stop_area_ids], as: :check_boxes, label: false, label_method: lambda{|l| ("<span>" + l.name + "</span>").html_safe}, required: false, wrapper_html: { class: 'checkbox_list'}, multiple: true#012 27: .form-group.togglable#012 28: = f.label Chouette::VehicleJourney.human_attribute_name(:purchase_window), class: 'control-label'#012 29: .filter_menu#012 app/models/chouette/stop_area.rb:51:in `block in <class:StopArea>'#012 app/views/referential_vehicle_journeys/_filters.html.slim:26:in `block in _app_views_referential_vehicle_journeys__filters_html_slim__692539656386688985_47285414929500'#012 app/views/referential_vehicle_journeys/_filters.html.slim:1:in `_app_views_referential_vehicle_journeys__filters_html_slim__692539656386688985_47285414929500'#012 app/views/referential_vehicle_journeys/index.html.slim:9:in `_app_views_referential_vehicle_journeys_index_html_slim___336434325333090796_47285419865660'
Feb 1 17:48:32 chouette-dev rails/chouette[18900]: Processing by ErrorsController#server_error as HTML
Feb 1 17:48:32 chouette-dev rails/chouette[18900]: Parameters: {"referential_id"=>"9"}
Feb 1 17:48:32 chouette-dev rails/chouette[18900]: locale set to :fr
or, this one locally which is easier to read:
ActiveModel::MissingAttributeError in ReferentialVehicleJourneys#index
Showing .../stif-boiv/app/views/referential_vehicle_journeys/_filters.html.slim where line #44 raised:
missing attribute: kind
Extracted source (around line #51):
after_initialize do
> self.kind ||= :commercial
end
def self.nullable_attributes
Trace of template inclusion: app/views/referential_vehicle_journeys/index.html.slim
Rails.root: .../stif-boiv
Application Trace | Framework Trace | Full Trace
app/models/chouette/stop_area.rb:51:in `block in <class:StopArea>'
app/views/referential_vehicle_journeys/_filters.html.slim:44:in `block in _app_views_referential_vehicle_journeys__filters_html_slim___131528464712770693_70134715599760'
app/views/referential_vehicle_journeys/_filters.html.slim:1:in `_app_views_referential_vehicle_journeys__filters_html_slim___131528464712770693_70134715599760'
app/views/referential_vehicle_journeys/index.html.slim:9:in `_app_views_referential_vehicle_journeys_index_html_slim__3124451149445015007_70134715309040'
This is because on `ReferentialVehicleJourneys#index`, the StopArea
filter makes this query:
@all_stop_areas.select(:id, :name).order(name: :asc)
Alban added a temporary fix for the release in
c2bc391ee91cac70e726d188be97dd2323df0df2.
After a discussion with Luc, he said we only want the initial value on
the `#new` form, and we decided to take this out of the model and put it
in the form template instead.
Here, when a new `StopArea` is created, `:commercial` will be pre-filled
in the `kind` field by default. On the `#edit` page, it should show the
existing value of `kind` but not permit editing.
Refs #5817
|
|
|
|
|
|
|
|
|
|
|
|
|
|
Note: Did not fix the missing workgroup in the calendar mailer
|
|
|
|
|
|
scoping
|
|
This determines if the StopArea is commercial or not
The useless fields are hidden in the form for the non-commercials ones
|
|
|
|
|
|
Note: Did not fix the missing workgroup in the calendar mailer
|
|
|
|
|
|
scoping
|
|
|
