[myexperiment-hackers] [1952] branches/event_logging/app/controllers/networks_controller.rb: Subscriptions.

[noreply]

Revision

1952

Author

alekses6

Date

2008-11-12 15:44:16 -0500 (Wed, 12 Nov 2008)

Log Message

Subscriptions. Performance improvement for shared items RSS of a group.

Modified Paths

• branches/event_logging/app/controllers/networks_controller.rb

Diff

Modified: branches/event_logging/app/controllers/networks_controller.rb (1951 => 1952)

--- branches/event_logging/app/controllers/networks_controller.rb	2008-11-12 20:34:13 UTC (rev 1951)
+++ branches/event_logging/app/controllers/networks_controller.rb	2008-11-12 20:44:16 UTC (rev 1952)
@@ -8,7 +8,7 @@
   
   before_filter :find_networks, : [:all]
   before_filter :find_network, : [:membership_request, :show, :comment, :comment_delete, :tag, :news, :shared_items]
-  before_filter :find_network_shared_items, : [:shared_items]
+  before_filter :find_network_shared_items_rss, : [:shared_items]
   before_filter :find_network_auth, : [:invite, :membership_invite, :membership_invite_external, :edit, :update, :destroy]
   
   # declare sweepers and which actions should invoke them
@@ -370,15 +370,17 @@
     end
   end
   
-  def find_network_shared_items
+  def find_network_shared_items_rss
     # only called for preparing item collection shown in RSS feed;
     # on 'show' page "contributions/table" partial takes care of authorization of each item
-    @shared_items_rss = []
-    shared_items = @network.shared_contributables
-    
-    shared_items.each do |item|
-      # should be authorized for anonymous users
-      @shared_items_rss << item if item.authorized?("show", nil)
+    if params[:format] and params[:format].downcase == 'rss'
+      @shared_items_rss = []
+      shared_items = @network.shared_contributables
+      
+      shared_items.each do |item|
+        # should be authorized for anonymous users
+        @shared_items_rss << item if item.authorized?("show", nil)
+      end
     end
   end