- Issue created by @clemorphy
- 🇬🇧United Kingdom aaron.ferris
Can confirm this, seems we fallback to the default webform settings should a 'per page' wizard next/previous label not be defined.
From a quick look, we could possibly add another condition in WebformSubmissionForm that uses the following order:
1. Per page wizard settings as a priority
2. Then Per webform settings
3. Then fallback to default settings - Merge request !506Issue #3468142: fallback to per webform next/prev labels should per page not be configured → (Open) created by aaron.ferris
- Status changed to Needs review
8 months ago 4:58pm 14 August 2024 - 🇫🇷France clemorphy Paris
Thanks a lot aaron.ferris
I have tested your MR from #3, applied on version 6.2.7, and it works perfectly for me. - Status changed to Needs work
8 months ago 1:37pm 19 August 2024 - 🇨🇦Canada Liam Morland Ontario, CA 🇨🇦
It looks like $next_button_custom is not getting set. That needs to be fixed. This also needs tests.
- 🇬🇧United Kingdom aaron.ferris
Ah yes, ive fixed that - id need more time to look at writing a test.
- 🇫🇷France clemorphy Paris
Ah yes, ive fixed that - id need more time to look at writing a test.
I confirm that the error is gone with the updated MR!506
Thank you. - 🇦🇺Australia darvanen Sydney, Australia
The button label doesn't get overridden *there*, but it does get overridden in \Drupal\webform\Element\WebformActions::processWebformActions
// Apply custom label. $has_custom_label = !empty($element[$button_name]['#webform_actions_button_custom']); if (!empty($element['#' . $settings_name . '__label']) && !$has_custom_label) { if (isset($element[$button_name]['#type']) && ($element[$button_name]['#type'] === 'link')) { $element[$button_name]['#title'] = $element['#' . $settings_name . '__label']; } else { $element[$button_name]['#value'] = $element['#' . $settings_name . '__label']; } }