Closed (fixed)
Project:
Simplenews Template
Version:
6.x-1.0-beta1
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Reporter:
Created:
12 Dec 2008 at 16:08 UTC
Updated:
23 Mar 2009 at 15:22 UTC
Jump to comment: Most recent file
Comments
Comment #1
arvinsingla commentedI can confirm. Same results, works in the preview, but the resulting email is not affected.
Currently tested with
Simplenews 6.x-1.x-dev 2008-Dec-11
mimemail 6.x-1.x-dev 2008-Sep-21
Comment #2
quadbyte commentedWhat I noticed :
- It works in the preview tab when editing a newsletter.
- It doesn't work if you send a test email
- it works when you send the newsletter.
Comment #3
oneklema commentedI am also only able to see the custom header/footer for a newsletter when previewing. When the newsletter is sent, as a test or not, the header/footer do not appear.
Comment #4
asak commentedConfirming.
For me it doesn't work on the test email, but does work when sending newsletter to subscribers.
Comment #5
oneklema commentedI also have similar behavior where the header/footer does not show in the emails. We also had the very strange behavior of email header/footers that had been deleted (i.e., old versions of the header/footer) showing up in the emails/preview. I can't seem to pin down exactly what conditions cause which behavior.
Comment #6
onejam commentedSame as all above. Here's the error:
And the sent email only shows:
Newsletter was sent as a test only.
Thanks,
Comment #7
blackeagle commentedI think I have the same Problem!
Here is the Error - I don't know what to do....
* warning: Missing argument 3 for _simplenews_template_theme_simplenews_newsletter_body() in /var/www/web1222/html/cms/sites/all/modules/simplenews_template/simplenews_template.module on line 370.
* recoverable fatal error: Object of class stdClass could not be converted to string in /var/www/web1222/html/cms/sites/all/modules/simplenews/simplenews.module on line 1555.
* recoverable fatal error: Object of class stdClass could not be converted to string in /var/www/web1222/html/cms/sites/all/modules/simplenews/simplenews.module on line 1555.
* recoverable fatal error: Object of class stdClass could not be converted to string in /var/www/web1222/html/cms/sites/all/modules/simplenews/simplenews.module on line 1555.
* recoverable fatal error: Object of class stdClass could not be converted to string in /var/www/web1222/html/cms/sites/all/modules/simplenews/simplenews.module on line 1555.
Comment #8
zoltansoos commentedConfirming:
Missing argument 3 for _simplenews_template_theme_simplenews_newsletter_body() in [....]/html/sites/all/modules/simplenews_template/simplenews_template.module on line 370
Comment #9
IvanSandpile commentedI had the same error. It seems the $body var is a stdClass but is treated like a string.
I changed the line 1555 to:
$body = strtr($body->body, $variables);and the errors stopped/ emails came through. Not sure how that will affect the rest of the code though.
Comment #10
michaeldhart commentedSubscribing...can confirm the same problem. Fix from #9 removes errors and email goes through but still without header and footer. Header and footer only visible through preview.
Comment #11
kenorb commentedMarked #353828: No Content in Newsletter as duplicate.
Description of issue: http://drupal.org/node/353828#comment-1196385
Comment #12
michaeldhart commentedI tried solution from duplicate issue here:
http://drupal.org/node/353828#comment-1196385
this resulted in no email body again. Only the unsubscribe portion at the end of the email was included. I tried this fix both with and without the fix mentioned above. neither worked for me.
Comment #13
airone35 commentedFooter and header are working for me but the body do not displays.
I try with and without : http://drupal.org/node/353828#comment-1196385 (without there is only "Object" in body, with the body is empty)
and with and without the fix mentionned in #9
It seems that $issue_body variable has no value or is not considered as a string even with the modification
($body = strtr($body->body, $variables);in simplenews.module (already exists in the last version)
Comment #14
McCool commentedSubscribing. Same problem...
Comment #15
jwilson3I've tested a handful of different hacks to get 6.x-1.0-beta1 working with simplenews 6.x-1.0-rc4 to no success. It unfortunately looks like simplenews and or mimemail have changed their functionality and the hooks to import the html + header + footer just arent getting into the sent mails.
This basically renders simplenews-template unusable for D6 :/
Comment #16
airone35 commentedWorks for me with simplenews last dev versionComment #17
kenorb commentedIt's almost working.
There is header, footer and some content (but I'm not sure if it's the correct one, because I'm using it with node reference).
Please make some testing.
Install the lastest simplenews dev version.
Patch applied from: http://drupal.org/node/353828#comment-1196385 by btsukuda
after some cleaning.
Please test it.
Comment #18
pawel.traczynski commentedAny other suggestions/solutions?
Comment #19
kenorb commentedPawle: What kind of suggestion? It still doesn't work? What's the error?
Comment #20
pawel.traczynski commentedKenorb: did you try patching it?
If yes, can you be so generous and send me the patched version of the file?
I'm using Windows platform. I have no possibilities of patching the file.
Comment #21
kenorb commentedThere is patch version for Windows, or some other tools like WinMerge, Eclipse, TortoiseSVN and some other developent stuff has it.
http://gnuwin32.sourceforge.net/packages/patch.htm
Find a dump in attachment.
Comment #22
pawel.traczynski commentedHey. I tried the patch and for me it's not working. I switched the *.module file with the file from the attachment from previous post.
Without the patch I receive newsletters with "Object" text in place where the message body/content should be - means between the header and footer. Also I get this error:
recoverable fatal error: Object of class stdClass could not be converted to string in /prezenty/sites/all/modules/simplenews/simplenews.module on line 1557.
After patching the simplenews module I get no errors and the "Object" text disapperas from the newsletter emails - but the content doesn't show up. So generally it looks like:
- Header
- Title
- [blank space]
- Footer
Without the patch:
- Header
- Title
- "object" text
- Footer
And of course should be:
- Header
- Title
- Content of the message
- Footer
Kenorb: does the patch works for You? Do you use home.pl as hosting company? I do.
Best regards.
Comment #23
kenorb commentedPawel@
I'm using home.pl as well, but most of Drupal pages I've got on HeartInternet servers (but it's not good enough as well).
But, I'm trying to make it in more difficult way, that user will select the existing template of email and the header and footer will be generated using some dynamic data (like logo of company, etc.)
But I don't know if somebody knows some other modules which can be used to do that this way;/
Comment #24
pawel.traczynski commentedSorry, but i'm unable to help :/ It is complicated...
But if you like see my other fix for the problem with including parameters in the email title.
(for example when someone wants to subscribe to the newsletter and you want the confirmation email to be sent with !site in title)
http://drupal.org/node/332199#comment-1216533
Comment #25
kenorb commentedMarked #363017: Empty email - one word "object" in the mail body as duplicated.
Comment #26
kenorb commentedNon-working patch at #17 can be related to this issue: #357799: drupal_render api documentation
Comment #27
kenorb commentedPlease do not try this patch, try the second one above.
Comment #28
kenorb commentedTry this patch.
Comment #29
keesje commented+1 for patch in #28.
Works fine for me.
Comment #30
ratnesh aarohi commentedCan somebody help me (a novice) on how to apply the patch in #28?
What exactly do i Do?
Do i open the patch file and copy the contents into "some" file?
Do i copy the contents into the file "simplenews_template.module"
If yes where do i copy or and if not pl guide me approrpitely.
Comment #31
kenorb commentedgeniekids@
Download, install and follow the instruction here:
http://drupal.org/node/60108
For Windows, type in Google: patch for Windows
See also here: http://drupal.org/node/60179
Comment #32
kenorb commentedThere is one warning to fix:
Comment #33
jwilson3re: #32, did you check the patch out here: http://drupal.org/node/353828, at the end of comment #4?
Comment #34
cwhitcoe commentedI have tried everything is the patch in #28, and I am still getting the problem...
Comment #35
matteo.boria commented+2 for patch in #28.
Works fine for me too.
Simplenews: 6.x-1.0-rc4
Mime Mail: 6.x-1.x-dev
Simplenews Template: 6.x-1.0-beta1
[multilingual site with Internationalization (i18n) 6.x-1.0]
Comment #36
kenorb commented#33: This is the same patch, but modified.
So patch you proposing has the same bug.
Comment #37
TBarregren commentedI've applied the patch in #28 (thanks kenorb) and rolled a new beta.
Please notice that the D6 port still contains bugs and other problems.
Comment #38
kenorb commentedIssue at #32 you can fix here: #372190: Wrong argument for reset()
Comment #40
jwilson3should this have been automatically closed if its a critical bug?
Comment #41
kenorb commentedIf it's fixed, why not.
http://drupal.org/node/367289