'add first pool to this smart pool' link was constructed incorrectly.
Signed-off-by: Scott Seago <sseago at redhat.com>
---
src/app/views/smart_pools/show_pools.rhtml | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/src/app/views/smart_pools/show_pools.rhtml
b/src/app/views/smart_pools/show_pools.rhtml
index 8e89701..fbf6858 100644
--- a/src/app/views/smart_pools/show_pools.rhtml
+++ b/src/app/views/smart_pools/show_pools.rhtml
@@ -67,7 +67,7 @@
<div class="no-grid-items-text">
No pools found in this pool. <br/><br/>
<%= image_tag "icon_addpool.png",
:style=>"vertical-align:middle;" %>
- <a href="<%= url_for :controller => 'pool',
:action => 'add_to_smart_pool', :smart_pool_id => @pool
%>" rel="facebox[.bolder]">Add first Pool to this smart
pool</a>
+ <a href="<%= url_for :controller =>
'smart_pools', :action => 'add_pool_dialog', :id => @pool
%>" rel="facebox[.bolder]">Add first Pool to this smart
pool</a>
</div>
</div>
</div>
--
1.5.5.1
On Mon, 2008-10-06 at 10:48 -0400, Scott Seago wrote:> 'add first pool to this smart pool' link was constructed incorrectly. > > Signed-off-by: Scott Seago <sseago at redhat.com> > --- > src/app/views/smart_pools/show_pools.rhtml | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/src/app/views/smart_pools/show_pools.rhtml b/src/app/views/smart_pools/show_pools.rhtml > index 8e89701..fbf6858 100644 > --- a/src/app/views/smart_pools/show_pools.rhtml > +++ b/src/app/views/smart_pools/show_pools.rhtml > @@ -67,7 +67,7 @@ > <div class="no-grid-items-text"> > No pools found in this pool. <br/><br/> > <%= image_tag "icon_addpool.png", :style=>"vertical-align:middle;" %> > - <a href="<%= url_for :controller => 'pool', :action => 'add_to_smart_pool', :smart_pool_id => @pool %>" rel="facebox[.bolder]">Add first Pool to this smart pool</a> > + <a href="<%= url_for :controller => 'smart_pools', :action => 'add_pool_dialog', :id => @pool %>" rel="facebox[.bolder]">Add first Pool to this smart pool</a> > </div> > </div> > </div>ACK, this works for me. However, you might want to change the image tag to point at the new file from my patch - icon_add_smartpool.png since this one (icon_addpool.png) does not exist. -j
Scott Seago wrote:> diff --git a/src/app/views/smart_pools/show_pools.rhtml b/src/app/views/smart_pools/show_pools.rhtml > index 8e89701..fbf6858 100644 > --- a/src/app/views/smart_pools/show_pools.rhtml > +++ b/src/app/views/smart_pools/show_pools.rhtml > @@ -67,7 +67,7 @@ > <div class="no-grid-items-text"> > No pools found in this pool. <br/><br/> > <%= image_tag "icon_addpool.png", :style=>"vertical-align:middle;" %> > - <a href="<%= url_for :controller => 'pool', :action => 'add_to_smart_pool', :smart_pool_id => @pool %>" rel="facebox[.bolder]">Add first Pool to this smart pool</a> > + <a href="<%= url_for :controller => 'smart_pools', :action => 'add_pool_dialog', :id => @pool %>" rel="facebox[.bolder]">Add first Pool to this smart pool</a> > </div> > </div> > </div>Seems like an obvious typo (using the wrong controller)... ACK -- Chris Lalancette