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