Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
I have this error messag ein watchdog:
Simplenews statistics click called with illegal parameter(s).
Comment | File | Size | Author |
---|---|---|---|
#6 | d7_simplenews_statistics_working.patch | 6.46 KB | fago |
Comments
Comment #1
weseze CreditAttribution: weseze commentedThe latest dev release has a new engine under the hood that is not compatible with previous dev versions. (see #1775364: Url shortener and encryption for more information on this)
Links in previously sent mails will not work anymore (unless they are not tracked). All new sent mails will (should) work properly.
Comment #2
egarias CreditAttribution: egarias commentedI am sending a new newsletter, but the links are not working.
They are sending me to something like : http://newsletter.alesport.com/sns/click/O1/O2, showing a window showing a completely broken html:
tags.
Opciones de la capa superpuesta de administración
Si tiene problemas para acceder a las páginas de administración de este sitio, desactive la superposición en su página de perfil. Retirar este mensaje
barra de herramientas administrativa
Ocultar atajos
Editar atajos
Page Text
Origen
Traducción a Español
Comment #3
weseze CreditAttribution: weseze commentedIt looks to me like there is some other module redirecting to a user login, but doing a poor job at it...
What other modules are you running? Custom modules?
Comment #4
egarias CreditAttribution: egarias commentedThanks for fast answer
I have: Mail System, Mime Mail, Feeds, and other common modules, tell if you need more detail
I have custom module
here my main hooks
Comment #5
egarias CreditAttribution: egarias commentedProblem is here:
My $snid and $urlid are numbers but not pass the validation on:
I had to put this dirty:
Now it works
I also had to put a
after the
My newsletter link is redirecting to another site and it didn't worked until i inserted the return FALSE
Comment #6
fagoI ran into the same issue. I made it work with the attached patch.
The attached patch fixes the is_int() check to use is_numeric() as well as some other small things. It also fixes the links to continue to work in case a test-mail is sent out by using the snid of "0".
Comment #7
weseze CreditAttribution: weseze commentedPatch looks great, I'll look into it with some more detail and try and commit it. Thanks a bunch!
Comment #8
weseze CreditAttribution: weseze commentedOK superpatch. I have comitted it with a bit more comment about what it all does.
Comment #9
egarias CreditAttribution: egarias commentedIt worked for me
Thanks!
Comment #10
summerjee CreditAttribution: summerjee commentedWhen clicking on a link it drops me to the front page with this message:
Notice: Undefined variable: url simplenews_statistics_click() row 155
Comment #11
weseze CreditAttribution: weseze commented@summerjee: should be fixed in latest commit. Updated dev release will be available shortly. (waiting on the d.o cron run for that)