X-Git-Url: https://git.openstreetmap.org./rails.git/blobdiff_plain/8bc8e537b273e8e260a12f598d77dc4135f9d32c..caa1a0128a057bf7358cf60926e0c243113b5a50:/app/controllers/browse_controller.rb diff --git a/app/controllers/browse_controller.rb b/app/controllers/browse_controller.rb index 4de490b2c..0ac45b926 100644 --- a/app/controllers/browse_controller.rb +++ b/app/controllers/browse_controller.rb @@ -1,105 +1,90 @@ class BrowseController < ApplicationController - layout 'site' + layout :map_layout - before_filter :authorize_web - before_filter :set_locale - before_filter { |c| c.check_database_readable(true) } + before_action :authorize_web + before_action :set_locale + before_action :except => [:query] { |c| c.check_database_readable(true) } + before_action :require_oauth + around_action :web_timeout - def start - end - - - def relation - @relation = Relation.find(params[:id]) - @next = Relation.find(:first, :order => "id ASC", :conditions => [ "visible = true AND id > :id", { :id => @relation.id }] ) - @prev = Relation.find(:first, :order => "id DESC", :conditions => [ "visible = true AND id < :id", { :id => @relation.id }] ) - rescue ActiveRecord::RecordNotFound @type = "relation" - render :action => "not_found", :status => :not_found - end - - def relation_history - @relation = Relation.find(params[:id]) + @feature = Relation.find(params[:id]) + render "feature" rescue ActiveRecord::RecordNotFound - @type = "relation" render :action => "not_found", :status => :not_found end - def relation_map - @relation = Relation.find(params[:id]) - rescue ActiveRecord::RecordNotFound + def relation_history @type = "relation" + @feature = Relation.find(params[:id]) + render "history" + rescue ActiveRecord::RecordNotFound render :action => "not_found", :status => :not_found end - - def way - @way = Way.find(params[:id], :include => [:way_tags, {:changeset => :user}, {:nodes => [:node_tags, {:ways => :way_tags}]}, :containing_relation_members]) - @next = Way.find(:first, :order => "id ASC", :conditions => [ "visible = true AND id > :id", { :id => @way.id }] ) - @prev = Way.find(:first, :order => "id DESC", :conditions => [ "visible = true AND id < :id", { :id => @way.id }] ) - # Used for edit link, takes approx middle node of way - @midnode = @way.nodes[@way.nodes.length/2] - rescue ActiveRecord::RecordNotFound + def way @type = "way" - render :action => "not_found", :status => :not_found - end - - def way_history - @way = Way.find(params[:id], :include => [:way_tags, {:old_ways => {:changeset => :user}}]) + @feature = Way.preload(:way_tags, :containing_relation_members, :changeset => :user, :nodes => [:node_tags, :ways => :way_tags]).find(params[:id]) + render "feature" rescue ActiveRecord::RecordNotFound - @type = "way" render :action => "not_found", :status => :not_found end - def way_map - @way = Way.find(params[:id]) - rescue ActiveRecord::RecordNotFound + def way_history @type = "way" + @feature = Way.preload(:way_tags, :old_ways => { :changeset => :user }).find(params[:id]) + render "history" + rescue ActiveRecord::RecordNotFound render :action => "not_found", :status => :not_found end def node - @node = Node.find(params[:id]) - @next = Node.find(:first, :order => "id ASC", :conditions => [ "visible = true AND id > :id", { :id => @node.id }] ) - @prev = Node.find(:first, :order => "id DESC", :conditions => [ "visible = true AND id < :id", { :id => @node.id }] ) - rescue ActiveRecord::RecordNotFound @type = "node" - render :action => "not_found", :status => :not_found - end - - def node_history - @node = Node.find(params[:id]) + @feature = Node.find(params[:id]) + render "feature" rescue ActiveRecord::RecordNotFound - @type = "way" render :action => "not_found", :status => :not_found end - def node_map - @node = Node.find(params[:id]) - rescue ActiveRecord::RecordNotFound + def node_history @type = "node" + @feature = Node.find(params[:id]) + render "history" + rescue ActiveRecord::RecordNotFound render :action => "not_found", :status => :not_found end - + def changeset + @type = "changeset" @changeset = Changeset.find(params[:id]) - @node_pages, @nodes = paginate(:old_nodes, :conditions => {:changeset_id => @changeset.id}, :per_page => 20, :parameter => 'node_page') - @way_pages, @ways = paginate(:old_ways, :conditions => {:changeset_id => @changeset.id}, :per_page => 20, :parameter => 'way_page') - @relation_pages, @relations = paginate(:old_relations, :conditions => {:changeset_id => @changeset.id}, :per_page => 20, :parameter => 'relation_page') - - @title = "#{I18n.t('browse.changeset.title')} | #{@changeset.id}" - @next = Changeset.find(:first, :order => "id ASC", :conditions => [ "id > :id", { :id => @changeset.id }] ) - @prev = Changeset.find(:first, :order => "id DESC", :conditions => [ "id < :id", { :id => @changeset.id }] ) + if @user && @user.moderator? + @comments = @changeset.comments.unscope(:where => :visible).includes(:author) + else + @comments = @changeset.comments.includes(:author) + end + @node_pages, @nodes = paginate(:old_nodes, :conditions => { :changeset_id => @changeset.id }, :per_page => 20, :parameter => "node_page") + @way_pages, @ways = paginate(:old_ways, :conditions => { :changeset_id => @changeset.id }, :per_page => 20, :parameter => "way_page") + @relation_pages, @relations = paginate(:old_relations, :conditions => { :changeset_id => @changeset.id }, :per_page => 20, :parameter => "relation_page") + if @changeset.user.data_public? + @next_by_user = @changeset.user.changesets.where("id > ?", @changeset.id).reorder(:id => :asc).first + @prev_by_user = @changeset.user.changesets.where("id < ?", @changeset.id).reorder(:id => :desc).first + end rescue ActiveRecord::RecordNotFound - @type = "changeset" render :action => "not_found", :status => :not_found end - def changeset_map - @changeset = Changeset.find(params[:id]) + def note + @type = "note" + + if @user && @user.moderator? + @note = Note.find(params[:id]) + @note_comments = @note.comments.unscope(:where => :visible) + else + @note = Note.visible.find(params[:id]) + @note_comments = @note.comments + end rescue ActiveRecord::RecordNotFound - @type = "changeset" render :action => "not_found", :status => :not_found end end