Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
On a clean new install of Drupal7, I am trying to use Simplenews
(7.x-1.x-dev tar.gz (79.42 KB) | zip (95.51 KB) 2011-Jan-23)
but I am always encountering this error:
"A newsletter category term is required"
I followed all the steps in README.txt file but... no chance.
Hmmm... Wish I could get this great mod running...
Comment | File | Size | Author |
---|---|---|---|
#17 | simplenews_1038248_17_taxonomy_handling.patch | 613 bytes | Simon Georges |
#8 | 1038248_simplenews.module.patch | 556 bytes | mordonez |
Comments
Comment #1
Anticosti CreditAttribution: Anticosti commentedI can use Simplenews now.
I had to comment out line 401 to 411 (file: simplenews/simplenews.module) :
401 // else {
402 // Check if a newsletter category term is selected.
403 //$terms = simplenews_get_term_values($node);
404 //if (!$terms) {
405 //form_set_error($field['field_name'], t('A newsletter category term is required.'));
406 //watchdog('simplenews', '@todo ... newsletter taxonomy is required, change settings.', array(), WATCHDOG_ERROR);
407 //}
408 //elseif (count($terms) > 1) {
409 //form_set_error($field['field_name'], t('Only one newsletter category term is allowed.'));
410 //watchdog('simplenews', '@todo ... newsletter taxonomy must be single value, change settings.', array(), WATCHDOG_ERROR);
411 //}
Comment #2
mordonez CreditAttribution: mordonez commentedSame issue here. Registering to the queue
Comment #3
olax CreditAttribution: olax commentedSame problem here.
Comment #4
mordonez CreditAttribution: mordonez commentedpriority change to major. It's not posible create newsletter content
Comment #5
eheuvel CreditAttribution: eheuvel commentedSame here, fresh Drupal 7 install
Comment #6
sw3b CreditAttribution: sw3b commentedSame for me ! And #1 did the trick !
Comment #7
Simon Georges CreditAttribution: Simon Georges commented@all, do you have a taxonomy associated to Simplenews ? Does this taxonomy have terms in it ?
(on a fresh install of Simplenews, everything was working fine for me).
Comment #8
mordonez CreditAttribution: mordonez commentedI attach a possible patch
Comment #9
Anticosti CreditAttribution: Anticosti commented@ Simon Georges (#7)
In my case I had previuosly installed Forum before Simplenews.
Before installing Simplenews I had these taxonomy categories:
Taxonomy
-- Forums
-- Tags
After Simplenews:
Taxonomy
-- Forums
-- Simplenews
-- Tags
Comment #10
Simon Georges CreditAttribution: Simon Georges commentedIf I understand correctly the patch in #8, it would be multi-language related ?
@Anticosti, is there a term in Simplenews category ?
Comment #11
sw3b CreditAttribution: sw3b commented#10 make sense. I'm on multi language. French by default.
#7 @Simon, yes I have a taxonomy assign. I always look the settings and save them before testing new module. It was a brand new setup !
Comment #12
Simon Georges CreditAttribution: Simon Georges commentedGreat, at least, we're going forward, I will try to reproduce it on a multi-language setup ;)
Comment #13
clauded CreditAttribution: clauded commentedApplied patch #8 on multilanguage site and it worked for me.
Comment #14
Simon Georges CreditAttribution: Simon Georges commentedActually, the problem persists if you try to change the language of your content (because the language of the submitted node won't match the language of the term (which will be the old language of the node).
Comment #15
czigor CreditAttribution: czigor commented#8 worked for me on a Hungarian site. Thanks! I suppose it will not work for more complicated cases though.
Comment #16
LeDucDuBleuet CreditAttribution: LeDucDuBleuet commentedI'm on multi language. French by default as well.
Patch in #8 made it all work!
Thx mordonez!
:-)
Comment #17
Simon Georges CreditAttribution: Simon Georges commentedOk, new patch that did seems to work for me, both to fix the message & to still allow changing the language of the node.
Can somebody test it so I can be sure it's the good way to go and commit it ?
Comment #18
czigor CreditAttribution: czigor commented#17 works for me.
Comment #19
Simon Georges CreditAttribution: Simon Georges commentedOk, thanks for your quick feedback, I'll wait for another positive answer, and then commit it asap (it's about time ;)).
Comment #20
Simon Georges CreditAttribution: Simon Georges commentedPatch in #17 committed. This should be available in the next 7.x-1.x-dev (within the next 12 hours).
Please reopen is you still have issues.