Closed (fixed)
Project:
Mass Contact
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
12 Aug 2008 at 18:38 UTC
Updated:
22 Oct 2010 at 17:10 UTC
Jump to comment: Most recent file
Comments
Comment #1
oadaeh commentedI don't know. I imagine there would be a way to do it, even if it's contrived, but I'll have to investigate to see if it's possible and how much effort it will be.
Comment #2
BluesmanEP commentedThanks very much for looking into this.
Unfortunately, I won't be much help in the coding department, but I'm happy to test and provide feedback on any patches etc. you may come up with.
Thanks again, and let me know how I can be of help.
Evan
Comment #3
Wolfgang Reszel commentedIts really bad that I have to force an insecure input filter as standard for Mass Contact to get correct html mails. The appended and prepended code shouldn't be filtered or it should be possible to disable it.
Comment #4
oadaeh commentedI've added the ability to specify the input formats here: http://drupal.org/cvs?commit=365884
This change is not included in the brand new 1.0 release, as it was quite a bit more work and included more areas of the code than I expected, so I kind of wanted to keep it separate to make sure I didn't cause any problems.
Here's what I added:
Please test it and let me know if the changes work for you and if they're what you're looking for.
Comment #5
Wolfgang Reszel commentedThanks, that sounds great. I'll test it next week.
Comment #6
Wolfgang Reszel commentedAt my first tests it works as expected.
Comment #7
kiphaas7 commentedIt does not respect the WYSIWYG module. (it does not load the wysiwyg editor, no firebug console errors, worked just fine in previous mass_contact versions).
If I have the time, I will look into the code and try and find the culprit...
Comment #8
kiphaas7 commentedPretty easy patch, restoring div wrapper and nesting body + filters in same array.
Comment #9
kiphaas7 commentedThe div wrapper above probably is optional, but is consistent with drupal core.
Above is only a partial fix: still not fixed for situations where one isn't allowed to override html AND isn't allowed to change filter format.
Comment #10
oadaeh commentedWhich client editor are you using with the Wysiwyg module?
Comment #11
kiphaas7 commentedtiny_mce 3.3.6
Comment #12
kiphaas7 commentedI fixed it for all situations now. (it has nothing to do with tiny_mce, more with the implementation of wysiwyg module itself)
I replaced some code, but most noticably the code below. It didn't seem to have any consequences, because even though you're saying you're saving the setting for later use, I couldn't find it in the _submit function.
Also, the next piece of code made me go o.O (as in: it worked...). I always thought private functions couldn't be used outside the module they were defined in..
All of the work in the patch is basically making sure the form array is back to the way core formats it, as wysiwyg module requires this array formatting for correctly hooking in.
Also changed a description to follow core text more closely.
Comment #13
oadaeh commentedThis should be fixed here: http://drupal.org/cvs?commit=433560.
If not, please reopen with more information.