freedom-social-xmpp - npm Package Compare versions

Comparing version 0.0.9 to 0.0.10

build/socialprovider.js

@@ -191,2 +191,11 @@ /*globals freedom:true,setTimeout,console,VCardStore */
   }
+
+  // If the destination client is ONLINE (i.e. using the same type of client)
+  // send this message with type 'normal' so it only reaches that client,
+  // otherwise use type 'chat' to send to all clients.
+  // Sending all messages as type 'normal' means we can't communicate across
+  // different client types, but sending all as type 'chat' means messages
+  // will be broadcast to all clients.
+  var messageType = (this.vCardStore.getClient(to).status == 'ONLINE') ?
+      'normal' : 'chat';
   
@@ -196,3 +205,3 @@ try {
       to: to,
-      type: 'chat'  // TODO: should be normal for users of the same client.
+      type: messageType
     }).c('body').t(msg));
@@ -199,0 +208,0 @@ } catch(e) {

demo_google/xmpp/socialprovider.js

@@ -191,2 +191,11 @@ /*globals freedom:true,setTimeout,console,VCardStore */
   }
+
+  // If the destination client is ONLINE (i.e. using the same type of client)
+  // send this message with type 'normal' so it only reaches that client,
+  // otherwise use type 'chat' to send to all clients.
+  // Sending all messages as type 'normal' means we can't communicate across
+  // different client types, but sending all as type 'chat' means messages
+  // will be broadcast to all clients.
+  var messageType = (this.vCardStore.getClient(to).status == 'ONLINE') ?
+      'normal' : 'chat';
   
@@ -196,3 +205,3 @@ try {
       to: to,
-      type: 'chat'  // TODO: should be normal for users of the same client.
+      type: messageType
     }).c('body').t(msg));
@@ -199,0 +208,0 @@ } catch(e) {

Gruntfile.js

@@ -1,4 +0,5 @@
-var FILES = require('freedom/Gruntfile.js').FILES;
-for (var key in FILES) {
-  FILES[key] = FILES[key].map(function(str) {
+var FILES = {},
+    freedomFiles = require('freedom/Gruntfile.js').FILES;
+for (var key in freedomFiles) {
+  FILES[key] = freedomFiles[key].map(function(str) {
     if (str[0] === '!') {
@@ -68,3 +69,3 @@ return '!node_modules/freedom/' + str.substr(1);
       freedomIntegration: {
-        src: FILES.src.concat(FILES.srcprovider).concat(FILES.jasminehelper).concat(['spec/helper.js']),
+        src: FILES.srcCore.concat(FILES.srcPlatform, FILES.srcProvider).concat(FILES.srcJasmineHelper).concat(['spec/helper.js']),
         options: {
@@ -75,2 +76,5 @@ specs: 'node_modules/freedom/spec/providers/social/**/*.integration.spec.js',
       }
+    },
+    jasmine_node: {
+      integration: ['spec/integration/']
     }
@@ -83,2 +87,3 @@ });
   grunt.loadNpmTasks('grunt-contrib-jasmine');
+  grunt.loadNpmTasks('grunt-jasmine-node');
 
@@ -85,0 +90,0 @@ // Default tasks.

package.json

 {
   "name": "freedom-social-xmpp",
   "description": "XMPP Social provider for freedomjs",
-  "version": "0.0.9",
+  "version": "0.0.10",
   "homepage": "http://freedomjs.org",
@@ -24,9 +24,11 @@ "bugs": {
     "grunt-contrib-copy": "~0.5.0",
-    "freedom-for-chrome": "~0.1.5",
+    "freedom-for-chrome": "~0.1.6",
     "grunt-contrib-jasmine": "^0.6.1",
-    "es5-shim": "^2.3.0"
+    "es5-shim": "^2.3.0",
+    "freedom-for-node": "~0.0.6",
+    "grunt-jasmine-node": "~0.2.1"
   },
   "peerDependencies": {
-    "freedom": "^0.4.3"
+    "freedom": "^0.4.6"
   }
 }

src/socialprovider.js

@@ -191,2 +191,11 @@ /*globals freedom:true,setTimeout,console,VCardStore */
   }
+
+  // If the destination client is ONLINE (i.e. using the same type of client)
+  // send this message with type 'normal' so it only reaches that client,
+  // otherwise use type 'chat' to send to all clients.
+  // Sending all messages as type 'normal' means we can't communicate across
+  // different client types, but sending all as type 'chat' means messages
+  // will be broadcast to all clients.
+  var messageType = (this.vCardStore.getClient(to).status == 'ONLINE') ?
+      'normal' : 'chat';
   
@@ -196,3 +205,3 @@ try {
       to: to,
-      type: 'chat'  // TODO: should be normal for users of the same client.
+      type: messageType
     }).c('body').t(msg));
@@ -199,0 +208,0 @@ } catch(e) {

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

925775

increased by0.11%

Number of package files

47

increased by6.82%

Lines of code

26675

increased by0.27%

Worsened metrics

Dev dependency count

10

increased by25%

No dependency changes