Fix publishing events responding to #to_hash on Ruby 2.7

When a method accepts both splatted positional & keyword parameters and
a single argument responding to `#to_hash` is given, the argument is
coerced to hash and assigned to the keyword arguments.

E.g.:

class Hashable
  def to_hash
    { im: :a_hash }
  end
end

def method_with_splat_on_the_positional_argument(*args, **kwargs) # method_with_splat_on_the_positional_argument(Hashable.new)
  puts args.inspect # =>  []
  puts kwargs.inspect # => {:im=>:a_hash}
end

However,

def method_without_splat_on_the_positional_argument(args, **kwargs) # method_without_splat_on_the_positional_argument(Hashable.new)
  puts args.inspect # => #<Hashable:0x000055cfe8b072b0>
  puts kwargs.inspect # => {}
end

We fix the problem on the Spree::Bus override by being explicit about
the expected parameters.

References solidusio/solidus_stripe#157 (comment)

Author: waiting-for-dev

core/lib/spree/bus.rb

@@ -9,12 +9,12 @@ module Spree
   # It has some modifications to support internal usage of the legacy event
   # system {see Spree::AppConfiguration#use_legacy_events}.
   Bus = Omnes::Bus.new
-  Bus.define_singleton_method(:publish) do |*args, **kwargs, &block|
+  Bus.define_singleton_method(:publish) do |event, **kwargs|
     if Spree::Config.use_legacy_events
-      Spree::Event.fire(*args, **kwargs, &block)
+      Spree::Event.fire(event, **kwargs)
     else
       # Override caller_location to point to the actual event publisher
-      super(*args, **kwargs, caller_location: caller_locations(1)[0], &block)
+      super(event, **kwargs, caller_location: caller_locations(1)[0])
     end
   end
 end