Fixing two recognize-gesture deprecation messages

.travis.yml

@@ -21,11 +21,10 @@ env:
   matrix:
     # we recommend new addons test the current and previous LTS
     # as well as latest stable release (bonus points to beta/canary)
-    - EMBER_TRY_SCENARIO=ember-lts-2.12
-    - EMBER_TRY_SCENARIO=ember-lts-2.16
-    - EMBER_TRY_SCENARIO=ember-lts-2.18
     - EMBER_TRY_SCENARIO=ember-lts-3.8
     - EMBER_TRY_SCENARIO=ember-lts-3.12
+    - EMBER_TRY_SCENARIO=ember-lts-3.16
+    - EMBER_TRY_SCENARIO=ember-lts-3.20
     - EMBER_TRY_SCENARIO=ember-release
     - EMBER_TRY_SCENARIO=ember-beta
     - EMBER_TRY_SCENARIO=ember-canary

addon/modifiers/recognize-gesture.js

@@ -1,5 +1,5 @@
 
-import Modifier from 'ember-class-based-modifier';
+import Modifier from 'ember-modifier';
 import { getOwner } from '@ember/application';
 
 export default class RecognizeGestureModifier extends Modifier {
@@ -9,30 +9,34 @@ export default class RecognizeGestureModifier extends Modifier {
         this.recognizers = null;
         this.manager = null;
         this.gestures = getOwner(this).lookup('service:-gestures');
-
-        if (this.args.positional) {
-            this.recognizers = this.gestures.retrieve(this.args.positional);
-        }
+        this.gestureNames = this.args.positional;
         this.managerOptions = (this.args.named && (Object.keys(this.args.named).length > 0)) ? Object.assign({}, this.args.named) : { domEvents: true };
         this.managerOptions.useCapture = this.gestures.useCapture;
-
     }
 
     didInstall() {
-        if (!this.recognizers) return;
-        this.element.style['touch-action'] = 'manipulation';
-        this.element.style['-ms-touch-action'] = 'manipulation'; 
-        this.recognizers.then ( (recognizers) => {
-            if (this.isDestroyed) return;
-            this.sortRecognizers(recognizers);
-            this.manager = new Hammer.Manager(this.element, this.managerOptions);
-            recognizers.forEach((recognizer) => { this.manager.add(recognizer); });        
-        });
+        if (this.args.positional) {
+            Promise.resolve().then( () => {
+                this.recognizers = this.gestures.retrieve(this.gestureNames);
+                if (this.recognizers) {
+                    this.element.style['touch-action'] = 'manipulation';
+                    this.element.style['-ms-touch-action'] = 'manipulation'; 
+                    this.recognizers.then ( (recognizers) => {
+                        if (this.isDestroyed) return;
+                        this.sortRecognizers(recognizers);
+                        this.manager = new Hammer.Manager(this.element, this.managerOptions);
+                        recognizers.forEach((recognizer) => { this.manager.add(recognizer); });        
+                    });    
+                }
+            });
+        }
     }
 
     willRemove() {
-        this.manager.destroy();
-        this.manager = null;
+        if (this.manager !== null) {
+            this.manager.destroy();
+            this.manager = null;    
+        }
     }
 
     // Move each recognizer after all recognizers it excludes in the list - why?

config/ember-try.js

@@ -59,6 +59,22 @@ module.exports = function() {
             }
           }
         },
+        {
+          name: 'ember-lts-3.16',
+          npm: {
+            devDependencies: {
+              'ember-source': '~3.16.0'
+            }
+          }
+        },
+        {
+          name: 'ember-lts-3.20',
+          npm: {
+            devDependencies: {
+              'ember-source': '~3.20.0'
+            }
+          }
+        },
         {
           name: 'ember-release',
           npm: {

jsconfig.json

@@ -0,0 +1 @@
+{"compilerOptions":{"target":"es6","experimentalDecorators":true},"exclude":["node_modules","bower_components","tmp","vendor",".git","dist"]}

package.json

@@ -43,7 +43,7 @@
     "animation-frame": "~0.2.4",
     "broccoli-funnel": "^2.0.2",
     "broccoli-merge-trees": "^3.0.2",
-    "ember-class-based-modifier": "^0.10.0",
+    "ember-modifier": "^2.1.1",
     "ember-cli-babel": "^7.7.0",
     "ember-cli-htmlbars": "^2.0.1",
     "ember-cli-version-checker": "^2.1.0",
@@ -73,7 +73,7 @@
     "ember-on-modifier": "1.0.0",
     "ember-qunit": "^4.5.1",
     "ember-resolver": "^4.0.0",
-    "ember-source": "~3.1.0",
+    "ember-source": "~3.3.0",
     "ember-source-channel-url": "^1.0.1",
     "ember-try": "^0.2.23",
     "eslint-plugin-ember": "^5.0.0",