Use Api.js to handle api calls to gitlab

Dmitriy Zaporozhets
1 parent a163135c
Showing 3 changed files with 70 additions and 42 deletions Show diff stats
app/assets/javascripts/api.js.coffee
app/assets/javascripts/users_select.js.coffee
app/assets/javascripts/wall.js.coffee
@@ -0,0 +1,53 @@
+@Api =
+  users_path: "/api/:version/users.json"
+  user_path: "/api/:version/users/:id.json"
+  notes_path: "/api/:version/projects/:id/notes.json"
+
+  # Get 20 (depends on api) recent notes 
+  # and sort the ascending from oldest to newest
+  notes: (project_id, callback) ->
+    url = Api.buildUrl(Api.notes_path)
+    url = url.replace(':id', project_id)
+    
+    $.ajax(
+      url: url,
+      data:
+        private_token: gon.api_token
+        gfm: true
+        recent: true
+      dataType: "json"
+    ).done (notes) ->
+      notes.sort (a, b) ->
+        return a.id - b.id
+      callback(notes)
+
+  user: (user_id, callback) ->
+    url = Api.buildUrl(Api.user_path)
+    url = url.replace(':id', user_id)
+
+    $.ajax(
+      url: url
+      data:
+        private_token: gon.api_token
+      dataType: "json"
+    ).done (user) ->
+      callback(user)
+
+  # Return users list. Filtered by query
+  # Only active users retrieved
+  users: (query, callback) ->
+    url = Api.buildUrl(Api.users_path)
+    
+    $.ajax(
+      url: url
+      data:
+        private_token: gon.api_token
+        search: query
+        per_page: 20
+        active: true
+      dataType: "json"
+    ).done (users) ->
+      callback(users)
+
+  buildUrl: (url) ->
+    return url.replace(':version', gon.api_version)
@@ -18,34 +18,19 @@ $ -&gt;
     placeholder: "Search for a user"
     multiple: $('.ajax-users-select').hasClass('multiselect')
     minimumInputLength: 0
-    ajax: # instead of writing the function to execute the request we use Select2's convenient helper
-      url: "/api/" + gon.api_version + "/users.json"
-      dataType: "json"
-      data: (term, page) ->
-        search: term # search term
-        per_page: 10
-        active: true
-        private_token: gon.api_token
-
-      results: (data, page) -> # parse the results into the format expected by Select2.
-        # since we are using custom formatting functions we do not need to alter remote JSON data
-        results: data
+    query: (query) ->
+      Api.users query.term, (users) ->
+        data = { results: users }
+        query.callback(data)
     initSelection: (element, callback) ->
       id = $(element).val()
       if id isnt ""
-        $.ajax(
-          "/api/" + gon.api_version + "/users/" + id + ".json",
-          dataType: "json"
-          data:
-            private_token: gon.api_token
-        ).done (data) ->
-          callback data
+        Api.user(id, callback)
-    formatResult: userFormatResult # omitted for brevity, see the source of this page
-    formatSelection: userFormatSelection # omitted for brevity, see the source of this page
-    dropdownCssClass: "ajax-users-dropdown" # apply css that makes the dropdown taller
+    formatResult: userFormatResult
+    formatSelection: userFormatSelection
+    dropdownCssClass: "ajax-users-dropdown"
     escapeMarkup: (m) -> # we do not want to escape markup since we are displaying html in results
       m
-
 @Wall =
   note_ids: []
-  notes_path: null
-  notes_params: null
   project_id: null
   init: (project_id) ->
     Wall.project_id = project_id
-    Wall.notes_path = "/api/" + gon.api_version + "/projects/" + project_id + "/notes.json"
     Wall.getContent()
     Wall.initRefresh()
     Wall.initForm()
@@ -15,22 +12,15 @@
   # Gets an initial set of notes.
   # 
   getContent: ->
-    $.ajax
-      url: Wall.notes_path,
-      data:
-        private_token: gon.api_token
-        gfm: true
-        recent: true
-      dataType: "json"
-      success: (notes) ->
-        notes.sort (a, b) ->
-          return a.id - b.id
-        $.each notes, (i, note)->
-          if $.inArray(note.id, Wall.note_ids) == -1
-            Wall.note_ids.push(note.id)
-            Wall.renderNote(note)
-            Wall.scrollDown()
-            $("abbr.timeago").timeago()
+    Api.notes Wall.project_id, (notes) ->
+      $.each notes, (i, note) ->
+        # render note if it not present in loaded list
+        # or skip if rendered
+        if $.inArray(note.id, Wall.note_ids) == -1
+          Wall.note_ids.push(note.id)
+          Wall.renderNote(note)
+          Wall.scrollDown()
+          $("abbr.timeago").timeago()
   initRefresh: ->
     setInterval("Wall.refresh()", 10000)
@@ -0,0 +1,53 @@		@@ -0,0 +1,53 @@
	1	+@Api =
	2	+ users_path: "/api/:version/users.json"
	3	+ user_path: "/api/:version/users/:id.json"
	4	+ notes_path: "/api/:version/projects/:id/notes.json"
	5	+
	6	+ # Get 20 (depends on api) recent notes
	7	+ # and sort the ascending from oldest to newest
	8	+ notes: (project_id, callback) ->
	9	+ url = Api.buildUrl(Api.notes_path)
	10	+ url = url.replace(':id', project_id)
	11	+
	12	+ $.ajax(
	13	+ url: url,
	14	+ data:
	15	+ private_token: gon.api_token
	16	+ gfm: true
	17	+ recent: true
	18	+ dataType: "json"
	19	+ ).done (notes) ->
	20	+ notes.sort (a, b) ->
	21	+ return a.id - b.id
	22	+ callback(notes)
	23	+
	24	+ user: (user_id, callback) ->
	25	+ url = Api.buildUrl(Api.user_path)
	26	+ url = url.replace(':id', user_id)
	27	+
	28	+ $.ajax(
	29	+ url: url
	30	+ data:
	31	+ private_token: gon.api_token
	32	+ dataType: "json"
	33	+ ).done (user) ->
	34	+ callback(user)
	35	+
	36	+ # Return users list. Filtered by query
	37	+ # Only active users retrieved
	38	+ users: (query, callback) ->
	39	+ url = Api.buildUrl(Api.users_path)
	40	+
	41	+ $.ajax(
	42	+ url: url
	43	+ data:
	44	+ private_token: gon.api_token
	45	+ search: query
	46	+ per_page: 20
	47	+ active: true
	48	+ dataType: "json"
	49	+ ).done (users) ->
	50	+ callback(users)
	51	+
	52	+ buildUrl: (url) ->
	53	+ return url.replace(':version', gon.api_version)
	@@ -18,34 +18,19 @@ $ ->		@@ -18,34 +18,19 @@ $ ->
18	placeholder: "Search for a user"	18	placeholder: "Search for a user"
19	multiple: $('.ajax-users-select').hasClass('multiselect')	19	multiple: $('.ajax-users-select').hasClass('multiselect')
20	minimumInputLength: 0	20	minimumInputLength: 0
21	- ajax: # instead of writing the function to execute the request we use Select2's convenient helper
22	- url: "/api/" + gon.api_version + "/users.json"
23	- dataType: "json"
24	- data: (term, page) ->
25	- search: term # search term
26	- per_page: 10
27	- active: true
28	- private_token: gon.api_token
29	-
30	- results: (data, page) -> # parse the results into the format expected by Select2.
31	- # since we are using custom formatting functions we do not need to alter remote JSON data
32	- results: data	21	+ query: (query) ->
		22	+ Api.users query.term, (users) ->
		23	+ data = { results: users }
		24	+ query.callback(data)
33		25
34	initSelection: (element, callback) ->	26	initSelection: (element, callback) ->
35	id = $(element).val()	27	id = $(element).val()
36	if id isnt ""	28	if id isnt ""
37	- $.ajax(
38	- "/api/" + gon.api_version + "/users/" + id + ".json",
39	- dataType: "json"
40	- data:
41	- private_token: gon.api_token
42	- ).done (data) ->
43	- callback data	29	+ Api.user(id, callback)
44		30
45		31
46	- formatResult: userFormatResult # omitted for brevity, see the source of this page
47	- formatSelection: userFormatSelection # omitted for brevity, see the source of this page
48	- dropdownCssClass: "ajax-users-dropdown" # apply css that makes the dropdown taller	32	+ formatResult: userFormatResult
		33	+ formatSelection: userFormatSelection
		34	+ dropdownCssClass: "ajax-users-dropdown"
49	escapeMarkup: (m) -> # we do not want to escape markup since we are displaying html in results	35	escapeMarkup: (m) -> # we do not want to escape markup since we are displaying html in results
50	m	36	m
51	-