Thank you olle:)

How is this a bug? When you're putting "MoreThan5" as value, you're breaking the interface of the question?

You need pg because with mysql : no issue.

With "MoreThan5" : you broke "Response save" here : https://github.com/LimeSurvey/LimeSurvey/blob/master/application/helpers/expressions/em_manager_helper.php#L5424

Then submitfailed function from front_end helper is called.

This function :

• return a message for the user : then it "echo" to the user , BEFORE the html
• kill the session
• don't kill the page or die or anything

But you need an adminemail ....

The "MoreThan5" broke pgsql : it's another issue.

Is the solution a proper error message? Because I don't want pgsql to fail silently, as mysql.

I work actually in 10827 . I want to show message for some "Error" too . But don't break Survey (don't reset session).

I want to do 2 thing:

• Construct the "DB/core validation" error message in EM somewhere
• Set to NULL in DB if value have an error.

Actually i search a way to add message in _validateUestion ... to add it in EM system dynamicly . But :

• Not the good way
• Not easy
• Must be rewritten for 3.0

If you're OK : i just condtruct an error message

Example here; https://github.com/LimeSurvey/LimeSurvey/blob/master/application/helpers/expressions/em_manager_helper.php#L5401

Adding regexp for DECIMAL(30,10), don't reset the $_SESSION var
Add an "error" tag/boolean

The we mist shown the "Question page" with error somewhere. Just like it was "mandatory" or somethiug like here.

I send some of code tomorrow :

• regexp for DECIMAL
• answer in possible answer for radio/dropdown

OK ?

Oups ... another poiunt:
Even with a good filtering system DB save can broke ....
Then : have a cleanest way for this part is a must done.

@olle : see comment here : https://github.com/LimeSurvey/LimeSurvey/commit/f438b28914917dec7944291a7bad67719ceec419#diff-796aba4d06254b9caea9d461e4ea80c9R5404

For information : i try to update $this->qid2validationEqn[$qid]['eqn'] and msg here : but _validateUestion is already done ....

Then i add it before // DETECT ANY VIOLATIONS OF VALIDATION RULES // manually but really .... it's not the best way .... EM is awfull ..

Yeah, we need a system to show messages that's not "echo".

I'd say Yii flash message.

flash message : only JS ?

Else we have a place for error here : https://github.com/LimeSurvey/LimeSurvey/blob/master/application/helpers/SurveyRuntimeHelper.php#L1207

You adde an infor on stepInfo ?

http://www.yiiframework.com/wiki/21/how-to-work-with-flash-messages/

Think it generates HTML, and then do some Bootstrap enhancement. Not sure.

I want to avoid adding things in run(). Also echo is hard to get good control-flow with.

What's stepinfo?

Yes, i know flashMessage :). Sorry bad reading didn't see the Yii before.

Very good idea : and popups can be moved to view system for flashMessage.

Actually : we don't have flashMessage for public.

I like to extedn flashMessage to have different "style".
https://github.com/LimeSurvey/LimeSurvey/commit/ee7e746113f136939ad02e49981aa8ccfee8a005 ;)

Version 2.06LTS Build 160417 released
Version 2.50+ Build 160418 released