diff --git a/app/assets/javascripts/repository/state.js.coffee b/app/assets/javascripts/repository/state.js.coffee index c37c50a..9e3d2df 100644 --- a/app/assets/javascripts/repository/state.js.coffee +++ b/app/assets/javascripts/repository/state.js.coffee @@ -2,7 +2,7 @@ class Repository.State constructor: (@repository_id) -> poll_state: (last_state) -> - $.post('/repositories/' + @repository_id + '/state', + $.get('/repositories/' + @repository_id + '/state', last_state: last_state) schedule_poll_state: (last_state) -> diff --git a/app/views/repositories/show.html.erb b/app/views/repositories/show.html.erb index 6664bfd..e8a93ab 100644 --- a/app/views/repositories/show.html.erb +++ b/app/views/repositories/show.html.erb @@ -39,7 +39,7 @@
<%= t('repository.show.date_processing') %>:
-<%= form_tag(repository_state_with_date_path(@repository.id), method: "post", remote: true) do %> +<%= form_tag(repository_state_with_date_path(@repository.id), method: "get", remote: true) do %><%= label_tag :day, t("day") %>: <%= select_tag(:day, options_for_select(day_options), :style => "width:55px; margin-top:5px") %> diff --git a/config/routes.rb b/config/routes.rb index 9e14440..825c46f 100644 --- a/config/routes.rb +++ b/config/routes.rb @@ -9,9 +9,10 @@ Rails.application.routes.draw do resources :repositories, except: [:update, :index] get '/repositories/:id/modules/:module_result_id' => 'repositories#show', as: :repository_module - post '/repositories/:id/state' => 'repositories#state', as: :repository_state - post '/repositories/:id/state_with_date' => 'repositories#state_with_date', as: :repository_state_with_date + get '/repositories/:id/state' => 'repositories#state', as: :repository_state + get '/repositories/:id/state_with_date' => 'repositories#state_with_date', as: :repository_state_with_date put '/repositories/:id' => 'repositories#update', as: :repository_update + # This route should be a POST to be semantically correct. But, RepositoriesController#create relies on a redirect to it which is not possible with a POST get '/repositories/:id/process' => 'repositories#process_repository', as: :repository_process get '/repository_branches' => 'repositories#branches', as: :repository_branches @@ -35,7 +36,7 @@ Rails.application.routes.draw do put '/readings/:id' => 'readings#update', as: :reading_update end - #resources :modules + # Modules post '/modules/:id/metric_history' => 'modules#metric_history' post '/modules/:id/tree' => 'modules#load_module_tree' diff --git a/spec/routing/repositories_routing_spec.rb b/spec/routing/repositories_routing_spec.rb index 620a5c1..edd2c93 100644 --- a/spec/routing/repositories_routing_spec.rb +++ b/spec/routing/repositories_routing_spec.rb @@ -18,9 +18,9 @@ describe RepositoriesController, :type => :routing do to(controller: :repositories, action: :update, id: 1) } it { is_expected.not_to route(:get, '/repositories'). to(controller: :repositories, action: :index) } - it { is_expected.to route(:post, '/repositories/1/state'). + it { is_expected.to route(:get, '/repositories/1/state'). to(controller: :repositories, action: :state, id: 1) } - it { is_expected.to route(:post, '/repositories/1/state_with_date'). + it { is_expected.to route(:get, '/repositories/1/state_with_date'). to(controller: :repositories, action: :state_with_date, id: 1) } it { is_expected.to route(:get, '/repositories/1/process'). to(controller: :repositories, action: :process_repository, id: 1) } -- libgit2 0.21.2