Skip to content

Commit 84ed0d6

Browse files
committedMar 28, 2012
Removes the added logging and switches some js to camelcase
Signed-off-by: Akash Manohar J <akash@akash.im>
1 parent ef209f7 commit 84ed0d6

File tree

10 files changed

+78
-53
lines changed

10 files changed

+78
-53
lines changed
 

‎Gemfile

+2
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,8 @@ group :assets do
3131
gem 'uglifier', '>= 1.0.3'
3232
end
3333

34+
gem "tane", :group => :development, :path => "../tane"
35+
3436
# Test gems, obviously
3537
group :test do
3638
gem 'rspec-rails'

‎Gemfile.lock

+15
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,14 @@
1+
PATH
2+
remote: ../tane
3+
specs:
4+
tane (0.0.3)
5+
awesome_print
6+
erubis
7+
highline
8+
launchy
9+
rake
10+
rest-client
11+
112
GEM
213
remote: https://rubygems.org/
314
specs:
@@ -30,6 +41,7 @@ GEM
3041
multi_json (~> 1.0)
3142
addressable (2.2.7)
3243
arel (3.0.2)
44+
awesome_print (1.0.2)
3345
aws-sdk (1.3.8)
3446
httparty (~> 0.7)
3547
json (~> 1.4)
@@ -105,6 +117,8 @@ GEM
105117
actionpack (>= 3.0.0)
106118
activesupport (>= 3.0.0)
107119
railties (>= 3.0.0)
120+
launchy (2.1.0)
121+
addressable (~> 2.2.6)
108122
mail (2.4.3)
109123
i18n (>= 0.4.0)
110124
mime-types (~> 1.16)
@@ -215,5 +229,6 @@ DEPENDENCIES
215229
sass-rails (~> 3.2.3)
216230
shoulda-matchers
217231
sqlite3
232+
tane!
218233
thin
219234
uglifier (>= 1.0.3)

‎app/assets/javascripts/backbone/broadcasters/faye.js.coffee

+11-2
Original file line numberDiff line numberDiff line change
@@ -15,14 +15,23 @@ class Kandan.Broadcasters.FayeBroadcaster
1515
console.log "incoming", message
1616
callback(message)
1717
}
18+
@faye_client.addExtension(auth_extension)
19+
1820
@faye_client.bind "transport:down", ()->
1921
console.log "Comm link to Cybertron is down!"
2022

2123
@faye_client.bind "transport:up", ()->
2224
console.log "Comm link is up!"
2325

24-
@faye_client.addExtension(auth_extension)
25-
@faye_client.subscribe "/app/activities", (data)=>
26+
27+
@faye_client.subscribe "/app/user_activities", (data)=>
28+
$(document).data('active_users', data.data.active_users)
29+
Kandan.Helpers.Channels.add_activity({
30+
user: data.data.user,
31+
action: data.event.split("#")[1]
32+
})
33+
34+
@faye_client.subscribe "/app/channel_activities", (data)=>
2635
$(document).data('active_users', data.data.active_users)
2736
Kandan.Helpers.Channels.add_activity({
2837
user: data.data.user,

‎app/assets/javascripts/backbone/data/active_users.js.coffee

+2-2
Original file line numberDiff line numberDiff line change
@@ -7,10 +7,10 @@ class Kandan.Data.ActiveUsers
77
@register_callback: (event, callback)->
88
@callbacks[event].push(callback)
99

10-
@run_callbacks: (event)->
10+
@runCallbacks: (event)->
1111
callback() for callback in @callbacks[event]
1212

13-
@unregister_callback: (event, callback)->
13+
@unregisterCallback: (event, callback)->
1414
delete @callbacks[@callbacks.indexOf(callback)]
1515
@callbacks.filter (element, index, array)->
1616
element!=undefined

‎app/assets/javascripts/backbone/data/channels.js.coffee

+4-4
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,15 @@
11
class Kandan.Data.Channels
22
@callbacks: {"change": [] }
33

4-
@active_channel_id: ()->
5-
Kandan.Helpers.Channels.get_active_channel_id()
4+
@activeChannelId: ()->
5+
Kandan.Helpers.Channels.getActiveChannelId()
66

77
@all: (callbacks)->
88
attachments = new Kandan.Collections.Attachments([], {
9-
channel_id: @active_channel_id()
9+
channel_id: @activeChannelId()
1010
})
1111

12-
@run_callbacks: (event)->
12+
@runCallbacks: (event)->
1313
callback() for callback in @callbacks[event]
1414

1515
@register_callback: (event, callback)->

‎app/assets/javascripts/backbone/helpers/channels.js.coffee

+38-38
Original file line numberDiff line numberDiff line change
@@ -9,33 +9,33 @@ class Kandan.Helpers.Channels
99
$tabNav.find(".create_channel").parent().remove()
1010
$tabNav.append("<li>"+$createButton+"</li>")
1111

12-
@pastAutoScrollThreshold: (channel_id)->
13-
currentPosition = @currentScrollPosition channel_id
12+
@pastAutoScrollThreshold: (channelId)->
13+
currentPosition = @currentScrollPosition channelId
1414
totalHeight = $(document).height() - $(window).height()
1515
scrollPercentage = (currentPosition) / (totalHeight)
1616
scrollPercentage > @options.autoScrollThreshold
1717

18-
@scrollToLatestMessage: (channel_id)->
19-
$("#channels-#{channel_id}").scrollTop(100000)
18+
@scrollToLatestMessage: (channelId)->
19+
$("#channels-#{channelId}").scrollTop(100000)
2020

21-
@currentScrollPosition: (channel_id)->
22-
$("#channels-#{channel_id}").scrollTop()
21+
@currentScrollPosition: (channelId)->
22+
$("#channels-#{channelId}").scrollTop()
2323

24-
@channel_activities_el: (channel_id)->
25-
$("#channel-activities-#{channel_id}")
24+
@channel_activities_el: (channelId)->
25+
$("#channel-activities-#{channelId}")
2626

27-
@channel_pagination_el: (channel_id)->
28-
$("#channels-#{channel_id} .pagination")
27+
@channel_pagination_el: (channelId)->
28+
$("#channels-#{channelId} .pagination")
2929

30-
@get_channel_id_from_tab_index: (tab_index)->
30+
@getChannelIdFromTabIndex: (tabIndex)->
3131
$("#channels .ui-tabs-panel")
32-
.eq(tab_index)
32+
.eq(tabIndex)
3333
.data('channel_id')
3434

3535
@selected_tab: ()->
3636
$('#channels').tabs('option', 'selected')
3737

38-
@get_active_channel_id: ()->
38+
@getActiveChannelId: ()->
3939
if $(document).data('active_channel_id') == undefined
4040
return $("#channels .ui-tabs-panel")
4141
.eq(@selected_tab())
@@ -51,16 +51,16 @@ class Kandan.Helpers.Channels
5151

5252

5353
@deleteChannel: (channelIndex)->
54-
channelID = @get_channel_id_from_tab_index(channelIndex)
55-
channel = new Kandan.Models.Channel({id: channelID})
54+
channelId = @getChannelIdFromTabIndex(channelIndex)
55+
channel = new Kandan.Models.Channel({id: channelId})
5656
return false if @confirmDeletion() == false
5757

5858
channel.destroy({success: ()=>
5959
$("#channels").tabs("remove", channelIndex)
6060
})
6161

62-
@channel_not_exists: (channel_id)->
63-
$("#channels-#{channel_id}").length == 0
62+
@channel_not_exists: (channelId)->
63+
$("#channels-#{channelId}").length == 0
6464

6565

6666
@create_channel_area: (channel)->
@@ -74,10 +74,10 @@ class Kandan.Helpers.Channels
7474
$(channel_area).data('channel_id', channel.get('id'))
7575

7676

77-
@new_activity_view: (activity_attributes)->
78-
activity = new Kandan.Models.Activity(activity_attributes)
79-
activity_view = new Kandan.Views.ShowActivity({activity: activity})
80-
return activity_view
77+
@new_activity_view: (activityAttributes)->
78+
activity = new Kandan.Models.Activity(activityAttributes)
79+
activityView = new Kandan.Views.ShowActivity({activity: activity})
80+
return activityView
8181

8282

8383
@add_activity: (activity_attributes, state)->
@@ -90,30 +90,30 @@ class Kandan.Helpers.Channels
9090
@add_notification(activity_attributes)
9191

9292
if activity_attributes.channel_id
93-
channel_id = activity_attributes.channel_id
93+
channelId = activity_attributes.channel_id
9494
else
95-
channel_id = @get_active_channel_id()
96-
@scrollToLatestMessage(channel_id) if @pastAutoScrollThreshold(channel_id)
95+
channelId = @getActiveChannelId()
96+
@scrollToLatestMessage(channelId) if @pastAutoScrollThreshold(channelId)
9797

98-
@add_message: (activity_attributes, state)->
99-
@channel_activities_el(activity_attributes.channel_id)
100-
.append(@new_activity_view(activity_attributes).render().el)
101-
@set_pagination_data(activity_attributes.channel_id)
98+
@add_message: (activityAttributes, state)->
99+
@channel_activities_el(activityAttributes.channel_id)
100+
.append(@new_activity_view(activityAttributes).render().el)
101+
@set_pagination_data(activityAttributes.channel_id)
102102

103103

104-
@add_notification: (activity_attributes)->
105-
$channel_elements = $(".channel-activities")
106-
activity_attributes["created_at"] = new Date()
107-
for el in $channel_elements
108-
$(el).append(@new_activity_view(activity_attributes).render().el)
104+
@add_notification: (activityAttributes)->
105+
$channelElements = $(".channel-activities")
106+
activityAttributes["created_at"] = new Date()
107+
for el in $channelElements
108+
$(el).append(@new_activity_view(activityAttributes).render().el)
109109

110110

111-
@set_pagination_state: (channel_id, more_activities, oldest)->
112-
@channel_pagination_el(channel_id).data('oldest', oldest)
113-
if more_activities == true
114-
@channel_pagination_el(channel_id).show()
111+
@set_pagination_state: (channelId, moreActivities, oldest)->
112+
@channel_pagination_el(channelId).data('oldest', oldest)
113+
if moreActivities == true
114+
@channel_pagination_el(channelId).show()
115115
else
116-
@channel_pagination_el(channel_id).hide()
116+
@channel_pagination_el(channelId).hide()
117117

118118

119119
@set_pagination_data: (channel_id)->

‎app/assets/javascripts/backbone/kandan.js.coffee

+3-3
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ window.Kandan =
4949

5050
$(document).bind 'changeData', (element, name, value)->
5151
if(name=="active_users")
52-
Kandan.Data.ActiveUsers.run_callbacks('change')
52+
Kandan.Data.ActiveUsers.runCallbacks('change')
5353

5454
active_users = new Kandan.Collections.ActiveUsers()
5555
active_users.fetch({
@@ -73,9 +73,9 @@ window.Kandan =
7373
$('#channels').tabs({
7474
select: (event, ui)->
7575
$(document).data('active_channel_id',
76-
Kandan.Helpers.Channels.get_channel_id_from_tab_index(ui.index))
76+
Kandan.Helpers.Channels.getChannelIdFromTabIndex(ui.index))
7777
console.log "channel changed to index", ui.index
78-
Kandan.Data.Channels.run_callbacks('change')
78+
Kandan.Data.Channels.runCallbacks('change')
7979
})
8080

8181
$("#channels").tabs 'option', 'tabTemplate', '''

‎app/assets/javascripts/backbone/plugins/attachments.js.coffee

+1-1
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ class Kandan.Plugins.Attachments
1313
''')
1414

1515
@channel_id: ()->
16-
Kandan.Data.Channels.active_channel_id()
16+
Kandan.Data.Channels.activeChannelId()
1717

1818
@csrf_param: ->
1919
$('meta[name=csrf-param]').attr('content')

‎app/assets/javascripts/backbone/views/chatbox.js.coffee

+1-1
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ class Kandan.Views.Chatbox extends Backbone.View
1919
activity = new Kandan.Models.Activity({
2020
'content': $chatbox.val(),
2121
'action': 'message',
22-
'channel_id': Kandan.Helpers.Channels.get_active_channel_id()
22+
'channel_id': Kandan.Helpers.Channels.getActiveChannelId()
2323
})
2424

2525
activity.save({},{success: ()->

‎lib/faye_extensions/devise_auth.rb

+1-2
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
class DeviseAuth
22
def incoming(message, callback)
33
if message['channel'] == "/meta/subscribe"
4-
puts "MESSAGE: #{message.inspect}"
54
auth_token = message['ext']['auth_token']
65
user = User.find_by_authentication_token(auth_token)
76
if user
@@ -11,7 +10,7 @@ def incoming(message, callback)
1110
message['error'] = "Invalid auth token"
1211
end
1312
end
14-
puts "Message: #{message.inspect}"
13+
# puts "Message: #{message.inspect}"
1514
callback.call(message)
1615
end
1716

0 commit comments

Comments
 (0)
Please sign in to comment.