Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DF default values do not work for article fields in masks based on ActionCommon. #2785

Closed
svenoe opened this issue Dec 22, 2023 · 3 comments
Closed
Labels
bug Something isn't working as intended
Milestone

Comments

@svenoe
Copy link
Contributor

svenoe commented Dec 22, 2023

Add a multiselect field with default value to AgentTicketNote and see that it is rendered empty (if empty value is given).

@svenoe svenoe added the bug Something isn't working as intended label Dec 22, 2023
@svenoe svenoe modified the milestones: OTOBO 10.1.9, OTOBO 10.0.20 Dec 22, 2023
stefanhaerter added a commit that referenced this issue Dec 28, 2023
stefanhaerter added a commit that referenced this issue Jan 11, 2024
…screens to enable usage of default values.
@stefanhaerter
Copy link
Contributor

This bug also affects article dynamic fields in the FollowUp of CustomerTicketZoom.

@stefanhaerter
Copy link
Contributor

Maybe also relevant: In the comparable place for handling the dynamic field values, AgentTicketPhone makes use of default values stored in the user preferences:

else {
# get default value from dynamic field config (if any)
$Value = $DynamicFieldConfig->{Config}->{DefaultValue} || '';
# override the value from user preferences if is set
if ( $UserPreferences{ 'UserDynamicField_' . $DynamicFieldConfig->{Name} } ) {
$Value = $UserPreferences{ 'UserDynamicField_' . $DynamicFieldConfig->{Name} };
}
}

I didn't research this thoroughly, but at least AgentTicketActionCommon, CustomerTicketZoom and a few other masks do not use this UserPreference values. Perhaps this will be relevant if we re-introduce user-modifiable sysconfigs as referenced in #2539 (comment).

svenoe pushed a commit that referenced this issue Jan 11, 2024
svenoe pushed a commit that referenced this issue Jan 11, 2024
@stefanhaerter
Copy link
Contributor

Closed with merging #2811

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working as intended
Projects
None yet
Development

No branches or pull requests

2 participants