Hi Mike,

We just noticed a strange issue with the SunMailer. In the view that I am using for sending mails out- we recently had more than ten nodes (for longest time we had fewer than 10 nodes so we didn't notice this issue) to mail out. I noticed that the SunMailer is mailing out content only for the first ten nodes and not more than that.

Is there a setting somewhere which limits the number of nodes mailed out per newsletter?

I noticed that the preview of the newsletter also has only the first ten nodes hence the issue doesn't seem to be with the XSL but maybe in the sunmailer.xml.inc (I have not yet fully figured out the code- so please pardon my ignorance)?

CommentFileSizeAuthor
#3 sunmailer-755206.patch892 bytesMike Wacker

Comments

Mike Wacker’s picture

Mike Wacker commented

I believe this relates to how the view is configured. When you edit the view, check out "Basic settings: Items per page". It's probably set to 10. You could also turn off "Basic settings: Use Pager" as well.

rsutaria’s picture

rsutaria commented

That was actually one of my first reactions too- so I went and checked the "Items per page" setting. But after increasing it to 100 I still couldn't see more than 10 nodes in the SunMailer newsletter preview.

I have turned off the "Use Pager" setting but I still can only see 10 nodes in the newsletter.

I have tested this with another view I have which has 30 nodes per page, and even when I use that view in the SunMailer "Newsletter sections" setting- I see 10 nodes only.

Is there any other setting/config error that I could be making with regards to the SunMailer?

Thanks,
Ronak

Mike Wacker’s picture

Mike Wacker commented
Status: Active » Needs review
StatusFileSize
new sunmailer-755206.patch892 bytes

Try the following the patch. I repro'd your problem on my laptop, and this patch appears to fix it.

It's a bit hard to believe that this patch makes a difference at face value, but for some reason it does. I guess its a testament to the complexity of the Views module :P

rsutaria’s picture

rsutaria commented

Mike,

Is this patch part of the 1.x-dev release? If so, I will install the whole dev release it-self. My test-server is not very stable hence I will be upgrading the sunmailer version on the live setup.

Thanks indeed for fixing this issue!

Mike Wacker’s picture

Mike Wacker commented

It's not checked into the dev branch yet. I have some other changes going on in there, so I opted to release a patch this time.

rsutaria’s picture

rsutaria commented

This patch is working for me.

Thanks so much for the quick patch and sorry for my delay in replying.

Mike Wacker’s picture

Mike Wacker commented
Status: Needs review » Fixed

Patch has been committed to the dev branch.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.