tonyair/silverstripe-reports
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-reports synced 2024-06-25 22:19:28 +02:00
Code Issues Projects Releases Wiki Activity

elofgren: BUGFIX: Don't save new Newsletter drafts as soon as they are created to prevent TinyMCE Javascript errors in IE. Always save Newsletter drafts before sending to prevent unsaved newsletter drafts from being sent out completely blank because change detection for new newsletter drafts does not work in IE.

Browse Source 
(merged from branches/gsoc)


git-svn-id: svn://svn.silverstripe.com/silverstripe/open/modules/cms/trunk@42053 467b73ca-7a2a-4603-9d3b-597d59a354a9
This commit is contained in:
Ingo Schommer 2007-09-16 02:28:14 +00:00
parent f72e6d9a56
commit 4167ab9108
2 changed files with 6 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
javascript/NewsletterAdmin_left.js
View File

@ -304,9 +304,6 @@ AddForm.prototype = {
} else {
$('sitetree').addTypeNode('New newsletter type', $('Form_EditForm_ID').value );
}
// Save it so that change detection will work
$('Form_EditForm').save();
statusMessage('Added new ' + type);
},

15
javascript/NewsletterAdmin_right.js
View File

@ -284,10 +284,9 @@ Behaviour.register({
}
this.sendingText.style.display = '';
var form = $('Form_EditForm');
if( form.isChanged() )
form.save();
// Save always because change detection doesn't work for IE on newly created drafts.
// Use onclick instead of form.save() to make things work in IE.
$('Form_EditForm_action_save').onclick();
// Send the request
ajaxSubmitForm(false, this.onCompleteTest.bind(this), this, '', 'sendnewsletter')
@ -327,11 +326,9 @@ Behaviour.register({
}
this.sendingText.style.display = '';*/
var form = $('Form_EditForm');
if( form.isChanged() )
form.save();
// Save always because change detection doesn't work for IE on newly created drafts.
// Use onclick instead of form.save() to make things work in IE.
$('Form_EditForm_action_save').onclick();
if( $('SendProgressBar') )
$('SendProgressBar').start();