Skip to main content

This site requires you to update your browser. Your browsing experience maybe affected by not having the most up to date version.

We're retiring the forums!

The SilverStripe forums have passed their heyday. They'll stick around, but will be read only. We'd encourage you to get involved in the community via the following channels instead:

All other Modules /

Discuss all other Modules here.

Moderators: martimiz, Sean, Ed, biapar, Willr, Ingo, swaiba

Module Newsletter v0.4-RC1 could work with Silverstripe 2.3.6 ?


Go to End


2 Posts   1262 Views

Avatar
Myrdhin

Community Member, 70 Posts

23 March 2010 at 5:02am

Edited: 24/04/2010 12:34am

I have errors when i try to use it...

For example, one error about the use of SS_HTTPRequest (only in SS 2.4?) in NewsletterAdmin.php, line 140 :

public function shownewsletter($params) {
	if($params instanceof SS_HTTPRequest){
		$params = $params->allParams();
	}

	return $this->showWithEditForm( $params, $this->getNewsletterEditForm( $params['ID'] ) );
}

I modified like this to "support" HTTPRequest (SS < 2.4) :

public function shownewsletter($params) {
	if($params instanceof SS_HTTPRequest || $params instanceof HTTPRequest){
		$params = $params->allParams();
	}
         
	return $this->showWithEditForm( $params, $this->getNewsletterEditForm( $params['ID'] ) );
}

Now i have another error when i try to send my newsletter to the email list :

236:
sentListNode is null

draft_sent_ok( '3', '2', '3' ); Behaviour.addLoader(hideLoading);

if((typeof tinyMCE != 'undefined')) {
	tinyMCE.init({"friendly_name":"Default CMS","priority":"50","mode":"none","editor_selector":"htmleditor","width":"100%","auto_resize":false,"theme":"advanced","theme_advanced_layout_manager":"SimpleLayout","theme_advanced_toolbar_location":"top","theme_advanced_toolbar_align":"left","theme_advanced_toolbar_parent":"right","blockquote_clear_tag":"p","table_inline_editing":true,"safari_warning":false,"relative_urls":true,"verify_html":true,"language":"en","content_css":"cms\/css\/editor.css, \/css\/editor.css","body_class":"typography","document_base_url":"http:\/\/dev.myserver.com\/","urlconverter_callback":"nullConverter","setupcontent_callback":"sapphiremce_setupcontent","cleanup_callback":"sapphiremce_cleanup","template_templates":[{"title":"Three column","src":"assets\/snippet.html","description":"A simple 3 column layout"}],"use_native_selects":true,"valid_elements":"+a[id|rel|rev|dir|tabindex|accesskey|type|name|href|target|title|class],-strong\/-b[class],-em\/-i[class],-strike[class],-u[class],#p[id|dir|class|align|style],-ol[class],-ul[class],-li[class],br,img[id|dir|longdesc|usemap|class|src|border|alt=|title|width|height|align],-sub[class],-sup[class],-blockquote[dir|class],-table[border=0|cellspacing|cellpadding|width|height|class|align|summary|dir|id|style],-tr[id|dir|class|rowspan|width|height|align|valign|bgcolor|background|bordercolor|style],tbody[id|class|style],thead[id|class|style],tfoot[id|class|style],-td[id|dir|class|colspan|rowspan|width|height|align|valign|scope|style],-th[id|dir|class|colspan|rowspan|width|height|align|valign|scope|style],caption[id|dir|class],-div[id|dir|class|align|style],-span[class|align|style],-pre[class|align],address[class|align],-h1[id|dir|class|align|style],-h2[id|dir|class|align|style],-h3[id|dir|class|align|style],-h4[id|dir|class|align|style],-h5[id|dir|class|align|style],-h6[id|dir|class|align|style],hr[class],dd[id|class|title|dir],dl[id|class|title|dir],dt[id|class|title|dir]","extended_valid_elements":"img[class|src|alt|title|hspace|vspace|width|height|align|onmouseover|onmouseout|name|usemap],iframe[src|name|width|height|align|frameborder|marginwidth|marginheight|scrolling],object[width|height|data|type],param[name|value],map[class|name|id],area[shape|coords|href|target|alt]","plugins":"contextmenu,table,emotions,paste,..\/..\/tinymce_advcode,spellchecker,media,..\/..\/tinymce_ssbuttons,fullscreen","theme_advanced_buttons1":"bold,italic,underline,strikethrough,separator,justifyleft,justifycenter,justifyright,justifyfull,styleselect,formatselect,separator,bullist,numlist,outdent,indent,blockquote,hr,charmap","theme_advanced_buttons2":"undo,redo,separator,cut,copy,paste,pastetext,pasteword,spellchecker,separator,ssimage,ssflash,sslink,unlink,anchor,separator,advcode,fullscreen,separator,search,replace,selectall,visualaid,separator","theme_advanced_buttons3":"tablecontrols"});
}

http://dev.myserver.com/newsletter/javascript/NewsletterAdmin_left.js?m=1227661179
Line 236

:'(
Could you help me ?
Thanks

Avatar
Myrdhin

Community Member, 70 Posts

23 March 2010 at 5:15am

Edited: 24/03/2010 10:57pm

Ho... I didn't have a sent folder created for this newsletter... Now it's OK : it was created automatically :?