minor changes requested per discussion on PR

update to CDN URLs from aframevr/assets#18

Author: machenmusik

docs/components/hand-controls.md

@@ -46,5 +46,5 @@ handles hand animations and poses.
 
 ## Assets
 
-- [Left hand model](https://cdn.aframe.io/controllers/oculus-hands/leftHand.json)
-- [Right hand model](https://cdn.aframe.io/controllers/oculus-hands/rightHand.json)
+- [Left hand model](https://cdn.aframe.io/controllers/oculus-hands/v2/leftHand.json)
+- [Right hand model](https://cdn.aframe.io/controllers/oculus-hands/v2/rightHand.json)

src/components/hand-controls.js

@@ -1,7 +1,7 @@
 var registerComponent = require('../core/component').registerComponent;
 
-var OCULUS_LEFT_HAND_MODEL_URL = 'https://rawgit.com/arturitu/assets/e698020d6f335fb46a6169f8985b4e12297e157d/controllers/oculus-hands/leftHand.json'; // 'https://cdn.aframe.io/controllers/oculus-hands/leftHand.json';
-var OCULUS_RIGHT_HAND_MODEL_URL = 'https://rawgit.com/arturitu/assets/e698020d6f335fb46a6169f8985b4e12297e157d/controllers/oculus-hands/rightHand.json'; // 'https://cdn.aframe.io/controllers/oculus-hands/rightHand.json';
+var OCULUS_LEFT_HAND_MODEL_URL = 'https://cdn.aframe.io/controllers/oculus-hands/v2/leftHand.json';
+var OCULUS_RIGHT_HAND_MODEL_URL = 'https://cdn.aframe.io/controllers/oculus-hands/v2/rightHand.json';
 
 /**
 *
@@ -156,12 +156,6 @@ module.exports.Component = registerComponent('hand-controls', {
     return controllerId && controllerId.indexOf('Oculus Touch') === 0;
   },
 
-  isOculusTouchController: function () {
-    var trackedControls = this.el.components['tracked-controls'];
-    var controllerId = trackedControls && trackedControls.controller && trackedControls.controller.id;
-    return controllerId && controllerId.indexOf('Oculus Touch') === 0;
-  },
-
   determineGesture: function () {
     var gesture;
     var isGripActive = this.pressedButtons['grip'];
@@ -189,12 +183,12 @@ module.exports.Component = registerComponent('hand-controls', {
   },
 
   gestureAnimationMapping: {
-    '': 'Open',
-    'pointing': 'Point',
-    'pistol': 'Point + Thumb',
-    'fist': 'Fist',
-    'touch': 'Hold',
-    'thumb': 'Thumb Up'
+    default: 'Open',
+    pointing: 'Point',
+    pistol: 'Point + Thumb',
+    fist: 'Fist',
+    touch: 'Hold',
+    thumb: 'Thumb Up'
   },
 
   animateGesture: function (gesture) {
@@ -204,17 +198,17 @@ module.exports.Component = registerComponent('hand-controls', {
       this.playAnimation('Open', true);
       return;
     }
-    var animation = this.gestureAnimationMapping[gesture || ''];
+    var animation = this.gestureAnimationMapping[gesture || 'default'];
     this.playAnimation(animation || 'Open', !animation && isOculusTouch);
   },
 
   // map to old vive-specific event names for now
   gestureEventMapping: {
-    'fist': 'grip',         // fist: e.g. grip active, trigger active, trackpad / surface active
-    'touch': 'point',       // 'touch' e.g. trigger active, grip not active
-    'thumb': 'thumb',       // thumbs up: e.g. grip active, trigger active, trackpad / surface not active
-    'pointing': 'pointing', // pointing: e.g. grip active, trackpad / surface active, trigger not active
-    'pistol': 'pistol'      // pistol: e.g. grip active, trigger not active, trackpad / surface not active
+    fist: 'grip',         // fist: e.g. grip active, trigger active, trackpad / surface active
+    touch: 'point',       // 'touch' e.g. trigger active, grip not active
+    thumb: 'thumb',       // thumbs up: e.g. grip active, trigger active, trackpad / surface not active
+    pointing: 'pointing', // pointing: e.g. grip active, trackpad / surface active, trigger not active
+    pistol: 'pistol'      // pistol: e.g. grip active, trigger not active, trackpad / surface not active
   },
 
   gestureEventName: function (gesture, active) {
@@ -258,7 +252,10 @@ module.exports.Component = registerComponent('hand-controls', {
 
     // play new animation.
     clipAction = mesh.mixer.clipAction(animation);
-    if (!clipAction) { return; }
+    // returning when no clipAction will prevent further issues
+    // (e.g. controllers no longer updating pose)
+    // but per https://github.com/aframevr/aframe/pull/2191#discussion_r93121878
+    // the preference is to NOT prevent the issues to catch bugs earlier in QA.
     clipAction.loop = 2200;
     clipAction.clampWhenFinished = true;
     clipAction.timeScale = timeScale;