Merge pull request #31 from rwjblue/dont-die-without-dom

Guard against `document` being undefined.
yapplabs · May 11, 2015 · 819f27d · 819f27d
2 parents 5cb8a41 + fbb8990
commit 819f27d
Showing 1 changed file with 17 additions and 5 deletions.
diff --git a/addon/initializers/add-modals-container.js b/addon/initializers/add-modals-container.js
@@ -1,11 +1,21 @@
-/*globals document*/
-export default function(container, application){
-  var rootEl = document.querySelector(application.rootElement);
+/*globals document */
+
+var hasDOM = typeof document !== 'undefined';
+
+function appendContainerElement(rootElementId, id) {
+  if (!hasDOM) {
+    return;
+  }
+
+  var rootEl = document.querySelector(rootElementId);
   var modalContainerEl = document.createElement('div');
+  modalContainerEl.id = id;
+  rootEl.appendChild(modalContainerEl);
+}
+
+export default function(container, application){
   var emberModalDialog = application.emberModalDialog || {};
   var modalContainerElId = emberModalDialog.modalRootElementId || 'modal-overlays';
-  modalContainerEl.id = modalContainerElId;
-  rootEl.appendChild(modalContainerEl);
 
   application.register('config:modals-container-id',
                        modalContainerElId,
@@ -14,4 +24,6 @@ export default function(container, application){
   application.inject('service:modal-dialog',
                      'destinationElementId',
                      'config:modals-container-id');
+
+  appendContainerElement(application.rootElement, modalContainerElId);
 }