Skip to content

Commit dfa2917

Browse files
committed
Merge pull request websocket-rails#100 from imton/master
on_close and on_error events fixed
2 parents 97ff6bb + 9e37dd5 commit dfa2917

File tree

6 files changed

+4375
-3563
lines changed

6 files changed

+4375
-3563
lines changed

Gemfile.lock

+1
Original file line numberDiff line numberDiff line change
@@ -82,6 +82,7 @@ GEM
8282
headless (0.3.1)
8383
hike (1.2.1)
8484
hiredis (0.4.5)
85+
hiredis (0.4.5-java)
8586
i18n (0.6.1)
8687
jasmine (1.2.1)
8788
jasmine-core (>= 1.2.0)

lib/assets/javascripts/websocket_rails/websocket_connection.js.coffee

+2-2
Original file line numberDiff line numberDiff line change
@@ -22,11 +22,11 @@ class WebSocketRails.WebSocketConnection
2222
@dispatcher.new_message data
2323

2424
on_close: (event) =>
25-
close_event = new WebSocketRails.Event(['connection_closed',{}])
25+
close_event = new WebSocketRails.Event(['connection_closed', event])
2626
@dispatcher.dispatch close_event
2727

2828
on_error: (event) =>
29-
error_event = new WebSocketRails.Event(['connection_error',event?.data])
29+
error_event = new WebSocketRails.Event(['connection_error', event])
3030
@dispatcher.dispatch error_event
3131

3232
flush_queue: =>

spec/javascripts/support/jasmine.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
#
1111
src_dir: spec/javascripts
1212
src_files:
13-
- support/vendor/sinon-1.3.4.js
13+
- support/vendor/sinon-1.7.1.js
1414
- generated/assets/websocket_rails.js
1515
- generated/assets/*.js
1616

0 commit comments

Comments
 (0)