Make require_package before action private

src/api/app/controllers/source_controller.rb

@@ -102,23 +102,6 @@ def delete_package
     render_ok
   end
 
-  # before_action for show_package, delete_package and package_command
-  def require_package
-    # init and validation
-    #--------------------
-    @deleted_package = params.key?(:deleted)
-
-    # FIXME: for OBS 3, api of branch and copy calls have target and source in the opossite place
-    if params[:cmd].in?(['branch', 'release'])
-      @target_package_name = params[:package]
-      @target_project_name = params[:target_project] # might be nil
-      @target_package_name = params[:target_package] if params[:target_package]
-    else
-      @target_project_name = params[:project]
-      @target_package_name = params[:package]
-    end
-  end
-
   def verify_can_modify_target_package!
     return if User.session!.can_modify?(@package)
 
@@ -430,6 +413,23 @@ def set_issues_defaults
 
   private
 
+  # before_action for show_package, delete_package and package_command
+  def require_package
+    # init and validation
+    #--------------------
+    @deleted_package = params.key?(:deleted)
+
+    # FIXME: for OBS 3, api of branch and copy calls have target and source in the opossite place
+    if params[:cmd].in?(['branch', 'release'])
+      @target_package_name = params[:package]
+      @target_project_name = params[:target_project] # might be nil
+      @target_package_name = params[:target_package] if params[:target_package]
+    else
+      @target_project_name = params[:project]
+      @target_package_name = params[:package]
+    end
+  end
+
   # GET /source/:project/:package?view=issues
   # called from show_package
   def show_package_issues