diff options
Diffstat (limited to 'app')
| -rw-r--r-- | app/assets/javascripts/select2.coffee | 1 | ||||
| -rw-r--r-- | app/controllers/devise/cas_sessions_controller.rb | 2 | ||||
| -rw-r--r-- | app/controllers/referentials_controller.rb | 7 | ||||
| -rw-r--r-- | app/models/referential.rb | 17 | ||||
| -rw-r--r-- | app/models/user.rb | 2 | ||||
| -rw-r--r-- | app/views/calendars/_form.html.slim | 4 | ||||
| -rw-r--r-- | app/views/referentials/_form.html.slim | 26 | ||||
| -rw-r--r-- | app/views/routes/_form.html.slim | 4 | ||||
| -rw-r--r-- | app/views/time_table_combinations/_form.html.slim | 12 | ||||
| -rw-r--r-- | app/views/time_tables/_form.html.slim | 2 |
10 files changed, 31 insertions, 46 deletions
diff --git a/app/assets/javascripts/select2.coffee b/app/assets/javascripts/select2.coffee index 5adaabda5..d54ddb811 100644 --- a/app/assets/javascripts/select2.coffee +++ b/app/assets/javascripts/select2.coffee @@ -20,6 +20,7 @@ bind_select2_ajax = (el, cfg = {}) -> delay: 125, processResults: (data, params) -> results: data minimumInputLength: 3 + placeholder: target.data('select2ed-placeholder') templateResult: (item) -> item.text templateSelection: (item) -> diff --git a/app/controllers/devise/cas_sessions_controller.rb b/app/controllers/devise/cas_sessions_controller.rb index 3b16603b9..0a9d9ecb2 100644 --- a/app/controllers/devise/cas_sessions_controller.rb +++ b/app/controllers/devise/cas_sessions_controller.rb @@ -20,7 +20,7 @@ class Devise::CasSessionsController < Devise::SessionsController if LoginPolicy.new(current_user).boiv? redirect_to after_sign_in_path_for(current_user) else - redirect_to cas_login_url, flash: {alert: t('devise.sessions.new.unauthorized')} + redirect_to root_path, flash: {alert: t('devise.sessions.new.unauthorized')} end end diff --git a/app/controllers/referentials_controller.rb b/app/controllers/referentials_controller.rb index e95670241..1239d512f 100644 --- a/app/controllers/referentials_controller.rb +++ b/app/controllers/referentials_controller.rb @@ -76,7 +76,6 @@ class ReferentialsController < BreadcrumbController alias_method :current_referential, :referential helper_method :current_referential - def resource @referential ||= current_organisation.find_referential(params[:id]) end @@ -132,13 +131,7 @@ class ReferentialsController < BreadcrumbController params.require(:referential).permit( :id, :name, - :slug, - :prefix, - :time_zone, - :upper_corner, - :lower_corner, :organisation_id, - :projection_type, :data_format, :archived_at, :created_from_id, diff --git a/app/models/referential.rb b/app/models/referential.rb index 22bfe0d25..0ce325bd6 100644 --- a/app/models/referential.rb +++ b/app/models/referential.rb @@ -4,9 +4,10 @@ class Referential < ActiveRecord::Base validates_presence_of :name validates_presence_of :slug validates_presence_of :prefix - validates_presence_of :time_zone - validates_presence_of :upper_corner - validates_presence_of :lower_corner + # Fixme #3657 + # validates_presence_of :time_zone + # validates_presence_of :upper_corner + # validates_presence_of :lower_corner validates_uniqueness_of :slug validates_uniqueness_of :name @@ -179,6 +180,8 @@ class Referential < ActiveRecord::Base before_validation :assign_line_and_stop_area_referential, :on => :create, if: :workbench, unless: :created_from before_validation :clone_associations, :on => :create, if: :created_from + before_validation :assign_slug, :on => :create + before_validation :assign_prefix, :on => :create before_create :create_schema, unless: :created_from after_create :clone_schema, if: :created_from @@ -288,6 +291,14 @@ class Referential < ActiveRecord::Base Apartment::Tenant.create slug end + def assign_slug + self.slug ||= "#{self.name.parameterize.gsub('-', '_')}_#{Time.now.to_i}" + end + + def assign_prefix + self.prefix = self.organisation.name.parameterize.gsub('-', '_') + end + def assign_line_and_stop_area_referential self.line_referential = workbench.line_referential self.stop_area_referential = workbench.stop_area_referential diff --git a/app/models/user.rb b/app/models/user.rb index 1f5260b28..1a06746da 100644 --- a/app/models/user.rb +++ b/app/models/user.rb @@ -46,7 +46,6 @@ class User < ActiveRecord::Base # TODO: Discuss the following behavior in the light of how the portal's permissions will evolve # boiv:edit-offer does not imply boiv:read-offer, which needs to be provided specifically for any connection rights self.permissions = extra[:permissions].include?('boiv:edit-offer') ? @@edit_offer_permissions : [] - self.permissions += extra[:permissions].grep( %r{^\Aboiv:read-offer\z} ) end def self.portail_api_request @@ -77,7 +76,6 @@ class User < ActiveRecord::Base # TODO: Discuss the following behavior in the light of how the portal's permissions will evolve # boiv:edit-offer does not imply boiv:read-offer, which needs to be provided specifically for any connection rights user.permissions = el['permissions'].include?('boiv:edit-offer') ? @@edit_offer_permissions : [] - user.permissions += el['permissions'].grep( %r{^\Aboiv:read-offer\z} ) user.save end end diff --git a/app/views/calendars/_form.html.slim b/app/views/calendars/_form.html.slim index 9aed9f7d8..3c152c61d 100644 --- a/app/views/calendars/_form.html.slim +++ b/app/views/calendars/_form.html.slim @@ -6,8 +6,8 @@ - if policy(@calendar).share? .form-group.has_switch - = f.label :shared, class: 'col-sm-4 control-label' - = f.input :shared, as: :boolean, checked_value: true, unchecked_value: false, label: content_tag(:span, t("#{@calendar.shared}"), class: 'switch-label', data: {checkedValue: t('true'), uncheckedValue: t('false')}), wrapper_html: { class: 'col-sm-8'} + = f.label :shared, class: 'col-sm-4 col-xs-5 control-label' + = f.input :shared, as: :boolean, checked_value: true, unchecked_value: false, label: content_tag(:span, t("#{@calendar.shared}"), class: 'switch-label', data: {checkedValue: t('true'), uncheckedValue: t('false')}), wrapper_html: { class: 'col-sm-8 col-xs-7'} .separator diff --git a/app/views/referentials/_form.html.slim b/app/views/referentials/_form.html.slim index f209b278e..31d71bcdc 100644 --- a/app/views/referentials/_form.html.slim +++ b/app/views/referentials/_form.html.slim @@ -3,34 +3,16 @@ .row .col-lg-12 = form.input :name - - - if @referential.new_record? - = form.input :slug, input_html: { title: t('formtastic.titles.referential.slug') } - - else - = form.input :slug, disabled: true, input_html: { title: t('formtastic.titles.referential.slug') } - - if @referential.new_record? - if @referential.created_from = form.input :created_from, disabled: true, input_html: { value: Referential.find(@referential.created_from_id).name } .hidden = form.input :created_from_id, as: :hidden - - = form.input :prefix, input_html: { title: t("formtastic.titles.referential.prefix") } - - else - if @referential.created_from = form.input :created_from, disabled: true, input_html: { value: Referential.find(@referential.created_from_id).name } - = form.input :prefix, input_html: { title: t("formtastic.titles.referential.prefix") } - - = form.input :projection_type, as: :select, collection: Referential.available_srids - = form.input :time_zone - - - = form.input :upper_corner, input_html: {title: t("formtastic.titles.referential.upper_corner")} - = form.input :lower_corner, input_html: {title: t("formtastic.titles.referential.lower_corner")} - .separator - + = form.simple_fields_for :metadatas do |subform| .row .col-lg-12 @@ -40,7 +22,7 @@ .alert.alert-danger - @referential.errors[:metadatas].each do |msg| p.small = "- #{msg}" - + .subform .nested-head .wrapper @@ -55,10 +37,10 @@ = t('simple_form.labels.referential.metadatas.periods.end') abbr title='requis' * div - + = subform.simple_fields_for :periods do |period_form| = render 'period_fields', f: period_form - + .links.nested-linker = link_to_add_association 'Ajouter une période', subform, :periods, class: 'btn btn-outline-primary' diff --git a/app/views/routes/_form.html.slim b/app/views/routes/_form.html.slim index ad389c482..bee86f91e 100644 --- a/app/views/routes/_form.html.slim +++ b/app/views/routes/_form.html.slim @@ -6,8 +6,8 @@ = f.input :published_name .form-group.has_switch - = f.label :wayback, class: 'col-sm-4 control-label' - = f.input :wayback, as: :boolean, checked_value: :straight_forward, unchecked_value: :backward, label: content_tag(:span, @route.wayback_text, class: 'switch-label', data: {checkedValue: t('enumerize.route.direction.straight_forward'), uncheckedValue: t('enumerize.route.direction.backward')}), wrapper_html: { class: 'col-sm-8'} + = f.label :wayback, class: 'col-sm-4 col-xs-5 control-label' + = f.input :wayback, as: :boolean, checked_value: :straight_forward, unchecked_value: :backward, label: content_tag(:span, @route.wayback_text, class: 'switch-label', data: {checkedValue: t('enumerize.route.direction.straight_forward'), uncheckedValue: t('enumerize.route.direction.backward')}), wrapper_html: { class: 'col-sm-8 col-xs-7'} = f.input :opposite_route_id, collection: @forward, disabled: @route.wayback.straight_forward?, wrapper_html: {class: input_opposite_route_id_css(@route, 'straight_forward')} diff --git a/app/views/time_table_combinations/_form.html.slim b/app/views/time_table_combinations/_form.html.slim index d8bebf0c4..b4f818828 100644 --- a/app/views/time_table_combinations/_form.html.slim +++ b/app/views/time_table_combinations/_form.html.slim @@ -2,21 +2,21 @@ .row .col-lg-12 .form-group.has_switch - = f.label :combined_type, class: 'col-sm-4 control-label required' do + = f.label :combined_type, class: 'col-sm-4 col-xs-5 control-label required' do = 'Type de calendriers ' abbr title='Champ requis' * - = f.input :combined_type, as: :boolean, checked_value: 'time_table', unchecked_value: 'calendar', required: false, label: content_tag(:span, t("time_table_combinations.combined_type.#{@combination.combined_type}"), class: 'switch-label', data: { checkedValue: 'Calendriers', uncheckedValue: 'Modèles de calendriers' }), wrapper_html: { class: 'col-sm-8' } + = f.input :combined_type, as: :boolean, checked_value: 'time_table', unchecked_value: 'calendar', required: false, label: content_tag(:span, t("time_table_combinations.combined_type.#{@combination.combined_type}"), class: 'switch-label', data: { checkedValue: 'Calendriers', uncheckedValue: 'Modèles de calendriers' }), wrapper_html: { class: 'col-sm-8 col-xs-7' } - = f.input :time_table_id, as: :select, input_html: {class: 'tt_combination_target', style: "width: 100%", data: { 'select2-ajax': 'true', term: 'comment_or_objectid_cont', url: referential_autocomplete_time_tables_path(@referential, format: :json)}}, wrapper_html: {class: @combination.combined_type != 'time_table' ? 'hidden' : ''} + = f.input :time_table_id, as: :select, input_html: {class: 'tt_combination_target', style: "width: 100%", data: { 'select2-ajax': 'true', 'select2ed-placeholder': 'Indiquez un calendrier...', term: 'comment_or_objectid_cont', url: referential_autocomplete_time_tables_path(@referential, format: :json)}}, wrapper_html: {class: @combination.combined_type != 'time_table' ? 'hidden' : ''} - = f.input :calendar_id, as: :select, input_html: { class: 'tt_combination_target', style: "width: 100%", data: { 'select2-ajax': 'true', term: 'name_cont', url: autocomplete_calendars_path}}, wrapper_html: {class: @combination.combined_type != 'calendar' ? 'hidden' : ''} + = f.input :calendar_id, as: :select, input_html: { class: 'tt_combination_target', style: "width: 100%", data: { 'select2-ajax': 'true', 'select2ed-placeholder': 'Indiquez un modèle de calendrier...', term: 'name_cont', url: autocomplete_calendars_path}}, wrapper_html: {class: @combination.combined_type != 'calendar' ? 'hidden' : ''} .separator .row .col-lg-12 - = f.label :operation, class: 'col-sm-4 control-label' - = f.input :operation, as: :radio_buttons, label: false, collection: TimeTableCombination.operations, label_method: lambda{|o| t("time_table_combinations.operations.#{o}")}, wrapper_html: { class: 'col-sm-8' } + = f.label :operation, class: 'col-sm-4 col-xs-5 control-label' + = f.input :operation, as: :radio_buttons, label: false, collection: TimeTableCombination.operations, label_method: lambda{|o| t("time_table_combinations.operations.#{o}")}, wrapper_html: { class: 'col-sm-8 col-xs-7' } = f.button :submit, t('actions.submit'), class: 'btn btn-default formSubmitr', form: 'tt_combination_form' diff --git a/app/views/time_tables/_form.html.slim b/app/views/time_tables/_form.html.slim index d4e1d838e..d06fdf444 100644 --- a/app/views/time_tables/_form.html.slim +++ b/app/views/time_tables/_form.html.slim @@ -5,7 +5,7 @@ = form.input :comment, :input_html => { :title => t("formtastic.titles#{format_restriction_for_locales(@referential)}.time_table.comment")} - if @time_table.new_record? && !@time_table.created_from - = form.input :calendar_id, as: :select, collection: current_organisation.calendars + = form.input :calendar_id, as: :select, input_html: { class: 'tt_target', style: "width: 100%", data: { 'select2-ajax': 'true', 'select2ed-placeholder': 'Indiquez un modèle de calendrier...', term: 'name_cont', url: autocomplete_calendars_path}} - if @time_table.created_from = form.input :created_from, disabled: true, input_html: { value: @time_table.created_from.comment } |
