课程通知访问增加权限控制

app/controllers/news_controller.rb

@@ -65,23 +65,27 @@ class NewsController < ApplicationController
         format.atom { render_feed(@newss, :title => (@project ? @project.name : Setting.app_title) + ": #{l(:label_news_plural)}") }
       end
     elsif @course
-      scope = @course ? @course.news.course_visible : News.course_visible
+      if (User.current.admin? || @course.is_public == 1 || (@course.is_public == 0 && User.current.member_of_course?(@course)))
-
+        scope = @course ? @course.news.course_visible : News.course_visible
-      @news_count = scope.count
+
-      @news_pages = Paginator.new @news_count, @limit, params['page']
+        @news_count = scope.count
-      @offset ||= @news_pages.offset
+        @news_pages = Paginator.new @news_count, @limit, params['page']
-      @newss = scope.all(:include => [:author, :course],
+        @offset ||= @news_pages.offset
-                         :order => "#{News.table_name}.created_on DESC",
+        @newss = scope.all(:include => [:author, :course],
-                         :offset => @offset,
+                           :order => "#{News.table_name}.created_on DESC",
-                         :limit => @limit)
+                           :offset => @offset,
-
+                           :limit => @limit)
-      respond_to do |format|
+
-        format.html {
+        respond_to do |format|
-          @news = News.new
+          format.html {
-          render :layout => 'base_courses'
+            @news = News.new
-        }
+            render :layout => 'base_courses'
-        format.api
+          }
-        format.atom { render_feed(@newss, :title => (@course ? @course.name : Setting.app_title) + ": #{l(:label_news_plural)}") }
+          format.api
+          format.atom { render_feed(@newss, :title => (@course ? @course.name : Setting.app_title) + ": #{l(:label_news_plural)}") }
+        end
+      else
+        render_403
       end
     end
    end