##// END OF EJS Templates
jro_apps » redmine
RSS
Fixed: SQL error when a non-admin user displays the project list...
Jean-Philippe Lang -
r552:97f6315bd0c3
parent child
Show More
Show file before | Show file after | Hide comments
@@ -77,7 +77,7 class Project < ActiveRecord::Base
77 def self.visible_by(user=nil)
77 def self.visible_by(user=nil)
78 if user && user.admin?
78 if user && user.admin?
79 return ["#{Project.table_name}.status=#{Project::STATUS_ACTIVE}"]
79 return ["#{Project.table_name}.status=#{Project::STATUS_ACTIVE}"]
80 elsif user && !user.memberships.empty?
80 elsif user && !user.memberships.any?
81 return ["#{Project.table_name}.status=#{Project::STATUS_ACTIVE} AND (#{Project.table_name}.is_public = ? or #{Project.table_name}.id IN (#{user.memberships.collect{|m| m.project_id}.join(',')}))", true]
81 return ["#{Project.table_name}.status=#{Project::STATUS_ACTIVE} AND (#{Project.table_name}.is_public = ? or #{Project.table_name}.id IN (#{user.memberships.collect{|m| m.project_id}.join(',')}))", true]
82 else
82 else
83 return ["#{Project.table_name}.status=#{Project::STATUS_ACTIVE} AND #{Project.table_name}.is_public = ?", true]
83 return ["#{Project.table_name}.status=#{Project::STATUS_ACTIVE} AND #{Project.table_name}.is_public = ?", true]
General Comments 0
You need to be logged in to leave comments. Login now