@@ -194,6 +194,7 @@ def logout_user
194
194
def check_if_login_required
195
195
# no check needed if user is already logged in
196
196
return true if User . current . logged?
197
+
197
198
require_login if Setting . login_required?
198
199
end
199
200
@@ -285,6 +286,7 @@ def require_login
285
286
286
287
def require_admin
287
288
return unless require_login
289
+
288
290
if !User . current . admin?
289
291
render_403
290
292
return false
@@ -375,6 +377,7 @@ def find_issue
375
377
# if the issue actually exists but requires authentication
376
378
@issue = Issue . find ( params [ :id ] )
377
379
raise Unauthorized unless @issue . visible?
380
+
378
381
@project = @issue . project
379
382
rescue ActiveRecord ::RecordNotFound
380
383
render_404
@@ -391,6 +394,7 @@ def find_issues
391
394
to_a
392
395
raise ActiveRecord ::RecordNotFound if @issues . empty?
393
396
raise Unauthorized unless @issues . all? ( &:visible? )
397
+
394
398
@projects = @issues . collect ( &:project ) . compact . uniq
395
399
@project = @projects . first if @projects . size == 1
396
400
rescue ActiveRecord ::RecordNotFound
@@ -491,6 +495,7 @@ def validate_back_url(back_url)
491
495
if uri . send ( component ) . present? && uri . send ( component ) != request . send ( component )
492
496
return false
493
497
end
498
+
494
499
uri . send ( :"#{ component } =" , nil )
495
500
end
496
501
# Always ignore basic user:password in the URL
@@ -575,6 +580,7 @@ def missing_template(exception)
575
580
# but have no HTML representation for non admin users
576
581
def require_admin_or_api_request
577
582
return true if api_request?
583
+
578
584
if User . current . admin?
579
585
true
580
586
elsif User . current . logged?
0 commit comments