]> git.openstreetmap.org Git - rails.git/blob - app/controllers/map_bugs_controller.rb
Detabify and tidy up some more of the bugs code
[rails.git] / app / controllers / map_bugs_controller.rb
1 class MapBugsController < ApplicationController
2
3   layout 'site', :only => [:my_bugs]
4
5   before_filter :check_api_readable
6   before_filter :authorize_web, :only => [:add_bug, :close_bug, :edit_bug, :delete, :my_bugs]
7   before_filter :check_api_writable, :only => [:add_bug, :close_bug, :edit_bug, :delete]
8   before_filter :require_moderator, :only => [:delete]
9   before_filter :set_locale, :only => [:my_bugs]
10   after_filter :compress_output
11   around_filter :api_call_handle_error, :api_call_timeout
12
13   # Help methods for checking boundary sanity and area size
14   include MapBoundary
15
16   def get_bugs
17
18     # Figure out the bbox
19     bbox = params['bbox']
20
21     if bbox and bbox.count(',') == 3
22       bbox = bbox.split(',')
23       @min_lon, @min_lat, @max_lon, @max_lat = sanitise_boundaries(bbox)
24     else
25       #Fallback to old style, this is deprecated and should not be used
26       raise OSM::APIBadUserInput.new("No l was given") unless params['l']
27       raise OSM::APIBadUserInput.new("No r was given") unless params['r']
28       raise OSM::APIBadUserInput.new("No b was given") unless params['b']
29       raise OSM::APIBadUserInput.new("No t was given") unless params['t']
30
31       @min_lon = params['l'].to_f
32       @max_lon = params['r'].to_f
33       @min_lat = params['b'].to_f
34       @max_lat = params['t'].to_f
35     end
36
37     limit = getLimit
38     conditions = closedCondition
39         
40     check_boundaries(@min_lon, @min_lat, @max_lon, @max_lat, :false)
41
42     @bugs = MapBug.find_by_area_no_quadtile(@min_lat, @min_lon, @max_lat, @max_lon, :include => :map_bug_comment, :order => "last_changed DESC", :limit => limit, :conditions => conditions)
43
44     respond_to do |format|
45       format.html {render :template => 'map_bugs/get_bugs.js', :content_type => "text/javascript"}
46       format.rss {render :template => 'map_bugs/get_bugs.rss'}
47       format.js
48       format.xml {render :template => 'map_bugs/get_bugs.xml'}
49       format.json { render :json => @bugs.to_json(:methods => [:lat, :lon], :only => [:id, :status, :date_created], :include => { :map_bug_comment => { :only => [:commenter_name, :date_created, :comment]}}) }          
50 #      format.gpx {render :template => 'map_bugs/get_bugs.gpx'}
51     end
52   end
53
54   def add_bug
55     raise OSM::APIBadUserInput.new("No lat was given") unless params['lat']
56     raise OSM::APIBadUserInput.new("No lon was given") unless params['lon']
57     raise OSM::APIBadUserInput.new("No text was given") unless params['text']
58
59     lon = params['lon'].to_f
60     lat = params['lat'].to_f
61     comment = params['text']
62
63     name = "NoName"
64     name = params['name'] if params['name']
65
66     #Include in a transaction to ensure that there is always a map_bug_comment for every map_bug
67     MapBug.transaction do
68       @bug = MapBug.create_bug(lat, lon)
69
70       #TODO: move this into a helper function
71       begin
72         url = "http://nominatim.openstreetmap.org/reverse?lat=" + lat.to_s + "&lon=" + lon.to_s + "&zoom=16" 
73         response = REXML::Document.new(Net::HTTP.get(URI.parse(url))) 
74                 
75         if result = response.get_text("reversegeocode/result") 
76           @bug.nearby_place = result.to_s 
77         else 
78           @bug.nearby_place = "unknown"
79         end
80       rescue Exception => err
81         @bug.nearby_place = "unknown"
82       end
83           
84       @bug.save
85       add_comment(@bug, comment, name,"opened")
86     end
87  
88     render_ok
89   end
90
91   def edit_bug
92     raise OSM::APIBadUserInput.new("No id was given") unless params['id']
93     raise OSM::APIBadUserInput.new("No text was given") unless params['text']
94
95     name = "NoName"
96     name = params['name'] if params['name']
97         
98     id = params['id'].to_i
99
100     bug = MapBug.find_by_id(id)
101     raise OSM::APINotFoundError unless bug
102     raise OSM::APIAlreadyDeletedError unless bug.visible
103
104     MapBug.transaction do
105       bug_comment = add_comment(bug, params['text'], name,"commented")
106     end
107
108     render_ok
109   end
110
111   def close_bug
112     raise OSM::APIBadUserInput.new("No id was given") unless params['id']
113         
114     id = params['id'].to_i
115     name = "NoName"
116     name = params['name'] if params['name']
117
118     bug = MapBug.find_by_id(id)
119     raise OSM::APINotFoundError unless bug
120     raise OSM::APIAlreadyDeletedError unless bug.visible
121
122     MapBug.transaction do
123       bug.close_bug
124       add_comment(bug,:nil,name,"closed")
125     end
126
127     render_ok
128   end 
129
130
131   def rss
132     # Figure out the bbox
133     bbox = params['bbox']
134
135     if bbox and bbox.count(',') == 3
136       bbox = bbox.split(',')
137       @min_lon, @min_lat, @max_lon, @max_lat = sanitise_boundaries(bbox)
138     else
139       @min_lon = -180.0
140       @min_lat = -90.0
141       @max_lon = 180.0
142       @max_lat = 90.0
143     end
144
145     limit = getLimit
146     conditions = closedCondition
147     conditions = cond_merge conditions, [OSM.sql_for_area_no_quadtile(@min_lat, @min_lon, @max_lat, @max_lon)]
148         
149     check_boundaries(@min_lon, @min_lat, @max_lon, @max_lat, :false)
150
151     @comments = MapBugComment.find(:all, :limit => limit, :order => "date_created DESC", :joins => :map_bug, :include => :map_bug, :conditions => conditions)
152     render :template => 'map_bugs/rss.rss'
153   end
154
155   def gpx_bugs
156     request.format = :xml
157     get_bugs
158   end
159   
160   def read
161     @bug = MapBug.find(params['id'])
162     raise OSM::APINotFoundError unless @bug
163     raise OSM::APIAlreadyDeletedError unless @bug.visible
164     
165     respond_to do |format|
166       format.rss
167       format.xml
168       format.json { render :json => @bug.to_json(:methods => [:lat, :lon], :only => [:id, :status, :date_created], :include => { :map_bug_comment => { :only => [:commenter_name, :date_created, :comment]}}) }   
169     end
170   end
171
172   def delete
173     bug = MapBug.find(params['id'])
174     raise OSM::APINotFoundError unless @bug
175     raise OSM::APIAlreadyDeletedError unless @bug.visible
176
177     MapBug.transaction do
178       bug.status = "hidden"
179       bug.save
180       add_comment(bug,:nil,name,"hidden")
181     end
182
183     render :text => "ok\n", :content_type => "text/html" 
184   end
185
186   def search
187     raise OSM::APIBadUserInput.new("No query string was given") unless params['q']
188     limit = getLimit
189     conditions = closedCondition
190     conditions = cond_merge conditions, ['map_bug_comment.comment ~ ?', params['q']]
191         
192     #TODO: There should be a better way to do this.   CloseConditions are ignored at the moment
193
194     bugs2 = MapBug.find(:all, :limit => limit, :order => "last_changed DESC", :joins => :map_bug_comment, :include => :map_bug_comment,
195                         :conditions => conditions)
196     @bugs = bugs2.uniq
197     respond_to do |format|
198       format.html {render :template => 'map_bugs/get_bugs.js', :content_type => "text/javascript"}
199       format.rss {render :template => 'map_bugs/get_bugs.rss'}
200       format.js
201       format.xml {render :template => 'map_bugs/get_bugs.xml'}
202       format.json { render :json => @bugs.to_json(:methods => [:lat, :lon], :only => [:id, :status, :date_created], :include => { :map_bug_comment => { :only => [:commenter_name, :date_created, :comment]}}) }
203 #      format.gpx {render :template => 'map_bugs/get_bugs.gpx'}
204     end
205   end
206
207   def my_bugs
208     if params[:display_name] 
209       @user2 = User.find_by_display_name(params[:display_name], :conditions => { :status => ["active", "confirmed"] }) 
210  
211       if @user2  
212         if @user2.data_public? or @user2 == @user 
213           conditions = ['map_bug_comment.commenter_id = ?', @user2.id] 
214         else 
215           conditions = ['false'] 
216         end 
217       else #if request.format == :html 
218         @title = t 'user.no_such_user.title' 
219         @not_found_user = params[:display_name] 
220         render :template => 'user/no_such_user', :status => :not_found 
221         return
222       end 
223     end
224
225     if @user2 
226       user_link = render_to_string :partial => "user", :object => @user2 
227     end 
228     
229     @title =  t 'bugs.user.title_user', :user => @user2.display_name 
230     @heading =  t 'bugs.user.heading_user', :user => @user2.display_name 
231     @description = t 'bugs.user.description_user', :user => user_link
232     
233     @page = (params[:page] || 1).to_i 
234     @page_size = 10
235
236     @bugs = MapBug.find(:all, 
237                         :include => [:map_bug_comment, {:map_bug_comment => :user}],
238                         :joins => :map_bug_comment,
239                         :order => "last_changed DESC",
240                         :conditions => conditions,
241                         :offset => (@page - 1) * @page_size, 
242                         :limit => @page_size).uniq
243   end
244
245 private 
246   #------------------------------------------------------------ 
247   # utility functions below. 
248   #------------------------------------------------------------   
249  
250   ## 
251   # merge two conditions 
252   # TODO: this is a copy from changeset_controler.rb and should be factored out to share
253   def cond_merge(a, b) 
254     if a and b 
255       a_str = a.shift 
256       b_str = b.shift 
257       return [ a_str + " AND " + b_str ] + a + b 
258     elsif a  
259       return a 
260     else b 
261       return b 
262     end 
263   end 
264
265   def render_ok
266     output_js = :false
267     output_js = :true if params['format'] == "js"
268
269     if output_js == :true
270       render :text => "osbResponse();", :content_type => "text/javascript" 
271     else
272       render :text => "ok " + @bug.id.to_s + "\n", :content_type => "text/html" if @bug
273       render :text => "ok\n", :content_type => "text/html" unless @bug
274     end
275   end
276
277   def getLimit
278     limit = 100
279     limit = params['limit'] if ((params['limit']) && (params['limit'].to_i < 10000) && (params['limit'].to_i > 0))
280     return limit
281   end
282
283   def closedCondition
284     closed_since = 7 unless params['closed']
285     closed_since = params['closed'].to_i if params['closed']
286         
287     if closed_since < 0
288       conditions = ["status != 'hidden'"]
289     elsif closed_since > 0
290       conditions = ["((status = 'open') OR ((status = 'closed' ) AND (date_closed > '" + (Time.now - closed_since.days).to_s + "')))"]
291     else
292       conditions = ["status = 'open'"]
293     end
294
295     return conditions
296   end
297
298   def add_comment(bug, comment, name,event) 
299     t = Time.now.getutc 
300     bug_comment = bug.map_bug_comment.create(:date_created => t, :visible => true, :event => event)
301     bug_comment.comment = comment unless comment == :nil
302     if @user  
303       bug_comment.commenter_id = @user.id
304       bug_comment.commenter_name = @user.display_name
305     else  
306       bug_comment.commenter_ip = request.remote_ip
307       bug_comment.commenter_name = name + " (a)"
308     end
309     bug_comment.save 
310     bug.last_changed = t 
311     bug.save
312
313     sent_to = Set.new
314     bug.map_bug_comment.each do | cmt |
315       if cmt.user
316         unless sent_to.include?(cmt.user)
317           Notifier.deliver_bug_comment_notification(bug_comment, cmt.user) unless cmt.user == @user
318           sent_to.add(cmt.user)
319         end
320       end
321     end
322   end
323 end