This is an automated email from the ASF dual-hosted git repository.

dklco pushed a commit to branch master
in repository 
https://gitbox.apache.org/repos/asf/sling-org-apache-sling-app-cms.git

commit 7f8b3b41056c1026742351af4618bc1c0782496a
Author: Dan Klco <[email protected]>
AuthorDate: Mon Feb 1 23:09:43 2021 -0500

    Fixes SLING-10115 - Requiring selects and not saving default
---
 .../apps/reference/components/forms/fields/selection/selection.jsp    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/reference/src/main/resources/jcr_root/apps/reference/components/forms/fields/selection/selection.jsp
 
b/reference/src/main/resources/jcr_root/apps/reference/components/forms/fields/selection/selection.jsp
index 0ee417e..e565a4c 100644
--- 
a/reference/src/main/resources/jcr_root/apps/reference/components/forms/fields/selection/selection.jsp
+++ 
b/reference/src/main/resources/jcr_root/apps/reference/components/forms/fields/selection/selection.jsp
@@ -47,7 +47,7 @@
                                 <c:set var="selected" value="${true}" />
                             </c:when>
                         </c:choose>
-                        <input class="${formConfig.checkInputClass}" 
type="${properties.multiple ? 'checkbox' : 'radio'}" name="${properties.name}" 
id="${properties.name}-${tag.name}" value="${tag.name}" ${selected ? 
'checked="checked"' : ''} />
+                        <input class="${formConfig.checkInputClass}" 
type="${properties.multiple ? 'checkbox' : 'radio'}" name="${properties.name}" 
id="${properties.name}-${tag.name}" value="${tag.name}" ${selected ? 
'checked="checked"' : ''} ${properties.required ? 'required="required"' : ''} />
                         <label class="${formConfig.checkLabelClass}" 
for="${properties.name}-${tag.name}">
                             <sling:encode value="${tag.valueMap['jcr:title']}" 
mode="HTML" />
                         </label>
@@ -55,7 +55,7 @@
                 </c:forEach>
             </c:when>
             <c:otherwise>
-                <select id="${properties.name}" class="form-control" 
${properties.multiple ? 'multiple="multiple"' : ''} name="${properties.name}">
+                <select id="${properties.name}" class="form-control" 
${properties.multiple ? 'multiple="multiple"' : ''} ${properties.required ? 
'required="required"' : ''} name="${properties.name}">
                     <c:if test="${not empty properties.noSelection && 
!properties.muliple}">
                         <option value="">
                             <sling:encode value="${properties.noSelection}" 
mode="HTML" />

Reply via email to