View Issue Details

This bug affects 1 person(s).
 2
IDProjectCategoryView StatusLast Update
11626Feature requestsPluginspublic2018-04-06 11:48
ReporterDenisChenu Assigned ToDenisChenu  
PrioritynoneSeverityfeature 
Status closedResolutionfixed 
Summary11626: beforeTokenEMail don't happen for confirm email
Description

I'm unsure it was a bug or a feature, but
beforeTokenEmail event don't happen for token email confirm.

The name of the event seems clearly it must happen ....

Additional Information

Related to the another feature (beforeEmail for all email : token, admin, anything ...)

TagsNo tags attached.
Bug heat2
Story point estimate
Users affected %

Users monitoring this issue

There are no users monitoring this issue.

Activities

DenisChenu

DenisChenu

2016-09-07 08:31

developer   ~40703

Fix committed to master_fix11626_beforeTokenEmail branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=20414

DenisChenu

DenisChenu

2016-09-12 10:43

developer   ~40755

Fix committed to global_participants branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=20451

Related Changesets

LimeSurvey: master_fix11626_beforeTokenEmail 22049dc9

2016-09-07 06:19:16

DenisChenu

Details Diff
New feature 11626: beforeTokenEmail for confirmation email
Dev: beforeTokenEMail must happen for all token email then bug ?
Affected Issues
11626
mod - application/helpers/frontend_helper.php Diff File

LimeSurvey: global_participants 3fb1e371

2016-09-07 06:19:16

DenisChenu

Details Diff
New feature 11626: beforeTokenEmail for confirmation email
Dev: beforeTokenEMail must happen for all token email then bug ?
Affected Issues
11626
mod - application/helpers/frontend_helper.php Diff File

LimeSurvey: master_fix11626_beforeTokenEmail c1117303

2016-09-07 06:31:22

DenisChenu

Details Diff
New feature : Add survey for survey id in beforeTokenEmail event
New feature : Add model for beforeTokenEmail event (name in the DB for surveyls_email_model)
Dev: survey can be found in another way, but really : better to send it directly
Dev: type != db name, type used in getSurveyInfo, but not in models, then need model type
Affected Issues
11626
mod - application/controllers/RegisterController.php Diff File
mod - application/controllers/admin/tokens.php Diff File
mod - application/helpers/frontend_helper.php Diff File

LimeSurvey: master 4befbe59

2016-09-09 06:08:45

DenisChenu


Committer: GitHub Details Diff
Merge pull request #532 from Shnoulle/master_fix11626_beforeTokenEmail

Fix and improve beforeTokenEmail
Affected Issues
11626
mod - application/controllers/RegisterController.php Diff File
mod - application/controllers/admin/tokens.php Diff File
mod - application/helpers/frontend_helper.php Diff File

Issue History

Date Modified Username Field Change
2016-09-06 11:16 DenisChenu New Issue
2016-09-06 11:16 DenisChenu Status new => assigned
2016-09-06 11:16 DenisChenu Assigned To => DenisChenu
2016-09-07 08:31 DenisChenu Changeset attached => LimeSurvey master_fix11626_beforeTokenEmail 22049dc9
2016-09-07 08:31 DenisChenu Note Added: 40703
2016-09-07 08:31 DenisChenu Resolution open => fixed
2016-09-09 08:45 DenisChenu Changeset attached => LimeSurvey master 4befbe59
2016-09-09 08:46 DenisChenu Changeset attached => LimeSurvey master_fix11626_beforeTokenEmail c1117303
2016-09-09 08:46 DenisChenu Status assigned => resolved
2016-09-09 08:46 DenisChenu Fixed in Version => 2.5+
2016-09-12 10:43 DenisChenu Changeset attached => LimeSurvey global_participants 3fb1e371
2016-09-12 10:43 DenisChenu Note Added: 40755
2018-04-06 11:48 markusfluer Status resolved => closed