Merge pull request #105 from nymag/create-page

just get /pages/new and clone it
clay · Jul 28, 2015 · 0688624 · 0688624
2 parents 56c6301 + 80d0cd7
commit 0688624
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 16 deletions.
diff --git a/controllers/editor-toolbar.js b/controllers/editor-toolbar.js
@@ -7,7 +7,6 @@
 
 var EditorToolbar,
   dom = require('../services/dom'),
-  db = require('../services/db'),
   references = require('../services/references'),
   forms = require('../services/forms'),
   edit = require('../services/edit'),
@@ -24,19 +23,15 @@ function publish() {
 
 /**
  * Create a new page with the same layout as the current page.
- * @param {string} layoutName
+ * currently, this just clones the current page
+ * (cloning special "new" instances of the page-specific components)
+ * e.g. /components/article/instances/new
+ * @returns {Promise}
  */
-function createPage(layoutName) {
+function createPage() {
   // todo: allow users to choose their layout / components
 
-  var articlePage = {
-    layout: '/components/' + layoutName + '/instances/article',
-    main: '/components/story'
-  };
-
-  db.postToReference('/pages', articlePage).then(function (res) {
-    location.href = res[references.referenceProperty] + '.html?edit=true';
-  }).catch(console.error);
+  return edit.createPage();
 }
 
 /**
@@ -79,11 +74,7 @@ EditorToolbar.prototype = {
   /**
    * On new page button
    */
-  onNewPage: function () {
-    var layoutName = dom.find('[data-layout]').getAttribute('data-layout');
-
-    createPage(layoutName);
-  },
+  onNewPage: createPage,
 
   /**
    * On edit settings button

diff --git a/services/edit.js b/services/edit.js
@@ -290,6 +290,21 @@ function publishPage() {
   });
 }
 
+/**
+ * create a new page, cloning the current page
+ * @returns {Promise}
+ */
+function createPage() {
+  var path = '/pages/new';
+
+  return getDataOnly(path).then(function (data) {
+    delete data._ref;
+    return db.postToReference('/pages', data).then(function (res) {
+      location.href = res[references.referenceProperty] + '.html?edit=true';
+    }).catch(console.error);
+  });
+}
+
 // expose main methods (alphabetical)
 module.exports = {
   addSchemaToData: addSchemaToData,
@@ -301,6 +316,7 @@ module.exports = {
   removeSchemaFromData: removeSchemaFromData,
   removeGroupFieldsFromData: removeGroupFieldsFromData,
   publishPage: publishPage,
+  createPage: createPage,
   setSchemaCache: setSchemaCache,
   setDataCache: setDataCache,
   update: update,