Defect #34933

Atom feed of the activity page does not contain items after the second page

Added by Yuichi HARADA 24 days ago. Updated 3 days ago.

Status:ConfirmedStart date:
Priority:NormalDue date:
Assignee:-% Done:

0%

Category:Activity view
Target version:4.1.3
Resolution: Affected version:

Description

I'm receiving Atom feeds on the Activity.
I thought I was receiving the number of feeds set in Administration > Settings > General > Maximum number of items in Atom feeds, but actually I received only the events displayed on the first page of the Activity.

34933.patch Magnifier (1.49 KB) Yuichi HARADA, 2021-03-19 06:48

34933-v2.patch Magnifier (1.48 KB) Go MAEDA, 2021-04-09 07:22

History

#1 Updated by Yuichi HARADA 24 days ago

I created a patch.

diff --git a/app/controllers/activities_controller.rb b/app/controllers/activities_controller.rb
index 745a40219..e1bba49f6 100644
--- a/app/controllers/activities_controller.rb
+++ b/app/controllers/activities_controller.rb
@@ -55,7 +55,12 @@ class ActivitiesController < ApplicationController
       end
     end

-    events = @activity.events(@date_from, @date_to)
+    events =
+      if params[:format] == 'atom'
+        @activity.events(nil, nil, :limit => Setting.feeds_limit.to_i)
+      else
+        @activity.events(@date_from, @date_to)
+      end

     if events.empty? || stale?(:etag => [@activity.scope, @date_to, @date_from, @with_subprojects, @author, events.first, events.size, User.current, current_language])
       respond_to do |format|

#2 Updated by Go MAEDA 15 days ago

  • Status changed from New to Confirmed
  • Target version set to Candidate for next minor release

#3 Updated by Go MAEDA 3 days ago

  • File 34933-v2.patchMagnifier added
  • Subject changed from Received Atom feed entries only of the first page of Activity to Atom feed of the activity page does not contain items after the second page
  • Target version changed from Candidate for next minor release to 4.1.3

In 34933-v2.patch, I have changed the test name.

Also available in: Atom PDF