From: Łukasz Rekucki Date: Wed, 26 Aug 2009 08:50:11 +0000 (+0200) Subject: Refactor kodu paneli. Wysyłanie danych do save(). X-Git-Url: https://git.mdrn.pl/redakcja.git/commitdiff_plain/654e7cf77f7241bfc866a4879829367d39179577?ds=sidebyside;hp=--cc Refactor kodu paneli. Wysyłanie danych do save(). --- 654e7cf77f7241bfc866a4879829367d39179577 diff --git a/project/static/js/editor.js b/project/static/js/editor.js index 3cd59f0f..4a1c9d1d 100644 --- a/project/static/js/editor.js +++ b/project/static/js/editor.js @@ -1,31 +1,152 @@ -$(function() -{ - function saveToBranch(data) { - $.log('Saving to local branch'); - var changed_panel = $('.panel-wrap.changed'); - - if( changed_panel.length == 0) - return; /* no changes */ - - if( changed_panel.length > 1) - alert('Błąd: więcej niż jeden panel został zmodyfikowany. Nie można zapisać.'); - - save_data = changed_panel.data('ctrl').saveData(); - - $.ajax({ - url: location.href, - dataType: 'json', - success: function(data, textStatus) { - $.log('Success:', data); - }, - error: function(rq, tstat, err) { - $.log('save error', rq, tstat, err); - }, - type: 'POST', - data: save_data - }); - } +function Panel(panelWrap) { + var self = this; + self.wrap = panelWrap; + self.contentDiv = $('.panel-content', panelWrap); + self.instanceId = Math.ceil(Math.random() * 1000000000); + $.log('new panel - wrap: ', self.wrap); + - $('#toolbar-button-save').click(saveToBranch); -}); + $(document).bind('panel:unload.' + self.instanceId, + function(event, data) { self.unload(event, data); }); + + $(document).bind('panel:contentChanged', function(event, data) { + $.log(self, ' got changed event from: ', data); + if(self != data) + self.otherPanelChanged(event.target); + else + self.changed(); + + return false; + }); + +} + +Panel.prototype.callHook = function(hookName) { + if(this.hooks && this.hooks[hookName]) + { +// arguments.shift(); + $.log('calling hook: ', hookName, 'with args: ', arguments); + this.hooks[hookName].apply(this, arguments); + } +} + +Panel.prototype.load = function (url) { + $.log('preparing xhr load: ', this.wrap); + $(document).trigger('panel:unload', this); + var self = this; + + $.ajax({ + url: url, + dataType: 'html', + success: function(data, tstat) { + panel_hooks = null; + $(self.contentDiv).html(data); + self.hooks = panel_hooks; + panel_hooks = null; + self.callHook('load'); + }, + error: function(request, textStatus, errorThrown) { + $.log('ajax', url, this.target, 'error:', textStatus, errorThrown); + } + }); +} + +Panel.prototype.unload = function(event, data) { + $.log('got unload signal', this, ' target: ', data); + + if( data == this ) { + $.log('unloading', this); + $(document).unbind('panel:unload.' + this.instanceId); + $(this.contentDiv).html(''); + this.callHook('unload'); + this.hooks = null; // flush the hooks + return false; + }; +} + +Panel.prototype.otherPanelChanged = function(other) { + $.log('panel ', other, ' changed.'); + $('.change-notification', this.wrap).fadeIn(); + this.callHook('dirty'); +} + +Panel.prototype.changed = function () { + this.wrap.addClass('changed'); +} + +Panel.prototype.saveInfo = function() { + var saveInfo = {}; + this.callHook('saveInfo', saveInfo); + return saveInfo; +} + + +function Editor() { + // editor initialization + // this is probably a good place to set config +} +Editor.prototype.setupUI = function() { + // set up the UI visually and attach callbacks + var self = this; + var panelRoot = $('#panels'); + + panelRoot.makeHorizPanel({}); // TODO: this probably doesn't belong into jQuery + panelRoot.css('top', ($('#header').outerHeight() ) + 'px'); + + $('#panels > *.panel-wrap').each(function() { + var panelWrap = $(this); + $.log('wrap: ', panelWrap); + panelWrap.data('ctrl', new Panel(panelWrap)); // attach controllers to wraps + + $('.panel-toolbar select', panelWrap).change(function() { + panelWrap.data('ctrl').load( $(this).val() ); + }); + }); + + $('#toolbar-button-save').click( function (event, data) { self.saveToBranch(); } ); + + +} + +Editor.prototype.loadConfig = function() { + // load options from cookie +} + +Editor.prototype.saveToBranch = function() { + var changed_panel = $('.panel-wrap.changed'); + $.log('Saving to local branch - panel:', changed_panel); + + if( changed_panel.length == 0) { + $.log('Nothing to save.'); + return; /* no changes */ + } + + if( changed_panel.length > 1) { + alert('Błąd: więcej niż jeden panel został zmodyfikowany. Nie można zapisać.'); + return; + } + + saveInfo = changed_panel.data('ctrl').saveInfo(); + + $.ajax({ + url: location.href + (saveInfo.part || ''), + dataType: (saveInfo.dataType || 'text'), + success: function(data, textStatus) { + $.log('Success:', data); + }, + error: function(rq, tstat, err) { + $.log('save error', rq, tstat, err); + }, + type: 'POST', + data: (saveInfo.content || '') + }); +}; + +$(function() { + editor = new Editor(); + + // do the layout + editor.loadConfig(); + editor.setupUI(); +}); diff --git a/project/templates/explorer/file_xml.html b/project/templates/explorer/file_xml.html index fe7e0c0e..baeb3d12 100644 --- a/project/templates/explorer/file_xml.html +++ b/project/templates/explorer/file_xml.html @@ -26,7 +26,7 @@ - +
@@ -40,7 +40,7 @@ - +
diff --git a/project/templates/explorer/panels/dceditor.html b/project/templates/explorer/panels/dceditor.html index ee5146cd..96c2c47e 100644 --- a/project/templates/explorer/panels/dceditor.html +++ b/project/templates/explorer/panels/dceditor.html @@ -5,17 +5,5 @@ diff --git a/project/templates/explorer/panels/gallery.html b/project/templates/explorer/panels/gallery.html index 6c44a19b..d339bce6 100644 --- a/project/templates/explorer/panels/gallery.html +++ b/project/templates/explorer/panels/gallery.html @@ -11,23 +11,20 @@ diff --git a/project/templates/explorer/panels/htmleditor.html b/project/templates/explorer/panels/htmleditor.html index bd1bd50e..d5e0acb9 100644 --- a/project/templates/explorer/panels/htmleditor.html +++ b/project/templates/explorer/panels/htmleditor.html @@ -1,20 +1,12 @@ {{ html|safe }} diff --git a/project/templates/explorer/panels/xmleditor.html b/project/templates/explorer/panels/xmleditor.html index dfe94774..9186f04a 100644 --- a/project/templates/explorer/panels/xmleditor.html +++ b/project/templates/explorer/panels/xmleditor.html @@ -6,18 +6,22 @@