prematurely fix bug as fixed by DMPRoadmap/roadmap#3325

DMPbelgium · Sep 21, 2023 · 795a2de · 795a2de
1 parent d65ab27
commit 795a2de
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/app/controllers/api/v0/plans_controller.rb b/app/controllers/api/v0/plans_controller.rb
@@ -101,7 +101,7 @@ def index
         max_per_page = Rails.configuration.x.application.api_max_page_size
         page = params.fetch('page', 1).to_i
         per_page = params.fetch('per_page', max_per_page).to_i
-        per_page = max_per_page if @per_page > max_per_page
+        per_page = max_per_page if per_page > max_per_page
         @args = { per_page: per_page, page: page }
         @plans = refine_query(@plans)
         respond_with @plans

diff --git a/ugent/CHANGES.txt b/ugent/CHANGES.txt
@@ -444,6 +444,10 @@
 
   used in share tab of a plan in ugent branch
 
+- app/controllers/api/v0/plans_controller.rb
+
+  prematurely fix bug as fixed by https://github.com/DMPRoadmap/roadmap/pull/3325
+
 - app/models/ugent/rest_user.rb
 
   ugent specific model