Move title tag cleanup to page-title-list service (#117 from Cropster…

…/engines)
ember-cli · Jan 3, 2018 · d536b34 · d536b34
2 parents 5c89467 + bffac38
commit d536b34
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 15 deletions.
diff --git a/addon/services/page-title-list.js b/addon/services/page-title-list.js
@@ -12,6 +12,7 @@ export default Ember.Service.extend({
     this._super();
     set(this, 'tokens', Ember.A());
     set(this, 'length', 0);
+    this._removeExistingTitleTag();
   },
 
   /**
@@ -185,5 +186,26 @@ export default Ember.Service.extend({
       }
     }
     return title.join('');
+  },
+
+  /**
+   * Remove any existing title tags from the head.
+   * @private
+   */
+  _removeExistingTitleTag() {
+    if (this._isFastboot()) {
+      return;
+    }
+
+    let titles = document.getElementsByTagName('title');
+    for (let i = 0; i < titles.length; i++) {
+      let title = titles[i];
+      title.parentNode.removeChild(title);
+    }
+  },
+
+  _isFastboot() {
+    return typeof FastBoot !== 'undefined';
   }
+
 });
diff --git a/app/instance-initializers/page-title-setup.js b/app/instance-initializers/page-title-setup.js