split (#2381)

app/controllers/concerns/admin/reorderable.rb

@@ -8,16 +8,19 @@ module Admin::Reorderable
       object = model.find_by(id: id)
       object.update_column(:position, index + 1) unless object.nil?
     end
-    if first_object&.respond_to?(:is_direct_object?)
-      first_object.is_direct_object?  ? first_object.sync_with_git
-                                      : first_object.touch # Sync indirect object's direct sources through after_touch
-    end
+    sync_after_reorder(first_object)
     # Used to add extra code
     yield first_object if block_given?
   end
 
   protected
 
+  def sync_after_reorder(first_object)
+    return unless first_object&.respond_to?(:is_direct_object?)
+    first_object.is_direct_object?  ? first_object.sync_with_git
+                                    : first_object.touch # Sync indirect object's direct sources through after_touch
+  end
+
   def model
     self.class.to_s.remove('Admin::').remove('Controller').singularize.safe_constantize
   end