View Issue Details

IDProjectCategoryView StatusLast Update
16662Bug reportsPluginspublic2020-09-16 07:45
ReporterJmantysalo Assigned To 
PrioritynoneSeverityminor 
Status newResolutionopen 
Product Version4.3.14 
Summary16662: Event beforePermissionSetSave is not dispatched
Description

In https://github.com/LimeSurvey/LimeSurvey/blob/master/application/models/Permission.php#L262 there is a new PluginEvent of type beforePermissionSetSave created, but it has no corresponding dispatchEvent. Hence the plugin AuditLog can not fully work.

TagsNo tags attached.
Complete LimeSurvey version number (& build)Version 4.3.14+200826
I will donate to the project if issue is resolvedNo
Sync to Zoho Project
Browser
Database & DB-VersionN/A
Server OS (if known)
Webserver software & version (if known)
PHP VersionN/A

Activities

Jmantysalo

Jmantysalo

2020-09-15 11:48

reporter   ~59809

It suffices to add

App()->getPluginManager()->dispatchEvent($oEvent);

after line 265 at the file. Currently wondering how to make a pull request in GitHub...

ollehar

ollehar

2020-09-15 15:07

administrator   ~59815

Good catch. Github has lots of documentation on how to do a pull request. Join us on telegram if you need guidance.

Jmantysalo

Jmantysalo

2020-09-16 07:45

reporter   ~59839

If I got it right this time, then the fix is at https://github.com/LimeSurvey/LimeSurvey/pull/1593 and I guess you will get a notification of a pull request after some automatic checks are done.

Telegram? More info on that is available at [please make a www-page if not already exists]?

Issue History

Date Modified Username Field Change
2020-09-11 12:17 Jmantysalo New Issue
2020-09-15 11:48 Jmantysalo Note Added: 59809
2020-09-15 15:07 ollehar Note Added: 59815
2020-09-16 07:45 Jmantysalo Note Added: 59839