Create shim and move handling to instance initializer

Author: akash-dsouza-3362

app/instance-initializers/hash-to-query-redirect.js

@@ -0,0 +1,20 @@
+import hashToQuery from 'hash-to-query';
+
+export function initialize(application) {
+  let redirectToUrl = hashToQuery();
+  if (redirectToUrl) {
+    window.location.href = window.location.origin + redirectToUrl;
+  }
+  window.onhashchange = function() {
+    let router = application.lookup('service:router');
+    let redirectToUrl = hashToQuery();
+    if (redirectToUrl) {
+      router.transitionTo(redirectToUrl);
+    }
+  }
+}
+
+export default {
+  name: 'hash-to-query-redirect',
+  initialize
+};

lib/hash-to-query/index.js

@@ -8,37 +8,8 @@ module.exports = {
     return true;
   },
 
-  contentFor (type, config) {
-    if (type === 'body') {
-      return `
-<script type='text/javascript'>
-  function hashToQuery() {
-    var hash = window.location.hash;
-    if (hash && hash.length > 1) {
-      var pathName = window.location.pathname;
-      if (pathName.indexOf('/methods/') === -1 &&
-          pathName.indexOf('/properties/') === -1 &&
-          pathName.indexOf('/events/') === -1) {
-        var type = hash.slice(1, hash.indexOf('_'));
-        var name = hash.slice(hash.indexOf('_') + 1, hash.length);
-        var anchor = '?anchor=' + name + '&show=inherited,protected,private,deprecated';
-        var newPath = pathName;
-        if (type === 'method') {
-          newPath = pathName + '/methods/' + name;
-        } else if (type === 'property') {
-          newPath = pathName + '/properties/' + name;
-        } else if (type === 'event') {
-          newPath = pathName + '/events/' + name;
-        }
-        window.location.href = window.location.origin + newPath + anchor;
-      }
-    }
-  }
-  hashToQuery()
-  window.onhashchange = hashToQuery;
-</script>
-`
-    }
-    return '';
+  included(app) {
+    this._super.included.apply(this, arguments);
+    this.import('vendor/hash-to-query/index.js');
   }
 };

lib/hash-to-query/vendor/hash-to-query/index.js

@@ -0,0 +1,32 @@
+(() => {
+  /* globals define */
+  function hashToQuery() {
+    var hash = window && window.location && window.location.hash;
+    if (hash && hash.length > 1) {
+      var pathName = window.location.pathname;
+      if (pathName.indexOf('/methods/') === -1 &&
+          pathName.indexOf('/properties/') === -1 &&
+          pathName.indexOf('/events/') === -1) {
+        var type = hash.slice(1, hash.indexOf('_'));
+        var name = hash.slice(hash.indexOf('_') + 1, hash.length);
+        var anchor = '?anchor=' + name + '&show=inherited,protected,private,deprecated';
+        var newPath = pathName;
+        if (type === 'method') {
+          newPath = pathName + '/methods/' + name;
+        } else if (type === 'property') {
+          newPath = pathName + '/properties/' + name;
+        } else if (type === 'event') {
+          newPath = pathName + '/events/' + name;
+        }
+        return newPath + anchor;
+      }
+    }
+  }
+
+  define('hash-to-query', function () {
+    'use strict';
+    return {
+      default: hashToQuery
+    }
+  });
+})()

package.json

@@ -72,6 +72,7 @@
     "ember-power-select": "^1.9.2",
     "ember-resolver": "^4.3.0",
     "ember-route-action-helper": "^2.0.5",
+    "ember-router-service-polyfill": "1.0.1",
     "ember-service-worker": "0.6.7",
     "ember-service-worker-asset-cache": "0.6.1",
     "ember-service-worker-cache-fallback": "0.6.1",

tests/unit/initializers/hash-to-query-redirect-test.js

@@ -0,0 +1,24 @@
+import Ember from 'ember';
+import { initialize } from 'ember-api-docs/initializers/hash-to-query-redirect';
+import { module, test } from 'qunit';
+import destroyApp from '../../helpers/destroy-app';
+
+module('Unit | Initializer | hash to query redirect', {
+  beforeEach() {
+    Ember.run(() => {
+      this.application = Ember.Application.create();
+      this.application.deferReadiness();
+    });
+  },
+  afterEach() {
+    destroyApp(this.application);
+  }
+});
+
+// Replace this with your real tests.
+test('it works', function(assert) {
+  initialize(this.application);
+
+  // you would normally confirm the results of the initializer here
+  assert.ok(true);
+});