Still no way to add an attribute in core Question ?
Still no way to have a replacer (or fix current plugin settings).

on last git version.

My suggestion: Add the same event.

Not sure if I got the bug report right but being able to add your own question attributes by a plugin is essential for plugin development!

@Mazi : I think you understand the issue here : no way to have Custom Question attributes by plugins in current develop :).

You can add QuestionAttribute : but it don't show any input in admin GUI part.

Looks like there's no "single point of entry" to fetch question attributes anymore. Sucks ass.

Found it. Event needs to be moved to getQuestionThemeAttributeValues in questionHelper.

Selection_910.png (12,225 bytes)   

Selection_910.png (12,225 bytes)   

Re-added the event, but needs more testing and fixing (attributes are not saved, etc).

Your plugin needs to be updated to LS4 requirements to display advanced options properly. Would be nice to have it documented... E.g., the switch input type needs option array.

No default on switch ?
Easy to fix :)

I look at it tomorrow : you can assign it to me for test :)

I guess switch can be sometimes Y/N, sometimes 1/0, and there's no point doing a default then.

In 3.X : seems there are a default 0/1 .

But i can send it in 4.X, not a problem (maybe need to update manaul …)

RE "Your plugin needs to be updated to LS4 requirements ": I am not sure about the details, but shouldn't plugins be compatible between v3.x and 4.x?

Is there some documentation for plugin developers regarding "LS4 requirements"?

shouldn't plugins be compatible between v3.x and 4.x?

API update, then unsure. But best is to have manual about this :)

Too many functions, code not clear:

getQuestionAttributesForEM
getQuestionAttributeValue
getQuestionAttributeData (ajax controller method)
getQuestionAttributes
getQuestionAttributesSettings
getQuestionAttribute
getQuestionAttributes
getQuestionThemeAttributeValues
ajaxquestionattributes (not used??)

We need:

A single point of entry to get the STRUCTURE of all attributes (general, advanced, XML, plugin), and another single point of entry to get the VALUE of all attributes.

I will put this task in backlog, and Patrick will have a look to see if he can clean it later.

Arg … unable to check
Starting by uncaught exception: Object and finally ReferenceError: LS is not defined develop:60:12

Hm, yeah, but those JS errors didn't affect the display of the extra attributes when I tested. But don't test now, Patric has to clean the code anyway.

Yes, maybe when you tested, but here : i can not load it. let me deactuvate my broken plugin maybe (and check with debug=0)

Fixed in Release 4.0.0-RC7+191111

In fact not fixed, just in prototype state.

1. Current situation :Broken with debug>1 if there are a lack of an option : must have $eventAttributes[$name] = array_merge($defaultQuestionAttributeValues, $attribute); somewhere (question XML have it)
2. Must send [options][option][[value=>0][text=>"Text"]] , but seems Question user theme xml can send array directmy (think it' sthe best option
3. Value not saved (even for textarea)

For 2 :

        $scriptAttributes['scriptActivate']['options']= array(
            0=>gT("No"),
             1=>gT("Yes"),
        );

is broken, need

        $scriptAttributes['scriptActivate']['options']= array(
          'option'=> array(
            array('value'=>0, 'text'=> gT("No")),
            array('value'=>1, 'text'=> gT("Yes")),
          ),
        );

But seems question user theme XML is OK with

            <options>
                <default>Default</default>
                <lg>Large</lg>
                <sm>Small</sm>
                <xs>Extra small</xs>
            </options>

https://github.com/LimeSurvey/LimeSurvey/blob/93f3d162d9751090f2e518c080271ab8801dcbdc/themes/question/bootstrap_buttons/survey/questions/answer/listradio/config.xml#L56-L61

@cdorin : someone work globally on question attribute management ?
Else : i do a quick fix to have something … (create new function to call it just before reading/save question attribute )

@DeniChenu, it's on Patrick's list, but he didn't get time to work on it yet. We have a refactor sprint first (to reduce complexity, was the goal).

You can do a quick fix if you want, if it's not too much hassle.

to reduce complexity, was the goal

Ouf … …

https://github.com/LimeSurvey/LimeSurvey/pull/1376/files

@ollehar and @p_teichmann , any thoughts on the last PR from Denis?

Yeah. I feel like I want to rip the entire system out with regard to question attributes and start from scratch. Design from TOP which domain objects we have and how they correspond to active record classes etc. I'm not sure what happened - the code was clear in LS3, and now complex and super-hard to follow in LS4. We need to take a step back. It's not acceptable that NEW code is more complex than OLD. If we keep going in that direction, LS will die, because no one will be able to understand it.

E.g., what's the difference between a question attribute in the database, and a question attribute in a question theme XMl file? Shouldn't they correspond to two different classes?

Looks good so far i ll test it locally again. In the old event it was allways applying the question attribute to all question types. Should there also be an option to assign it to a specific questiontype or theme?

@ollehar : yes, but here : i just add a simple function … and use it when seems needed.

For information : without this event :

• pdfReport broken https://demo.sondages.pro/index.php/476956?lang=en# (check forum : seems used a lot)
• questionInsideSurvey : https://demo.sondages.me/217682?token=tokenTest&newtest=Y : used by my major client …
• A lot of another (Think Garbriel or @Mazi can give some other)

It's better if it's broken than accepting we have code no one can understand. Just don't upgrade yet! I can take time next week to try to clean it

Just don't upgrade yet! I can take time next week to try to clean it

OK :) week or nexwt month is OK. I'm afraid of more …

what's the difference between a question attribute in the database, and a question attribute in a question theme XMl file?

In the database : it's ONLY the value saved , in QuestionAttribute model.

XML file (or plugin) give the «definition» : utf8 or not, options list, default values etc …

In the old event it was allways applying the question attribute to all question types. Should there also be an option to assign it to a specific questiontype or theme?

Before we read it one time at start for ALL question type. Now we read X times for each question type. Unsure of the better solution.

My opinion : we must save in a specific Cache the definitions of the QuestionAttribute.

There are other mantis related to this, becaus ein my opinion : all QuestionAttribute MUST not come from QuestionTheme …

For example

• CSS Class : it's clearly a Survey theme related attribute
• Subquestion or Question Logic : seems related to core for me
• Timer attribute : my opinion : muts be a plugin (or another solution) but clearly not a QuestionTheme attribute.

OK, I'll create a new manual page we can use as a workboard for a new class design.

@ollehar, you should consider if it is worth the effort to completely re-build that part now for LS4 if we have to completely re-buils lots of code for LS5 anyway!

@Mazi, LS5 is not going to happen soon, I sincerely hope. Yeah, we're doing some refactoring right now, but the goal is to not break anything. Reduce complexity and weird stuff, keep same functionality.

https://manual.limesurvey.org/Question_attributes_redesign

Fixed in Release 4.0.0-RC7+191111

Why close this issue ?

There are NO WORKING WAY to add question attribute in 4.X

Yeah, this should not have been closed. Guess we had a related commit and it was closed by our bot. :|

Seems saved OK :+1:

But :
In beforeQuestionRender

$aAttributes = QuestionAttribute::model()->getQuestionAttributes($oEvent->get('qid')); didn't have javascript (empty or not)

We can use QuestionAttribute::model()->find('qid = :qid and attribute = attribute', ['qid'=> $oEvent->get('qid')),'attribute' => javascript]);

Maybe open another issue for QuestionAttribute::model()->getQuestionAttributes function difference ?

Hey @DenisChenu, about the last issue - could you please open another ticket about it ? So that we mark this one closed since is old and we forgot about it

https://bugs.limesurvey.org/view.php?id=16669

fixed in 4.4.0 rc1