Closed (fixed)
Project:
Maillog / Mail Developer (obsolete)
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
28 Mar 2011 at 14:30 UTC
Updated:
1 Dec 2011 at 00:11 UTC
Jump to comment: Most recent file
Comments
Comment #1
geek-merlin$ join forces #1107986: d7 port of mail_logger?
Comment #2
damienmckennaSubscribe.
Comment #3
pluess commentedHi
This is an initial port to D7. Logging and showing the mail via devel module are working. The view at admin/reports/maillog is partially working. I didn't find any up to date information about views for D7 to migrate this code.
The Mimemail and SMTP support does not work yet. How am I supposed to achieve this in D7. The D6 code is based on the hooks supported by mimemail. In D7 we have http://api.drupal.org/api/drupal/includes--mail.inc/function/drupal_mail... but as far as I understand, this does not fully support our needs, since we cannot have a cascade of system wide mail handlers like mimemail -> maillog -> smtp.
Another approach would be to extend http://drupal.org/project/mailsystem with this functionality.
How should I proceed with this?
And of course any other issues about the D7 port are very welcome!
Comment #4
miro_dietikerGreat to find some help for an initial D7 version.
There's not that much special about D7 views port.
Note your patch contains tabs and thus is not following the drupal coding standard. Please resubmit without tabs.
BTW: you can replace the tabs e.g. in vi with this command:
:%s/\t/ /g
We'll need to work more on details to provide an implementation that works for almost all development / debug cases.
Comment #5
pluess commentedThanks for the vi command :-) ! Removed the tabs and reformatted the new code with 2 spaces.
About views: I simply increased the version number in
maillog_views_apito 3. Guess there are more small changes necessary to get everything working again. I think it's the custom handlers which are not working.I do agree 100% with your last point. It would be great if all major mail modules could agree an common API.
I think
mail_systemis a good starting point but does not provide enough. If it could take a list of default mail systems and attach a weight for everyone in order to tell in which order to call them, that would be great.It would be easy to write a module for this, but it only works if mimemail and phpmailer would be based on this module as well. In D6 mimemail did something similar (and still does in the 7.x-1.x-dev code) with
hook_mailengine.Maybe we can discus this next Wednesday when we meet in Zürich anyway.
Comment #6
miro_dietikerCommitted this to repo as an initial start.
Further more work needed.
Comment #7
Michael Zetterberg fd. Lopez commentedThe /maillog/details/# did not load for me.
Tracked it down to D7 changes to hook_theme and theme implementations.
Patch attached.
Comment #8
miro_dietikerLooks pretty right to me, thank you Michael. I'll review in detail and commit later.
Comment #9
james.williamssubscribe. I'm working with this today, and it looks fine so far to me.
Comment #10
james.williamsPermissions were not working due to a typo. Attached patch builds on the one in comment #7 and fixes this. I include a Drush-compatible patch too.
Comment #11
james.williamsOther than the permissions sorted in the above comment, everything else seems to be fine.
Comment #12
miro_dietikerFixed, james / michael
Still needs work to make maillog transparent and cleanup D7 interfaces.
Letting this on needs-work.
However please add separate D7 issues and provide separate patches there.
This issue should not be a general collection of further D7 issues.
Comment #13
miro_dietikerMarking this fixed for now. The basic port is available and works.
Please create individual issues to add bug reports and fixes.
Comment #14
tenken commentedHi. I was reading this thread and read that a 7x version exsisted. But, I could not find it on the project page. There is a "7.x-1.x-dev" version choice in the Issue form. So via Drush I was able to grab maillog-7.x-1.x-dev, but it would be helpful to show 7x-dev as a row in available versions at the bottom of the project page.
Comment #15
miro_dietikerThe 7.x-1.x-dev version is being shown on public website.
http://drupal.org/project/simplenews