Project

General

Profile

« Previous | Next » 

Revision d9a2ebac

Added by Ohad Levy almost 12 years ago

  • ID d9a2ebac6e9ec0082e60265b794f54a29a1f7e65

[SQL optimizations] - many small optimizations

This is one of a few patches aim to improve Foreman performance
  • Bookmarks have only users, not usergroups
  • added caching to the settings table
  • removed notice message lookups (we are not really using those)
  • added caching to top bar (tabs + bookmarks) and expiry.
  • removed non needed JS to load

View differences:

app/views/home/_topbar.rhtml
<%=image_tag("foreman.png", :class=>"logo") %><a class="brand logo-text" href="#">Foreman</a>
<!-- menu -->
<% if User.current -%>
<div class="nav-collapse nav1">
<ul class="nav" id="menu">
<% my_bookmarks = Bookmark.my_bookmarks %>
<% if User.current -%>
<% cache("tabs_and_title_bookmarks_for_#{User.current}") do %>
<div class="nav-collapse nav1">
<ul class="nav" id="menu">
<% my_bookmarks = Bookmark.my_bookmarks %>
<%= menu 'dashboard', my_bookmarks %>
<%= menu 'hosts', my_bookmarks %>
<%= menu 'reports', my_bookmarks, hash_for_reports_path.merge(:search => 'eventful = true') %>
<%= menu 'facts' , my_bookmarks, hash_for_fact_values_path %>
<%= menu 'audits', my_bookmarks, hash_for_audits_path %>
<%= menu 'statistics', my_bookmarks %>
</ul>
</div>
<div class="nav-collapse nav2">
<ul class="nav pull-right" id="menu2">
<%= render "home/settings" -%>
<% if SETTINGS[:login]%>
<%= menu 'dashboard', my_bookmarks %>
<%= menu 'hosts', my_bookmarks %>
<%= menu 'reports', my_bookmarks, hash_for_reports_path.merge(:search => 'eventful = true') %>
<%= menu 'facts' , my_bookmarks, hash_for_fact_values_path %>
<%= menu 'audits', my_bookmarks, hash_for_audits_path %>
<%= menu 'statistics', my_bookmarks %>
</ul>
</div>
<div class="nav-collapse nav2">
<ul class="nav pull-right" id="menu2">
<%= render "home/settings" -%>
<% if SETTINGS[:login]%>
<%= render 'home/user_dropdown' %>
<% end -%>
</ul>
</div>
<% end -%>
</ul>
</div>
<% end -%>
<% end -%>

Also available in: Unified diff