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

https://github.com/LimeSurvey/LimeSurvey/commit/942d0f20d0a006c229e92416e676dbb77fe67d66

Hi Denis.. Thanx...
But I miss the setting of aOptions in the 3.x branche now, but maybe this is not a problem in 3.x
$this->setOptions($this->gsid);

https://github.com/LimeSurvey/LimeSurvey/blob/3.x-LTS/application/models/Survey.php

I never try …

$this->getEvent()->set('gsid',1);

The way to set with event

in 4.x the survey-object contains the array 'aOptions' and 'oOptions
in 3.x the survey-object contains only 'oOptions'

I think it has something to do with Twig (twig likes array instead of object?)

We use 4.x so both are there (luxury)

It's NOT to update options, is to update property
https://github.com/LimeSurvey/LimeSurvey/blob/942d0f20d0a006c229e92416e676dbb77fe67d66/application/models/Survey.php#L26

in 4.x the survey-object contains the array 'aOptions' and 'oOptions

Where what line ? Really don't think in afterFind …

https://github.com/LimeSurvey/LimeSurvey/commit/942d0f20d0a006c229e92416e676dbb77fe67d66 this is the 4.x fix/feature today
In this one is a
$this->setOptions($this->gsid);

https://github.com/LimeSurvey/LimeSurvey/commit/408683da06de984ecf3fa360b0779bd75e36fab9 This is the 3.x fix/feature couple of month ago
there is not suche a line

And I noticed when left out,, aOptions is not set! But this happens in 4.x (confusing!)

So the question is: why $this->setOptions($this->gsid); in 4.x and not in 3.x?

$this->setAttribute in both .

Never seen setOptions

hard to explain:
I have made 2 screenshots
Both from github... the difference is better to see!

lime4x = https://github.com/LimeSurvey/LimeSurvey/blob/942d0f20d0a006c229e92416e676dbb77fe67d66/application/models/Survey.php
lime3x = https://github.com/Shnoulle/LimeSurvey-Shnoulle/blob/d3cf0a81fab469f11ed660f1c1fd898fd42fa623/application/models/Survey.php

Mabey there is nothing wrong.. but there is a difference!

lime3x.png (50,026 bytes)   

lime3x.png (50,026 bytes)   

lime4x.png (58,428 bytes)   

lime4x.png (58,428 bytes)   

LimeSurvey 4 have inherited value, not 3 :)

I never use setOptions …
https://github.com/LimeSurvey/LimeSurvey/commit/07c3a949afe8aacbd65869cb438f086c28f9e54b

See with @dominikvitt …

the event-object contains 'surveyid'

I also found this in beforeRegister event (although that is a realy old one and the manual states surveyid correctly)

All other event-objects contain: 'surveyId'

So maybe change surveyid to surveyId ?

Both in 3.x and 4.x

I think i already reported this issue :)

There are one event receiving survey for surveyId …

Fixed in Release 4.3.16+200915