View Issue Details

This bug affects 1 person(s).
IDProjectCategoryView StatusLast Update
12468Feature requestsExpression Managerpublic2017-09-22 10:52
ReporterMazi Assigned ToDenisChenu  
Status closedResolutionfixed 
Summary12468: EM placeholder {SID} does not get replaced at email templates

When using placeholder {SID} within invitation emails (and probably also other emails) the placeholder does NOT get replaced but instead simply "SID" is shown within the email.

Steps To Reproduce

Add placeholder {SID} at the invitation email template and send yourself an email.

Additional Information

This affects both LS versions. I tested Version 2.65.2+170606 and Version 2.6.4-lts Build 170202.

TagsNo tags attached.
Bug heat6

Users monitoring this issue

User List There are no users monitoring this issue.




2017-06-13 12:13

developer   ~43889

And i think it never work



2017-06-15 09:38

partner   ~43909

I can confirm that I have never seen this to work, but I think it would be a very good addition.

Currently I have to update the SID hardcoded in the email template every time that I copy a survey that uses this.



2017-06-15 09:55

updater   ~43910

This is also needed for the following case: Some companies allow accessing the admin interface from the Intranet only e.g. while making surveys accessible to the public using e.g. But this means that when using tokens and the {SURVEYURL} placeholder, users will be emailed links that point to the domain only reachable from the Intranet.
You can solve this by hard-coding the URL and dynamically appending SID and TOKEN, e.g.{SID}/newtest/Y/lang/en/token/{TOKEN}

But this only works when being able to use the {SID} placeholder properly.

I hope it's not that complex to extend the list of supported placeholders?!
If implemented, please also push the change to our LTS release since the user who reported this is paying for the LTS support.



2017-06-15 10:05

developer   ~43911

@Mazi : NO : you don't need to make the url manually like this. You have information 2 times in manual



2017-06-15 10:23

updater   ~43912

@DenisChenu: Thanks for the links, that indeed solves this problem.

Still, it would be very nice if we could use this placeholder for various reasons.



2017-06-17 15:47

developer   ~43930

@Mazi : Please ALWAYS add a little lss file with one or two questions .



2017-06-17 16:36

developer   ~43932

Fix committed to master branch:



2017-06-17 16:36

developer   ~43933

This is really a new feature

Related Changesets

LimeSurvey: master fadb0cc4

2017-06-17 15:51:04


Details Diff
Fixed issue #12101: Placeholder "TOKEN" does not work in Equation question
Dev: Add it in LimeExpressionManager->setVariableAndTokenMappingsForExpressionManager function
Dev: Add SID too (see 12468)
Affected Issues
12101, 12468
mod - application/helpers/expressions/em_manager_helper.php Diff File
mod - application/helpers/replacements_helper.php Diff File

LimeSurvey: master f94dcebc

2017-06-17 16:34:21


Details Diff
New feature 12468: EM placeholder {SID} in email templates
Dev: feature is here in
Dev: Show it better in replacement helper
Affected Issues
mod - application/controllers/admin/limereplacementfields.php Diff File

Issue History

Date Modified Username Field Change
2017-06-13 12:01 Mazi New Issue
2017-06-13 12:13 DenisChenu Note Added: 43889
2017-06-15 09:38 tammo Note Added: 43909
2017-06-15 09:55 Mazi Note Added: 43910
2017-06-15 10:05 DenisChenu Note Added: 43911
2017-06-15 10:23 Mazi Note Added: 43912
2017-06-17 15:47 DenisChenu Note Added: 43930
2017-06-17 16:21 DenisChenu Changeset attached => LimeSurvey master fadb0cc4
2017-06-17 16:22 DenisChenu Project Bug reports => Feature requests
2017-06-17 16:22 DenisChenu Assigned To => DenisChenu
2017-06-17 16:22 DenisChenu Status new => assigned
2017-06-17 16:22 DenisChenu Severity @60@ => feature
2017-06-17 16:22 DenisChenu Product Version 2.65.x =>
2017-06-17 16:36 DenisChenu Changeset attached => LimeSurvey master f94dcebc
2017-06-17 16:36 DenisChenu Note Added: 43932
2017-06-17 16:36 DenisChenu Resolution open => fixed
2017-06-17 16:36 DenisChenu Status assigned => resolved
2017-06-17 16:36 DenisChenu Note Added: 43933
2017-09-22 10:52 c_schmitz Status resolved => closed
2017-09-22 10:52 c_schmitz Fixed in Version => 2.x.x