1 class NoteController < ApplicationController
3 layout 'site', :only => [:mine]
5 before_filter :check_api_readable
6 before_filter :authorize_web, :only => [:create, :close, :update, :delete, :mine]
7 before_filter :check_api_writable, :only => [:create, :close, :update, :delete]
8 before_filter :require_moderator, :only => [:delete]
9 before_filter :set_locale, :only => [:mine]
10 after_filter :compress_output
11 around_filter :api_call_handle_error, :api_call_timeout
13 # Help methods for checking boundary sanity and area size
20 if bbox and bbox.count(',') == 3
21 bbox = bbox.split(',')
22 @min_lon, @min_lat, @max_lon, @max_lat = sanitise_boundaries(bbox)
24 #Fallback to old style, this is deprecated and should not be used
25 raise OSM::APIBadUserInput.new("No l was given") unless params['l']
26 raise OSM::APIBadUserInput.new("No r was given") unless params['r']
27 raise OSM::APIBadUserInput.new("No b was given") unless params['b']
28 raise OSM::APIBadUserInput.new("No t was given") unless params['t']
30 @min_lon = params['l'].to_f
31 @max_lon = params['r'].to_f
32 @min_lat = params['b'].to_f
33 @max_lat = params['t'].to_f
37 conditions = closedCondition
39 check_boundaries(@min_lon, @min_lat, @max_lon, @max_lat, MAX_NOTE_REQUEST_AREA)
41 @notes = Note.find_by_area(@min_lat, @min_lon, @max_lat, @max_lon, :include => :comments, :order => "updated_at DESC", :limit => limit, :conditions => conditions)
43 respond_to do |format|
44 format.html {render :template => 'note/list.rjs', :content_type => "text/javascript"}
45 format.rss {render :template => 'note/list.rss'}
47 format.xml {render :template => 'note/list.xml'}
48 format.json { render :json => @notes.to_json(:methods => [:lat, :lon], :only => [:id, :status, :created_at], :include => { :comments => { :only => [:author_name, :created_at, :body]}}) }
49 format.gpx {render :template => 'note/list.gpx'}
54 raise OSM::APIBadUserInput.new("No lat was given") unless params['lat']
55 raise OSM::APIBadUserInput.new("No lon was given") unless params['lon']
56 raise OSM::APIBadUserInput.new("No text was given") unless params['text']
58 lon = params['lon'].to_f
59 lat = params['lat'].to_f
60 comment = params['text']
63 name = params['name'] if params['name']
65 #Include in a transaction to ensure that there is always a note_comment for every note
67 @note = Note.create_bug(lat, lon)
69 #TODO: move this into a helper function
71 url = "http://nominatim.openstreetmap.org/reverse?lat=" + lat.to_s + "&lon=" + lon.to_s + "&zoom=16"
72 response = REXML::Document.new(Net::HTTP.get(URI.parse(url)))
74 if result = response.get_text("reversegeocode/result")
75 @note.nearby_place = result.to_s
77 @note.nearby_place = "unknown"
79 rescue Exception => err
80 @note.nearby_place = "unknown"
85 add_comment(@note, comment, name, "opened")
92 raise OSM::APIBadUserInput.new("No id was given") unless params['id']
93 raise OSM::APIBadUserInput.new("No text was given") unless params['text']
96 name = params['name'] if params['name']
98 id = params['id'].to_i
101 raise OSM::APINotFoundError unless note
102 raise OSM::APIAlreadyDeletedError unless note.visible
105 add_comment(note, params['text'], name, "commented")
112 raise OSM::APIBadUserInput.new("No id was given") unless params['id']
114 id = params['id'].to_i
116 name = params['name'] if params['name']
118 note = Note.find_by_id(id)
119 raise OSM::APINotFoundError unless note
120 raise OSM::APIAlreadyDeletedError unless note.visible
124 add_comment(note, :nil, name, "closed")
132 conditions = closedCondition
134 # Figure out the bbox
135 bbox = params['bbox']
137 if bbox and bbox.count(',') == 3
138 bbox = bbox.split(',')
139 @min_lon, @min_lat, @max_lon, @max_lat = sanitise_boundaries(bbox)
141 check_boundaries(@min_lon, @min_lat, @max_lon, @max_lat, MAX_NOTE_REQUEST_AREA)
143 conditions = cond_merge conditions, [OSM.sql_for_area(@min_lat, @min_lon, @max_lat, @max_lon)]
146 @comments = NoteComment.find(:all, :limit => limit, :order => "created_at DESC", :joins => :note, :include => :note, :conditions => conditions)
147 render :template => 'note/rss.rss'
151 @note = Note.find(params['id'])
152 raise OSM::APINotFoundError unless @note
153 raise OSM::APIAlreadyDeletedError unless @note.visible
155 respond_to do |format|
158 format.json { render :json => @note.to_json(:methods => [:lat, :lon], :only => [:id, :status, :created_at], :include => { :comments => { :only => [:author_name, :created_at, :body]}}) }
164 note = note.find(params['id'])
165 raise OSM::APINotFoundError unless note
166 raise OSM::APIAlreadyDeletedError unless note.visible
169 note.status = "hidden"
171 add_comment(note, :nil, name, "hidden")
174 render :text => "ok\n", :content_type => "text/html"
178 raise OSM::APIBadUserInput.new("No query string was given") unless params['q']
180 conditions = closedCondition
181 conditions = cond_merge conditions, ['note_comments.body ~ ?', params['q']]
183 #TODO: There should be a better way to do this. CloseConditions are ignored at the moment
185 @notes = Note.find(:all, :limit => limit, :order => "updated_at DESC", :joins => :comments, :include => :comments, :conditions => conditions).uniq
186 respond_to do |format|
187 format.html {render :template => 'note/list.rjs', :content_type => "text/javascript"}
188 format.rss {render :template => 'note/list.rss'}
190 format.xml {render :template => 'note/list.xml'}
191 format.json { render :json => @notes.to_json(:methods => [:lat, :lon], :only => [:id, :status, :created_at], :include => { :comments => { :only => [:author_name, :created_at, :body]}}) }
192 format.gpx {render :template => 'note/list.gpx'}
197 if params[:display_name]
198 @user2 = User.find_by_display_name(params[:display_name], :conditions => { :status => ["active", "confirmed"] })
201 if @user2.data_public? or @user2 == @user
202 conditions = ['note_comments.author_id = ?', @user2.id]
204 conditions = ['false']
206 else #if request.format == :html
207 @title = t 'user.no_such_user.title'
208 @not_found_user = params[:display_name]
209 render :template => 'user/no_such_user', :status => :not_found
215 user_link = render_to_string :partial => "user", :object => @user2
218 @title = t 'note.mine.title', :user => @user2.display_name
219 @heading = t 'note.mine.heading', :user => @user2.display_name
220 @description = t 'note.mine.description', :user => user_link
222 @page = (params[:page] || 1).to_i
225 @notes = Note.find(:all,
226 :include => [:comments, {:comments => :author}],
228 :order => "updated_at DESC",
229 :conditions => conditions,
230 :offset => (@page - 1) * @page_size,
231 :limit => @page_size).uniq
235 #------------------------------------------------------------
236 # utility functions below.
237 #------------------------------------------------------------
240 # merge two conditions
241 # TODO: this is a copy from changeset_controler.rb and should be factored out to share
246 return [ a_str + " AND " + b_str ] + a + b
256 output_js = :true if params['format'] == "js"
258 if output_js == :true
259 render :text => "osbResponse();", :content_type => "text/javascript"
261 render :text => "ok " + @note.id.to_s + "\n", :content_type => "text/html" if @note
262 render :text => "ok\n", :content_type => "text/html" unless @note
268 limit = params['limit'] if ((params['limit']) && (params['limit'].to_i < 10000) && (params['limit'].to_i > 0))
273 closed_since = 7 unless params['closed']
274 closed_since = params['closed'].to_i if params['closed']
277 conditions = ["status != 'hidden'"]
278 elsif closed_since > 0
279 conditions = ["((status = 'open') OR ((status = 'closed' ) AND (closed_at > '" + (Time.now - closed_since.days).to_s + "')))"]
281 conditions = ["status = 'open'"]
287 def add_comment(note, text, name, event)
288 comment = note.comments.create(:visible => true, :event => event)
289 comment.body = text unless text == :nil
291 comment.author_id = @user.id
292 comment.author_name = @user.display_name
294 comment.author_ip = request.remote_ip
295 comment.author_name = name + " (a)"
301 note.comments.each do | cmt |
303 unless sent_to.include?(cmt.author)
304 Notifier.deliver_note_comment_notification(note_comment, cmt.author) unless cmt.author == @user
305 sent_to.add(cmt.author)