View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0016706 | phpList 3 application | Attachments | public | 31-10-12 11:11 | 28-11-12 15:20 |
Reporter | dubwise259 | ||||
Priority | normal | Severity | minor | Reproducibility | always |
Status | resolved | Resolution | fixed | ||
Platform | Linux | OS | Ubuntu | OS Version | 10.04 LTS |
Product Version | 2.11.7 | ||||
Target Version | Fixed in Version | 2.11.8 | |||
Summary | 0016706: EMBEDUPLOADIMAGES functionality | ||||
Description | I'm still unable to get embedded image, even with the new version of class.phpmailer.php provided by michiel in issue 0016702 https://mantis.phplist.com/view.php?id=16702 Path/URL now looks like http://phplist_url/upload/image/my_logo.jpg, so image is properly displayed with its full url path but it is not embedded / encoded in base64, only powerphplist.png is embedded. | ||||
Steps To Reproduce | Fresh install phplist 2.11.7 Start a new message, add an image to the content and send a test message. Email receiver can see the image but if you look at the source code, image is not embedded. | ||||
Additional Information | As issue 0016702 had been closed, I've had to open a new issue. | ||||
Tags | No tags attached. | ||||
|
are you sending your queue with commandline or in-browser? presumably commandline. For commandline, it needs to save the "Image Path" in the DB, because that's not available without the Webserver. Can you check the DB and see the value for: select value from phplist_config where item = "uploadimageroot"; |
|
I'm sending the queue from the browser using "send test" button. I've checked the value of the field "uploadimageroot", it was empty so I put the same value as in config.php Then I send another test message but nothing changed, image appears well because url is correct but message's source code reveals that it is not embedded, only "powerphplist.png" is embedded. Regards |
|
The / is missing between the docroot and uploadimagesdir in function filesystem_image_exists. This should solve it: diff class.phplistmailer.php.org class.phplistmailer.php 427,428c427,428 < is_file($_SERVER['DOCUMENT_ROOT'].UPLOADIMAGES_DIR.'/image/'.$localfile) < || is_file($_SERVER['DOCUMENT_ROOT'].UPLOADIMAGES_DIR.'/'.$localfile) --- > is_file($_SERVER['DOCUMENT_ROOT'].'/'.UPLOADIMAGES_DIR.'/image/'.$localfile) > || is_file($_SERVER['DOCUMENT_ROOT'].'/'.UPLOADIMAGES_DIR.'/'.$localfile) |
|
thanks for finding that revision 3483 |