View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|17754||Feature requests||Theme editor||public||2021-11-23 11:52||2021-12-21 16:12|
|Summary||17754: Inject questionAttributes to short free question twig views|
|Description||First of - in a broader view I think each question template view should have access to relevant question attributes. |
I need to tweak the numbers_only fields appearance, so questionAttributes should be accessible to short free text views.
|Tags||No tags attached.|
> First of - in a broader view I think each question template view should have access to relevant question attributes.
Yes : just add a function for this ? No ?
Else we do again and again same code …
I am quoting you from 15814 : "14228 is better , but more time to do ?" :)
what is the status of 14228?
It's just life, I need the is_numeric in short free text and it just seems more realistic to get that in a realistic time frame than realizing the whole concept of getting similar data for all questions. Surely this would be better, no question. What I did was a small addition to the current not ideal solution to have a quick result.
@tpartner seems to prefer an array … then need to update each question type ?
And here : you update qanda, but qanda is deprecated …
Did you test if it waork in 5.X : https://github.com/LimeSurvey/LimeSurvey/tree/master/application/core/QuestionTypes/ShortFreeText
With a new twig function : no need to update all question type …
Is this usefull?
In the questions theme, the array question_template_attribute is available.
That allows to pickup attributes.
Are you saying that the std attributes are not available while custom are?
Is that the issue? Maybe it is a bug instead of a Feature Request?
This is a bug.
The config.xml file for base question types are not being found as these are only searched in user themes folder and "core" themes folder.
Initially, we believe that standard question types folder should also be added as a place where to lookup for config.xml files.
Still, that may be only part of the solution, not sure if all.
@ollehar let me know if I should take this or not.
|Hm, Ahmed already closed the PR tho?|
|@gabrieljenik Seems like we have a couple of issues with higher priority right now|
|Since we won't be merging partial fix, I am closing the ticket. Already picked another report which addresses this issue in the fix.|
|2021-11-23 11:52||TonisOrmisson||New Issue|
|2021-11-23 11:52||TonisOrmisson||Status||new => assigned|
|2021-11-23 11:52||TonisOrmisson||Assigned To||=> TonisOrmisson|
|2021-11-23 11:54||TonisOrmisson||Note Added: 67496|
|2021-11-23 11:54||TonisOrmisson||Bug heat||0 => 2|
|2021-11-23 12:44||DenisChenu||Note Added: 67499|
|2021-11-23 12:44||DenisChenu||Bug heat||2 => 4|
|2021-11-23 12:45||DenisChenu||Relationship added||related to 14228|
|2021-11-23 13:10||TonisOrmisson||Note Added: 67500|
|2021-11-23 13:15||DenisChenu||Note Added: 67501|
|2021-11-29 18:41||gabrieljenik||Note Added: 67601|
|2021-11-29 18:41||gabrieljenik||File Added: image.png|
|2021-11-29 18:41||gabrieljenik||File Added: image-2.png|
|2021-11-29 18:41||gabrieljenik||Bug heat||4 => 6|
|2021-11-29 18:57||gabrieljenik||Note Added: 67603|
|2021-12-02 13:36||ollehar||Note Added: 67649|
|2021-12-02 13:36||ollehar||Bug heat||6 => 8|
|2021-12-02 14:23||ollehar||Note Added: 67654|
|2021-12-21 16:12||galads||Note Added: 67831|
|2021-12-21 16:12||galads||Bug heat||8 => 10|
|2021-12-21 16:12||galads||Status||assigned => closed|
|2021-12-21 16:12||galads||Resolution||open => won't fix|