diff --git a/app/controllers/attachments_controller.rb b/app/controllers/attachments_controller.rb
index 4964bf6a6..bef128e50 100644
--- a/app/controllers/attachments_controller.rb
+++ b/app/controllers/attachments_controller.rb
@@ -55,10 +55,10 @@ class AttachmentsController < ApplicationController
     candown = false
     if @attachment.container.has_attribute?(:project) && @attachment.container.project
       project = @attachment.container.project
-      candown= User.current.member_of?(project) || (project.is_public==1 && @attachment.is_public == 1)
+      candown= User.current.member_of?(project) || (project.is_public && @attachment.is_public == 1)
     elsif  @attachment.container.is_a?(Project)
       project = @attachment.container
-      candown= User.current.member_of?(project) || (project.is_public==1 && @attachment.is_public == 1)
+      candown= User.current.member_of?(project) || (project.is_public && @attachment.is_public == 1)
     elsif @attachment.container.has_attribute?(:course) && @attachment.container.course
       course = @attachment.container.course
       candown= User.current.member_of_course?(course) || (course.is_public==1 && @attachment.is_public == 1)