View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
15811 | Bug reports | Survey editing | public | 2020-02-01 15:12 | 2021-01-28 23:47 |
Reporter | Waterlijn | Assigned To | cdorin | ||
Priority | none | Severity | minor | ||
Status | closed | Resolution | reopened | ||
Product Version | 4.1.0 | ||||
Summary | 15811: Embedding images not working well | ||||
Description | There is an old bug (14343) that causes problems when there is a subdirectory and relative images in the text. So this difference in picking the image should be solved first (or add the domain manually). Ones it is a full url, the following function can be added to: '/application/core/LimeMailer.php'
| ||||
Steps To Reproduce | Add image to the description text in a survey and add the text to a email template. | ||||
Tags | No tags attached. | ||||
Bug heat | 6 | ||||
Complete LimeSurvey version number (& build) | 4.1.0+200128 | ||||
I will donate to the project if issue is resolved | No | ||||
Browser | Any | ||||
Database type & version | not important | ||||
Server OS (if known) | not important | ||||
Webserver software & version (if known) | not important | ||||
PHP Version | not important | ||||
duplicate of | 14343 | new | Included image are not send with email if subdirectory is set |
Update: The relative url will be inserted when using 'Insert image' (upload) instead of 'Select image from server'. |
|
Best is to make the pull request ? No ? |
|
Here, you import external image too ? Right ? Then : it can take a really more long time to send email … |
|
Well, this is a solution that works for me. I personally don't want to have any external image in my mails. But it might rather be an option. |
|
In order to prevent slowness, the images might be cached like this. |
|
My opinion are : Best in a core plugin :) You can check https://github.com/LimeSurvey/LimeSurvey/tree/master/application/core/plugins/mailSenderToFrom for sample
|
|
PS : happy to help you to construct the core plugin :) |
|
Created a plugin, thanks for info. :) |
|
Shall I have this ticket closed? :) |
|
Sure |
|
Thanks for quick reply! |
|
No … Waterlijn cp,nstruct a plugin but this plugin are not public or in LimeSurvey core plugins . |
|
Yeah, but the idea is that it isn't a core problem. This would rather be solved in a plugin, so I did. |
|
Thanks for the public plugin :)
You seems to don't use |
|
Actually I didn't use the whole system for about 8 months now due to corona. But at the time it worked for me. I also wrote a plugin for DKIM, you might be interested. |
|
LS_mailDKIM is really great ! About LS_mailSendMessage : there are other purpose than «just» include image ? Right ? |
|
Sorry my bad, wrong source. This mailSendMessage was an experiment, not for publishing. |
|
Great plugin too ! Not same purpose than the original issue (include image with subdirectoy for URL) |
|
@Waterlijn : you're OK if i add your plugins in our wiki ? |
|
Yes sure, no problem. |
|
Don't find a good way … duplicate 14343: Included image are not send with email if subdirectory is set (3.X version) |
|
So, can this ticket be closed :) ? |
|
Yes please |
|
Date Modified | Username | Field | Change |
---|---|---|---|
2020-02-01 15:12 | Waterlijn | New Issue | |
2020-02-01 16:30 | Waterlijn | Note Added: 55612 | |
2020-02-01 18:12 | DenisChenu | Note Added: 55614 | |
2020-02-01 18:15 | DenisChenu | Note Added: 55615 | |
2020-02-01 18:34 | Waterlijn | Note Added: 55617 | |
2020-02-01 20:58 | Waterlijn | Note Added: 55620 | |
2020-02-02 10:35 | DenisChenu | Note Added: 55622 | |
2020-02-02 10:35 | DenisChenu | Note Added: 55623 | |
2020-02-02 10:36 | DenisChenu | Note Edited: 55623 | |
2020-02-02 12:28 | Waterlijn | Note Added: 55627 | |
2020-02-05 08:52 | cdorin | Assigned To | => cdorin |
2020-02-05 08:52 | cdorin | Status | new => assigned |
2020-04-02 20:25 | cdorin | Assigned To | cdorin => |
2020-04-03 08:27 | DenisChenu | Status | assigned => new |
2020-12-22 15:58 | cdorin | Status | new => feedback |
2020-12-22 15:58 | cdorin | Note Added: 61026 | |
2020-12-22 16:07 | Waterlijn | Note Added: 61034 | |
2020-12-22 16:07 | Waterlijn | Status | feedback => new |
2020-12-22 16:10 | cdorin | Assigned To | => cdorin |
2020-12-22 16:10 | cdorin | Status | new => closed |
2020-12-22 16:10 | cdorin | Resolution | open => no change required |
2020-12-22 16:10 | cdorin | Note Added: 61037 | |
2020-12-22 18:17 | DenisChenu | Status | closed => feedback |
2020-12-22 18:17 | DenisChenu | Resolution | no change required => reopened |
2020-12-22 18:17 | DenisChenu | Note Added: 61054 | |
2020-12-22 18:37 | Waterlijn | Note Added: 61056 | |
2020-12-22 18:37 | Waterlijn | Status | feedback => assigned |
2020-12-22 21:03 | DenisChenu | Note Added: 61058 | |
2020-12-22 21:04 | DenisChenu | Assigned To | cdorin => DenisChenu |
2020-12-22 21:15 | Waterlijn | Note Added: 61059 | |
2020-12-23 09:28 | DenisChenu | Note Added: 61066 | |
2020-12-23 11:52 | Waterlijn | Note Added: 61078 | |
2020-12-24 10:10 | DenisChenu | Note Added: 61157 | |
2020-12-26 14:34 | DenisChenu | Note Added: 61168 | |
2020-12-26 14:36 | Waterlijn | Note Added: 61169 | |
2021-01-13 07:54 | DenisChenu | Relationship added | duplicate of 14343 |
2021-01-13 07:55 | DenisChenu | Assigned To | DenisChenu => |
2021-01-13 07:55 | DenisChenu | Status | assigned => new |
2021-01-13 07:55 | DenisChenu | Complete LimeSurvey version number (& build) | Versie 4.1.0+200128 => 4.1.0+200128 |
2021-01-13 07:55 | DenisChenu | Note Added: 61512 | |
2021-01-28 19:53 | cdorin | Note Added: 61773 | |
2021-01-28 21:41 | Waterlijn | Note Added: 61794 | |
2021-01-28 23:47 | cdorin | Assigned To | => cdorin |
2021-01-28 23:47 | cdorin | Status | new => closed |