Attachment 157527 Details for Bug 34497 – Bug 34497: Fix required text field

[patch] Bug 34497: Fix required text field

Bug-34497-Fix-required-text-field.patch (text/plain), 1.55 KB, created by Pedro Amorim on 2023-10-20 14:35:58 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Pedro Amorim

Created: 2023-10-20 14:35:58 UTC

Size: 1.55 KB

patch

obsolete

>From 0f196970da9244be4706a9031e3c836230470fe9 Mon Sep 17 00:00:00 2001
>From: Pedro Amorim <pedro.amorim@ptfs-europe.com>
>Date: Fri, 20 Oct 2023 14:35:26 +0000
>Subject: [PATCH] Bug 34497: Fix required text field
>
>input.value can be either null (if flatpickr date field empty) or '' (if simple text input empty),
>so check for !input.value instead to account for both situations.
>Add missing 'required' attribute to text inputs.
>---
> koha-tmpl/intranet-tmpl/prog/js/vue/components/Dialog.vue | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/js/vue/components/Dialog.vue b/koha-tmpl/intranet-tmpl/prog/js/vue/components/Dialog.vue
>index 452038cd6d..b40436fd51 100644
>--- a/koha-tmpl/intranet-tmpl/prog/js/vue/components/Dialog.vue
>+++ b/koha-tmpl/intranet-tmpl/prog/js/vue/components/Dialog.vue
>@@ -38,6 +38,7 @@
>                                 v-if="input.type == 'Text'"
>                                 :id="`confirmation_input_${input.id}`"
>                                 v-model="input.value"
>+                                :required="input.required"
>                             />
>                         </div>
>                     </div>
>@@ -87,7 +88,7 @@ export default {
>             if (
>                 this.confirmation.inputs &&
>                 this.confirmation.inputs.filter(
>-                    input => input.required && input.value == null
>+                    input => input.required && !input.value
>                 ).length
>             ) {
>                 this.$refs.confirmationform.reportValidity()
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 34497: 154319 | 154329 | 154332 | 154333 | 157519 | 157520 | 157521 | 157522 | 157527 | 157530 | 157531 | 157532