Permalink
Browse files

fixes some links and paths

  • Loading branch information...
1 parent e49ccc4 commit ad8b87168fdbb7e75af35f1269780b3f6882e629 Luciano Sousa committed May 13, 2012
@@ -2,14 +2,14 @@
<div class="home-information create-database">
<h2>Create a new database</h2>
- <% form_for @datab do |f| %>
+ <%= form_for databases_path(@datab) do |f| %>
<div class="field">
<%= f.label :name, 'Name' %>
<%= f.text_field :name %>
</div>
<div class="field">
<%= f.label :collation, 'Collation' %>
- <%= select_tag 'datab[collation]', option_groups_from_collection_for_select(Datab::Collations, "values or []", "title", "value", "title"), :id => 'datab_collation' %>
+ <%#= select_tag 'datab[collation]', option_groups_from_collection_for_select(Datab::Collations, "values or []", "title", "value", "title"), :id => 'datab_collation' %>
<%= f.submit 'Create' %>
</div>
<% end %>
@@ -39,11 +39,11 @@
<th>Name</th>
<th>Tables</th>
</tr>
-<% for datab in @databs %>
+<% @databs.each do |datab| %>
<tr>
- <td><%= link_to image_tag('delete.png', :title => 'destroy this database'), datab_path(datab), :method => :delete, :confirm => 'Are you sure you want delete this database?' %></td>
- <td><%= link_to datab.name, datab %></td>
- <td><%= datab.tables.size %></td>
+ <td><%#= link_to image_tag('delete.png', :title => 'destroy this database'), datab_path(datab), :method => :delete, :confirm => 'Are you sure you want delete this database?' %></td>
+ <td><%= link_to datab.name, database_path(datab) %></td>
+ <td><%#= datab.tables.size %></td>
</tr>
<% end %>
</table>
@@ -1,5 +1,5 @@
<% @title = "#{@datab.name}" %>
-<%= link_to "Delete", @datab, :method => :delete, :confirm => "Are you really sure you want to destroy the #{@datab.name} database?" %>
+<%= link_to "Delete", database_path(@datab), :method => :delete, :confirm => "Are you really sure you want to destroy the #{@datab.name} database?" %>
<% if @datab.tables.empty? %>
<div class="notification">This database is empty</div>
<% else %>
@@ -21,7 +21,7 @@
<% for table in @datab.tables %>
<tr>
<td><%= table_icon table %></td>
- <td><%= link_to table.name, datab_table_path(@datab, table) %></td>
+ <td><%= link_to table.name, databases_table_path(@datab, table) %></td>
<td class="int" sorttable_customkey="<%= table.rows %>"><%= number_with_delimiter table.rows %></td>
<td class="int" sorttable_customkey="<%= table.data_length.to_i %>"><%= number_to_human_size table.data_length.to_i %></td>
<td><%= table.engine %></td>
@@ -46,7 +46,7 @@
<% end %>
<div class="create_table">
- <% form_tag new_datab_table_path(@datab), :method => :get do %>
+ <% form_tag new_database_table_path(@datab), :method => :get do %>
<div class"field">
Create a new table: <%= text_field_tag :name %>
<%= submit_tag 'Create' %>
@@ -1,5 +1,5 @@
<div class="header">
- <h1 class="title"><%= link_to "<i>rb</i>DB", databs_path %></h1>
+ <h1 class="title"><%= link_to "<i>rb</i>DB", databases_path %></h1>
<div class="db-selection">
@@ -12,7 +12,7 @@
<script type="text/javascript" charset="utf-8">
function onchange_database(url) {
- window.location = (url == "") ? '<%= databs_path %>' : url
+ window.location = (url == "") ? '<%= databases_path %>' : url
}
</script>
my databases:
@@ -30,7 +30,7 @@
<% current_prefix = prefix %>
<% end %>
<% selected = (@datab and @datab.name == db.name) %>
- <option value="<%= datab_url(db.id) %>"<%= 'selected="selected"' if selected %>>
+ <option value="<%#= datab_url(db.id) %>"<%= 'selected="selected"' if selected %>>
<%= suffix %>
</option>
<% end %>
@@ -39,29 +39,29 @@
<ul class="menu">
<%= menu = [
- { :title => 'Databases', :path => databs_path },
+ { :title => 'Databases', :path => databases_path },
{
:title => 'Tables',
- :path => datab_path(@datab),
+ :path => @datab,
:if => (@datab and !@datab.new_record?)
},
{
:title => 'SQL Query',
- :path => new_datab_sql_path(@datab),
+ :path => '',
@thiagocifani

thiagocifani May 18, 2012

A rota foi removida aqui, e em outros lugares também, como lembraremos dessa ausencia?

@fernandokosh

fernandokosh May 18, 2012

Owner

Abre um issue. Acho que é o melhor jeito.

@lucianosousa

lucianosousa May 18, 2012

foi uma forma de fazer o projeto funcionar. pressa... foi falha minha não documentar isso.
tem muita coisa nesse projetinho que tá macarrão... olhando a lógica dele dá pra começar uma parada do zero bem melhor...

@tauil

tauil May 21, 2012

Também achei bem macarrão em diversos pontos. Derrepente era uma boa mostrar essa lógica no próximo encontro e checar com a galera se vale a pena terminar de portar e arrumar as tosqueiras ou se é mais válido começar um novo usando a mesma lógica.

@lunks

lunks May 21, 2012

Acho que tá razoavelmente legal, mas se quiserem refazer, fiquem à vontade. :P

@tauil

tauil May 23, 2012

Acho válido trocar uma idéia sobre isso semana que vem no ForkinRails. @lunks Vê se aparece!

:if => (@datab and !@datab.new_record?)
},
{ :title => 'Import/Export', :path => nil },
{ :title => 'Report', :path => nil },
{
:title => 'Environment',
- :path => environments_path,
+ :path => '',
:if => (@datab and @datab.new_record? or request.path == environments_path)
}
]
menu.map do |entry|
next if entry.has_key?(:if) and !entry[:if]
active = request.path == entry[:path]
- v = entry[:path].nil? ? entry[:title] : link_to(entry[:title], entry[:path])
+ v = entry[:path].nil? ? entry[:title] : link_to('entry[:title], entry[:path]')
"<li#{' class="current"' if active}>#{v}</li>"
end.join %>
</ul>
@@ -3,4 +3,4 @@
:button_text => "Execute"
} %>
-<%= link_to 'Back', datab_sqls_path %>
+<%= link_to 'Back', database_sql_path %>

0 comments on commit ad8b871

Please sign in to comment.