REFACTORING: Storing user activity

According to CouchDB documentation, Batch Mode Write improves performances and is ideal for applications such as log data.

We leave the 9 seconds delay introduced in commit aeabdb9

Author: christophe-lejeune

app/_attachments/log.js

@@ -108,6 +108,36 @@ function updateTooltip(id, content) {
     .tooltip({ trigger: 'hover', offset: "0, 8" });
 }
 
+function track_memo(user, memoid) {
+  $.ajax({
+    url: '../'+user,
+    type: 'GET'
+  }).done(function(u){
+    u = JSON.parse(u);
+    if (!u.contributors) u.contributors = [user]
+    if (!u.activity) u.activity = []
+    var obj = {
+      'doc': memoid,
+      'date': new Date().toJSON()
+    };
+    var i = -1, j = 0;
+    for (j = 0; j < u.activity.length; j++) {
+      if (u.activity[j].doc === memoid) i = j
+    }
+    if (i > -1) {
+      u.activity.splice(i, 1, obj)
+    } else {
+      u.activity.push(obj)
+    }
+    $.ajax({
+      url: '../'+user+'?batch=ok',
+      type: 'PUT',
+      contentType: 'application/json',
+      data: JSON.stringify(u),
+    })
+  })
+}
+
 function createUserDoc(user, sponsor, success, error) {
   $.ajax({
     url: relpath+'userfullname/'+fullname,

app/lib/shared.js

@@ -300,33 +300,26 @@ var shared = {
   {{/list}}\
   if ('{{logged}}') user = '{{logged}}';\
   function track(memo) {\
+    {{^list}}track_memo(user, memo){{/list}}\
+    {{#list}}\
     $.ajax({\
-      {{^list}}\
-      url: '../track_memo/'+user,\
-      {{/list}}\
-      {{#list}}\
       url: '../save_diary_order/'+user,\
-      {{/list}}\
       type: 'PUT',\
       contentType: 'application/json',\
-      {{^list}}\
-      data: memo,\
-      {{/list}}\
-      {{#list}}\
       data: JSON.stringify({\
         'diary': '{{diary}}',\
         'by': memo\
       }),\
-      {{/list}}\
-    }).done({{#list}}reload{{/list}})\
-    .fail({{#list}}function(data){$('#storing_fullname_dialog').modal('show')}{{/list}});\
+    }).done(reload)\
+    .fail(function(data){$('#storing_fullname_dialog').modal('show')});\
+    {{/list}}\
   }",
   render: "\
     render();\
     {{^statements}}\
     {{^link}}if (($('#groundings>li[id]').length > 1) || ('{{type}}' == 'coding' && $('#groundings li').first().find('.preview a').length > 1))\
         $('#remove_grounding_btn').removeClass('d-none');{{/link}}\
-    {{#type}}{{#logged}}setTimeout(function() {track('{{_id}}')}, 9000);{{/logged}}{{/type}}\
+    {{#type}}{{#logged}}track('{{_id}}'){{/logged}}{{/type}}\
     {{/statements}}\
   "
 };

app/rewrites.js

@@ -310,7 +310,6 @@ function(req2) {
     case 'rename_diagram':
     case 'adapt_diagram':
     case 'adapt_graph':
-    case 'track_memo':
     case 'modify_rights':
     case 'reader_unsubscribe':
     case 'username':

app/shows/index.js

@@ -7,7 +7,7 @@ function(o, req) {
     return {
       body: JSON.stringify({
         service: 'Cassandre',
-        revision: '3.22.09.06',
+        revision: '3.22.09.11',
         update_seq: req.info.update_seq
       })
     }