Commit 382ad0462322959728da2acf9f9eb3e93149fe5e
Committed by
Rafael Manzo
1 parent
d18a1144
Exists in
colab
and in
4 other branches
removing repository is working, editing view is rendered but not saved
Showing
2 changed files
with
5 additions
and
3 deletions
Show diff stats
app/controllers/repositories_controller.rb
| @@ -25,6 +25,8 @@ class RepositoriesController < ApplicationController | @@ -25,6 +25,8 @@ class RepositoriesController < ApplicationController | ||
| 25 | # GET /repositories/1/edit | 25 | # GET /repositories/1/edit |
| 26 | def edit | 26 | def edit |
| 27 | @project = Project.find(params[:project_id]) | 27 | @project = Project.find(params[:project_id]) |
| 28 | + set_repository | ||
| 29 | + @repository_types = KalibroEntities::Entities::Repository.repository_types | ||
| 28 | end | 30 | end |
| 29 | 31 | ||
| 30 | # POST /repositories | 32 | # POST /repositories |
| @@ -68,7 +70,7 @@ class RepositoriesController < ApplicationController | @@ -68,7 +70,7 @@ class RepositoriesController < ApplicationController | ||
| 68 | @project = Project.find(params[:project_id]) | 70 | @project = Project.find(params[:project_id]) |
| 69 | @repository.destroy | 71 | @repository.destroy |
| 70 | respond_to do |format| | 72 | respond_to do |format| |
| 71 | - format.html { redirect_to project_repositories_path(@project) } | 73 | + format.html { redirect_to project_path(@project) } |
| 72 | format.json { head :no_content } | 74 | format.json { head :no_content } |
| 73 | end | 75 | end |
| 74 | end | 76 | end |
| @@ -76,7 +78,7 @@ class RepositoriesController < ApplicationController | @@ -76,7 +78,7 @@ class RepositoriesController < ApplicationController | ||
| 76 | private | 78 | private |
| 77 | # Use callbacks to share common setup or constraints between actions. | 79 | # Use callbacks to share common setup or constraints between actions. |
| 78 | def set_repository | 80 | def set_repository |
| 79 | - @repository = Repository.find(params[:id]) | 81 | + @repository = Repository.find(params[:id].to_i) |
| 80 | end | 82 | end |
| 81 | 83 | ||
| 82 | # Never trust parameters from the scary internet, only allow the white list through. | 84 | # Never trust parameters from the scary internet, only allow the white list through. |
app/views/repositories/edit.html.erb
| @@ -2,5 +2,5 @@ | @@ -2,5 +2,5 @@ | ||
| 2 | 2 | ||
| 3 | <%= render 'form' %> | 3 | <%= render 'form' %> |
| 4 | 4 | ||
| 5 | -<%= link_to 'Show', @repository %> | | 5 | +<%= link_to 'Show', project_repository_path(@project, @repository) %> | |
| 6 | <%= link_to 'Back', project_repositories_path(@project) %> | 6 | <%= link_to 'Back', project_repositories_path(@project) %> |