More API changes

lib/action_cable/channel/base.rb

@@ -27,7 +27,9 @@ module ActionCable
         @_active_periodic_timers = []
         @params = params
 
-        setup
+        connect
+
+        subscribe
       end
 
       def receive(data)
@@ -47,7 +49,7 @@ module ActionCable
       end
 
       protected
-        def setup
+        def connect
           # Override in subclasses
         end
 
@@ -56,8 +58,10 @@ module ActionCable
         end
 
         def start_periodic_timers
-          self.class.periodic_timers.each do |method, options|
-            @_active_periodic_timers << EventMachine::PeriodicTimer.new(options[:every]) { send(method) }
+          self.class.periodic_timers.each do |callback, options|
+            @_active_periodic_timers << EventMachine::PeriodicTimer.new(options[:every]) do
+              callback.respond_to?(:call) ? callback.call : send(callback)
+            end
           end
         end
 

lib/action_cable/channel/callbacks.rb

@@ -21,8 +21,8 @@ module ActionCable
           self.on_unsubscribe_callbacks += methods
         end
 
-        def periodic_timer(method, every:)
-          self.periodic_timers += [ [ method, every: every ] ]
+        def periodically(callback, every:)
+          self.periodic_timers += [ [ callback, every: every ] ]
         end
       end
 

lib/action_cable/server.rb

@@ -54,7 +54,6 @@ module ActionCable
 
         if subscription_klass
           @subscriptions[id_key] = subscription_klass.new(self, id_key, id_options)
-          @subscriptions[id_key].subscribe
         else
           # No channel found
         end