this is a followup for #357885: multilingual: select a newsletter term before you can send
I applied your recommended fix of updating to the dev version, but now im experiencing another error.
When I edit some properties of a newsletter on the simplenews admin interface (admin/content/simplenews/types/edit/$tid) the language gets set back to the current website's interface language. IE, it loses the original language setting. I have to subsequently go edit the page in taxonomy edit (admin/content/taxonomy/edit/term/$tid) and reset the language again.
Could not the information on the custom simplenews newsletter term edit page be PLUGGED INTO the taxonomy edit page, so everything regarding a simplenews newsletter term is all in one place, and get rid of this custom edit page altogether?
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | simplenews.361352.patch | 834 bytes | sutharsan |
Comments
Comment #1
sutharsan commentedOn the contrary, plans are to move away from using (only) taxonomy for issue identification.
I will look into this error.
Comment #2
sutharsan commentedAttached patch fixes this. Patch committed, please inform me your results.
Comment #3
jwilson3fail against 6.x-1.0-rc5, I couldn't even apply it by hand, apparently there's quite a big difference between the dev branch and the rc5 tag as rc5 doesnt even contain the affected function simplenews_admin_types_form_sub().
yes, i realize this was filed against dev... because when i filed it rc5 didnt exist.
Comment #4
sutharsan commentedThis has been applied to 6.x-1.x-dev (and HEAD) just before the RC5 release. It is therefore included in this release.
See cvs for details.