Browse Source

[Issue #10] Hide cancel button when expense is already cancelled

Jacqueline Maldonado 7 years ago
parent
commit
926d3523e3
1 changed files with 4 additions and 2 deletions
  1. 4 2
      app/views/expenses/_expenses_for_admin.html.erb

+ 4 - 2
app/views/expenses/_expenses_for_admin.html.erb

@@ -52,7 +52,7 @@
                   </td>
                   <td class="text-center">
                     <% if expense.status == "active" %>
-                      <%= link_to expense , method: :delete, :class => "btn btn-icon-only btn-danger", :title=>"Cancelar egreso", data: { confirm: '¿Esta seguro que desea cancelar el egreso?'}   do %> <i class="fa fa-ban"></i><% end %>
+                      <%= link_to expense, method: :delete, :class => "btn btn-icon-only btn-danger", :title=>"Cancelar egreso", data: { confirm: '¿Está seguro que desea cancelar el egreso?' } do %> <i class="fa fa-ban"></i><% end %>
                     <% end %>
                   </td>
                 </tr>
@@ -96,7 +96,9 @@
                     <% end %>
                   </td>
                   <td class="text-center">
-                      <%= link_to expense , method: :delete, :class => "btn btn-icon-only btn-danger", :title=>"Cancelar egreso", data: { confirm: '¿Esta seguro que desea cancelar el egreso?'}   do %> <i class="fa fa-ban"></i><% end %>
+                    <% if expense.active? %>
+                      <%= link_to expense, method: :delete, :class => "btn btn-icon-only btn-danger", :title=>"Cancelar egreso", data: { confirm: '¿Está seguro que desea cancelar el egreso?' } do %> <i class="fa fa-ban"></i><% end %>
+                    <% end %>
                   </td>
                 </tr>
                 <% end %>