From: Andreas Bauer --- src/webui/app/views/package/show.rhtml | 13 ++++++++----- src/webui/app/views/project/show.rhtml | 12 ++++++------ 2 files changed, 14 insertions(+), 11 deletions(-) diff --git a/src/webui/app/views/package/show.rhtml b/src/webui/app/views/package/show.rhtml index c32e7a4..b5b0e4a 100644 --- a/src/webui/app/views/package/show.rhtml +++ b/src/webui/app/views/package/show.rhtml @@ -6,10 +6,11 @@ <br/> <%= link_to "[Edit Package Information]", :action => :edit, :project => @project, :package => @package %> -<%= link_to "[Remove Package]", :action => :remove, :project => @project, :package => @package %> +<%= link_to "[Remove Package]", {:action => :remove, :project => @project, :package => @package}, + {:confirm => "Really remove package '#{@package}'?"}%> <h3>Files</h3> -<% if @package.has_element? :file %> +<% if @package.has_element? file %> <table> <% @package.each_file do |file| %> <tr> @@ -21,7 +22,8 @@ <% end %> </td> <td><%= link_to "[Download]", file_url( @project, @package, file.filename ) %></td> - <td><%= link_to "[Remove]", :action => :remove_file, :project => @project, :package => @package, :filename => file.filename %></td> + <td><%= link_to "[Remove]", {:action => :remove_file, :project => @project, :package => @package, :filename => file.filename}, + {:confirm => "Really remove file '#{file.filename}'?" }%></td> </tr> <% end %> </table> @@ -49,8 +51,9 @@ <%- if @package.each_person.length > 1 %> <td> - <%= link_to "[Remove User]", :action => :remove_person, :package => @package, - :project => params[:project], :userid => person.userid, :role => person.role %> + <%= link_to "[Remove User]", {:action => :remove_person, :package => @package, + :project => params[:project], :userid => person.userid, :role => person.role}, + {:confirm => "Really remove user '#{person.userid}'?"}%> </td> <%- end %> diff --git a/src/webui/app/views/project/show.rhtml b/src/webui/app/views/project/show.rhtml index 454317f..c69f0b9 100644 --- a/src/webui/app/views/project/show.rhtml +++ b/src/webui/app/views/project/show.rhtml @@ -32,9 +32,9 @@ </td> <td></td> <td></td> - <td><%= link_to "[Remove Target]", :action => :remove_target, + <td><%= link_to "[Remove Target]", {:action => :remove_target, :project => params[:project], - :target => repository.name %></td> + :target => repository.name}, {:confirm => "Really delete target '#{repository.name}'?"} %></td> </tr> <% repository.each_arch do |arch| -%> <tr> @@ -83,9 +83,9 @@ <td><%= person.role %> </td> <%- if @project.each_person.length > 1 %> <td> - <%= link_to "[Remove User]", :action => :remove_person, + <%= link_to "[Remove User]", {:action => :remove_person, :project => params[:project], :userid => person.userid, - :role => person.role%> + :role => person.role}, {:confirm => "Really remove user '#{person.userid}'?"}%> </td> <%- end %> @@ -112,8 +112,8 @@ <tr> <td><%= link_to package.name, :controller => 'package', :action => :show, :project=> @project, :package => package %></td> - <td><%= link_to "[Remove]", :controller => 'package', :action => :remove, - :project => @project, :package => package %></td> + <td><%= link_to "[Remove]", {:controller => 'package', :action => :remove, + :project => @project, :package => package}, {:confirm => "Really remove package '#{package.name}'?"} %></td> </tr> <% end -%> </table> -- 1.7.7 -- To unsubscribe, e-mail: obs-commits+unsubscribe@opensuse.org To contact the owner, e-mail: obs-commits+owner@opensuse.org