Sorry for not implementing a proper patch
But simplenews_tokens() function is using url () with the language option as a string "es", "fr", etc... instead of the whole language object, as it is supposed to be.
$language = isset($account->language->language) ? $account->language->language : language_default()->language;
Should be used user_preferred_language instead. Probably using it inside simplenews_subscribe_user() function, when account is loaded.
| Comment | File | Size | Author |
|---|---|---|---|
| #9 | simplenews_1230796_09.patch | 624 bytes | giorgosk |
| #6 | simplenews_1230796_02.patch | 665 bytes | sgabe |
| #2 | simplenews-language-1230796.patch | 650 bytes | bjaxelsen |
Comments
Comment #0.0
corbacho commentedfixing solution
Comment #1
vogelsang commentedI got this problem. Anyway I can fix it? I like to get a newsletter up and running on my new D7 site.
I'm not sure wether or not you suggest to do something or the issue is just not prioritized right now. At least I can't figure out what to with the line you have written.
Thanks in advance :-)
Edit: I found a work around. I turned of every language but one from my Drupal installation, so apparently the language check isn't done anymore and I don't get the error.
Comment #2
bjaxelsen commentedI've made this patch, please check if it works :-)
Comment #3
simon georges commentedComment #4
.jon commentedThe patch in #2 fixes the issue.
Thanks bjaxelsen!
Comment #5
simon georges commentedWhat about using
user_preferred_language($account)instead, as suggested in the initial post ?Comment #6
sgabe commentedPatch attached using
user_preferred_language($account);as Simon requested.Comment #7
simon georges commentedThanks, sgabe!
Comment #8
miro_dietiker#6: simplenews_1230796_02.patch queued for re-testing.
Comment #9
giorgoskline numbers in module have changed
but otherwise change gets rid of the notices
Comment #11
berdirThanks, commited.
Comment #12.0
(not verified) commentedadd suggestion