diff --git a/app/controllers/issues_controller.rb b/app/controllers/issues_controller.rb index c24054f77..1baf533f4 100644 --- a/app/controllers/issues_controller.rb +++ b/app/controllers/issues_controller.rb @@ -1,4 +1,6 @@ class IssuesController < ApplicationController + include PaginationMethods + layout "site" before_action :authorize_web @@ -11,6 +13,8 @@ class IssuesController < ApplicationController before_action :check_database_writable, :only => [:resolve, :ignore, :reopen] def index + @params = params.permit(:before, :after, :limit, :status, :search_by_user, :issue_type, :last_updated_by) + @params[:limit] ||= 50 @title = t ".title" @issue_types = [] @@ -18,7 +22,7 @@ class IssuesController < ApplicationController @issue_types.push("DiaryEntry", "DiaryComment", "User") if current_user.administrator? @users = User.joins(:roles).where(:user_roles => { :role => current_user.roles.map(&:role) }).distinct - @issues = Issue.visible_to(current_user).order(:updated_at => :desc) + @issues = Issue.visible_to(current_user) # If search if params[:search_by_user].present? @@ -39,6 +43,8 @@ class IssuesController < ApplicationController last_updated_by = params[:last_updated_by].to_s == "nil" ? nil : params[:last_updated_by].to_i @issues = @issues.where(:updated_by => last_updated_by) end + + @issues, @newer_issues_id, @older_issues_id = get_page_items(@issues, :limit => @params[:limit]) end def show diff --git a/app/views/issues/_page.html.erb b/app/views/issues/_page.html.erb new file mode 100644 index 000000000..ed23fb8cd --- /dev/null +++ b/app/views/issues/_page.html.erb @@ -0,0 +1,34 @@ +
<%= t ".status" %> | +<%= t ".reports" %> | +<%= t ".reported_item" %> | +<%= t ".reported_user" %> | +<%= t ".last_updated" %> | +
---|---|---|---|---|
<%= t ".states.#{issue.status}" %> | +<%= link_to t(".reports_count", :count => issue.reports_count), issue %> | +<%= link_to reportable_title(issue.reportable), reportable_url(issue.reportable) %> | +<%= link_to issue.reported_user.display_name, issue.reported_user if issue.reported_user %> | ++ <% if issue.user_updated %> + <%= t ".last_updated_time_ago_user_html", :user => link_to(issue.user_updated.display_name, issue.user_updated), + :time_ago => friendly_date_ago(issue.updated_at) %> + <% else %> + <%= friendly_date_ago(issue.updated_at) %> + <% end %> + | +
<%= t ".issues_not_found" %>
<% else %> -<%= t ".status" %> | -<%= t ".reports" %> | -<%= t ".reported_item" %> | -<%= t ".reported_user" %> | -<%= t ".last_updated" %> | -
---|---|---|---|---|
<%= t ".states.#{issue.status}" %> | -<%= link_to t(".reports_count", :count => issue.reports_count), issue %> | -<%= link_to reportable_title(issue.reportable), reportable_url(issue.reportable) %> | -<%= link_to issue.reported_user.display_name, issue.reported_user if issue.reported_user %> | -- <% if issue.user_updated %> - <%= t ".last_updated_time_ago_user_html", :user => link_to(issue.user_updated.display_name, issue.user_updated), - :time_ago => friendly_date_ago(issue.updated_at) %> - <% else %> - <%= friendly_date_ago(issue.updated_at) %> - <% end %> - | -