small fix for lists

[fnpeditor.git] / src / editor / modules / data / data.js

diff --git a/src/editor/modules/data/data.js b/src/editor/modules/data/data.js
index 7d0837b..789899c 100644 (file)
--- a/src/editor/modules/data/data.js
+++ b/src/editor/modules/data/data.js
@@ -11,39 +11,37 @@ define([
 'use strict';
 /* global gettext, alert, window */
 
 'use strict';
 /* global gettext, alert, window */
 

-var logger = logging.getLogger('editor.modules.data'),
-    stubDocument = '<section><div>' + gettext('This is an empty document.') + '</div></section>';
+var logger = logging.getLogger('editor.modules.data');

 
 
 return function(sandbox) {
 
 
 
 return function(sandbox) {
 

-    var document_id = sandbox.getBootstrappedData().document_id;
-    var history = sandbox.getBootstrappedData().history;
-    var documentDirty = false;
-    var draftDirty = false;
+    var data = sandbox.getBootstrappedData(),
+        documentDirty = false,
+        draftDirty = false,
+        wlxmlDocument;

 
 

-    var documentProperties = {};
-    var data = sandbox.getBootstrappedData();
-    Object.keys(data)
-        .filter(function(key) {
-            return key !== 'history' && key !== 'document';
-        })
-        .forEach(function(key) {
-            documentProperties[key] = data[key];
-        });
-
-    var wlxmlDocument, text;

 
     var loadDocument = function(text, isDraft, draftTimestamp) {
         logger.debug('loading document');
 
     var loadDocument = function(text, isDraft, draftTimestamp) {
         logger.debug('loading document');

+        var xmlValid = true;

         try {
         try {

-            wlxmlDocument = wlxml.WLXMLDocumentFromXML(text, {editorConfig: sandbox.getConfig()}, Document);
+            wlxmlDocument = wlxml.WLXMLDocumentFromXML(text, {editorConfig: sandbox.getConfig()}, Document.Document);

         } catch(e) {
             logger.exception(e);
         } catch(e) {
             logger.exception(e);

-            alert(gettext('This document contains errors and can\'t be loaded. :(')); // TODO
-            wlxmlDocument = wlxml.WLXMLDocumentFromXML(stubDocument, {}, Document);
+            alert(gettext('The content of this document seems to be invalid - only XML source editing will be possible. :(')); // TODO
+            wlxmlDocument = wlxml.WLXMLDocumentFromXML(text, {}, Document.DumbDocument);
+            xmlValid = false;

         }
 
         }
 

+        Object.keys(data)
+            .filter(function(key) {
+                return key !== 'history' && key !== 'document';
+            })
+            .forEach(function(key) {
+                wlxmlDocument.setProperty(key, data[key]);
+            });
+

         wlxmlDocument.registerExtension(listExtension);
         sandbox.getPlugins().forEach(function(plugin) {
             if(plugin.documentExtension) {
         wlxmlDocument.registerExtension(listExtension);
         sandbox.getPlugins().forEach(function(plugin) {
             if(plugin.documentExtension) {


@@ -62,22 +60,23 @@ return function(sandbox) {
 
         if(window.localStorage) {
             window.setInterval(function() {
 
         if(window.localStorage) {
             window.setInterval(function() {

+                var timestamp = datetime.currentStrfmt(),
+                    key = getLocalStorageKey();

                 if(modificationFlag) {
                     modificationFlag = false;
                     return;
                 }
                 if(wlxmlDocument && documentDirty && draftDirty) {
                 if(modificationFlag) {
                     modificationFlag = false;
                     return;
                 }
                 if(wlxmlDocument && documentDirty && draftDirty) {

-                    var timestamp = datetime.currentStrfmt();

                     logger.debug('Saving draft to local storage.');
                     sandbox.publish('savingStarted', 'local');
                     logger.debug('Saving draft to local storage.');
                     sandbox.publish('savingStarted', 'local');

-                    window.localStorage.setItem(getLocalStorageKey().content, wlxmlDocument.toXML());
-                    window.localStorage.setItem(getLocalStorageKey().contentTimestamp, timestamp);
+                    window.localStorage.setItem(key.content, wlxmlDocument.toXML());
+                    window.localStorage.setItem(key.contentTimestamp, timestamp);

                     sandbox.publish('savingEnded', 'success', 'local', {timestamp: timestamp});
                     draftDirty = false;
                 }
             }, sandbox.getConfig().autoSaveInterval || 2500);
         }
                     sandbox.publish('savingEnded', 'success', 'local', {timestamp: timestamp});
                     draftDirty = false;
                 }
             }, sandbox.getConfig().autoSaveInterval || 2500);
         }

-        sandbox.publish('ready', isDraft, draftTimestamp);
+        sandbox.publish('ready', isDraft, draftTimestamp, xmlValid);

     };
     
     function readCookie(name) {
     };
     
     function readCookie(name) {


@@ -108,16 +107,16 @@ return function(sandbox) {
     var reloadHistory = function() {
         $.ajax({
             method: 'get',
     var reloadHistory = function() {
         $.ajax({
             method: 'get',

-            url: sandbox.getConfig().documentHistoryUrl(document_id),
-            success: function(data) {
-                history = data;
-                sandbox.publish('historyItemAdded', data.slice(-1)[0]);
+            url: sandbox.getConfig().documentHistoryUrl(data.document_id),
+            success: function(history) {
+                data.history = history;
+                sandbox.publish('historyItemAdded', history.slice(-1)[0]);

             },
         });
     };
 
             },
         });
     };
 

-    var getLocalStorageKey = function() {
-        var base = 'draft-id:' + document_id + '-ver:' + documentProperties.version;
+    var getLocalStorageKey = function(forVersion) {
+        var base = 'draft-id:' + data.document_id + '-ver:' + (forVersion || wlxmlDocument.properties.version);

         return {
             content: base,
             contentTimestamp: base + '-content-timestamp'
         return {
             content: base,
             contentTimestamp: base + '-content-timestamp'


@@ -127,10 +126,11 @@ return function(sandbox) {
    
     return {
         start: function() {
    
     return {
         start: function() {

+            var text;

             if(window.localStorage) {
             if(window.localStorage) {

-                text = window.localStorage.getItem(getLocalStorageKey().content);
+                text = window.localStorage.getItem(getLocalStorageKey(data.version).content);

 
 

-                var timestamp = window.localStorage.getItem(getLocalStorageKey().contentTimestamp),
+                var timestamp = window.localStorage.getItem(getLocalStorageKey(data.version).contentTimestamp),

                     usingDraft;
                 if(text) {
                     logger.debug('Local draft exists');
                     usingDraft;
                 if(text) {
                     logger.debug('Local draft exists');


@@ -185,7 +185,7 @@ return function(sandbox) {
 
                 var formData = event.formData;
                 formData[documentSaveForm.content_field_name] = wlxmlDocument.toXML();
 
                 var formData = event.formData;
                 formData[documentSaveForm.content_field_name] = wlxmlDocument.toXML();

-                formData[documentSaveForm.version_field_name] = documentProperties.version;
+                formData[documentSaveForm.version_field_name] = wlxmlDocument.properties.version;

                 if(sandbox.getConfig().jsonifySentData) {
                     formData = JSON.stringify(formData);
                 }
                 if(sandbox.getConfig().jsonifySentData) {
                     formData = JSON.stringify(formData);
                 }


@@ -193,9 +193,18 @@ return function(sandbox) {
                 dialog.toggleButtons(false);
                 $.ajax({
                     method: 'post',
                 dialog.toggleButtons(false);
                 $.ajax({
                     method: 'post',

-                    url: sandbox.getConfig().documentSaveUrl(document_id),
+                    url: sandbox.getConfig().documentSaveUrl(data.document_id),

                     data: formData,
                     success: function(data) {
                     data: formData,
                     success: function(data) {

+                        // ugly!
+                        $.each(sandbox.getConfig().documentSaveForm.fields, function(i, field) {
+                            if (field.name.indexOf('for_cybernauts') !== -1) {
+                                field.checked = event.formData['textsave-for_cybernauts'] === 'on';
+                            }
+                            if (field.name.indexOf('publishable') !== -1) {
+                                field.checked = event.formData['textsave-publishable'] === 'on';
+                            }
+                        });

                         event.success();
                         sandbox.publish('savingEnded', 'success', 'remote', data);
 
                         event.success();
                         sandbox.publish('savingEnded', 'success', 'remote', data);
 


@@ -204,7 +213,7 @@ return function(sandbox) {
                                 return key !== 'text';
                             })
                             .forEach(function(key) {
                                 return key !== 'text';
                             })
                             .forEach(function(key) {

-                                documentProperties[key] = data[key];
+                                wlxmlDocument.setProperty(key, data[key]);

                             });
 
                         reloadHistory();
                             });
 
                         reloadHistory();


@@ -219,12 +228,12 @@ return function(sandbox) {
 
         },
         getHistory: function() {
 
         },
         getHistory: function() {

-            return history;
+            return data.history;

         },
         fetchDiff: function(ver1, ver2) {
             $.ajax({
                 method: 'get',
         },
         fetchDiff: function(ver1, ver2) {
             $.ajax({
                 method: 'get',

-                url: sandbox.getConfig().documentDiffUrl(document_id),
+                url: sandbox.getConfig().documentDiffUrl(data.document_id),

                 data: {from: ver1, to: ver2},
                 success: function(data) {
                     sandbox.publish('diffFetched', {table: data, ver1: ver1, ver2: ver2});
                 data: {from: ver1, to: ver2},
                 success: function(data) {
                     sandbox.publish('diffFetched', {table: data, ver1: ver1, ver2: ver2});


@@ -255,7 +264,7 @@ return function(sandbox) {
                 $.ajax({
                     method: 'post',
                     dataType: 'json',
                 $.ajax({
                     method: 'post',
                     dataType: 'json',

-                    url: sandbox.getConfig().documentRestoreUrl(document_id),
+                    url: sandbox.getConfig().documentRestoreUrl(data.document_id),

                     data: formData,
                     success: function(data) {
                         Object.keys(data)
                     data: formData,
                     success: function(data) {
                         Object.keys(data)


@@ -263,7 +272,7 @@ return function(sandbox) {
                                 return key !== 'document';
                             })
                             .forEach(function(key) {
                                 return key !== 'document';
                             })
                             .forEach(function(key) {

-                                documentProperties[key] = data[key];
+                                wlxmlDocument.setProperty(key, data[key]);

                             });
                         reloadHistory();
                         wlxmlDocument.loadXML(data.document);
                             });
                         reloadHistory();
                         wlxmlDocument.loadXML(data.document);


@@ -281,12 +290,6 @@ return function(sandbox) {
             draftDirty = false;
             logger.debug('Draft dropped');
             sandbox.publish('draftDropped');
             draftDirty = false;
             logger.debug('Draft dropped');
             sandbox.publish('draftDropped');

-        },
-        getDocumentId: function() {
-            return document_id;
-        },
-        getDocumentProperties: function() {
-            return documentProperties;

         }
     };
 };
         }
     };
 };