Skip to content

Commit 112dcd0

Browse files
committed
Merge branch 'stable' into feature-PRESIDECMS-2749_email-stats-and-logging-performance
2 parents 2b72eff + d2987d2 commit 112dcd0

File tree

4 files changed

+11
-7
lines changed

4 files changed

+11
-7
lines changed

system/coldboxModifications/RequestContextDecorator.cfc

+7-5
Original file line numberDiff line numberDiff line change
@@ -765,13 +765,15 @@ component accessors=true extends="preside.system.coldboxModifications.RequestCon
765765

766766
clearBreadCrumbs();
767767

768-
if ( !IsNull( arguments.parentpage ?: NullValue() ) && arguments.parentPage.recordCount ) {
769-
page.parent_page = arguments.parentPage.id;
768+
announceInterception( "preInitializeDummyPresideSiteTreePage", { page = page } );
770769

771-
var ancestors = sitetreeSvc.getAncestors( id = arguments.parentPage.id );
770+
if ( !IsNull( page.parentpage ?: NullValue() ) && page.parentPage.recordCount ) {
771+
page.parent_page = page.parentPage.id;
772+
773+
var ancestors = sitetreeSvc.getAncestors( id = page.parentPage.id );
772774

773775
page.ancestorList = ancestors.recordCount ? ValueList( ancestors.id ) : "";
774-
page.ancestorList = ListAppend( page.ancestorList, arguments.parentPage.id );
776+
page.ancestorList = ListAppend( page.ancestorList, page.parentPage.id );
775777

776778
page.permissionContext = [];
777779
for( var i=ListLen( page.ancestorList ); i > 0; i-- ){
@@ -783,7 +785,7 @@ component accessors=true extends="preside.system.coldboxModifications.RequestCon
783785
page.ancestors.append( ancestor );
784786
}
785787

786-
for( var p in arguments.parentPage ){
788+
for( var p in page.parentPage ){
787789
addBreadCrumb( title=p.title, link=buildLink( page=p.id ), menuTitle=p.navigation_title ?: "" );
788790
page.ancestors.append( p );
789791
}

system/forms/formbuilder/item-types/select.xml

+2-1
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,8 @@
22
<form i18nBaseUri="formbuilder.item-types.select:">
33
<tab id="default">
44
<fieldset id="default">
5-
<field name="multiple" control="yesnoswitch" required="false" sortorder="32" />
5+
<field name="multiple" control="yesNoSwitch" required="false" sortorder="32" />
6+
<field name="includeEmptyOption" control="yesNoSwitch" required="false" sortorder="33" />
67
</fieldset>
78
</tab>
89
<tab id="values">

system/handlers/formbuilder/item-types/Select.cfc

+1-1
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ component {
1010
formControlArgs.layout = "";
1111
formControlArgs.required = IsTrue( args.mandatory ?: "" );
1212
formControlArgs.multiple = IsTrue( args.multiple ?: "" );
13+
formControlArgs.includeEmptyOption = IsTrue( args.includeEmptyOption ?: "" );
1314
formControlArgs.class = "form-control";
1415
formControlArgs.removeObjectPickerClass = true;
1516

@@ -27,7 +28,6 @@ component {
2728
return renderFormControl( argumentCollection = formControlArgs );
2829
}
2930

30-
3131
private string function renderResponse( event, rc, prc, args={} ) {
3232
return ArrayToList( _renderResponses( argumentCollection=arguments, useLabel=true ), args.delim ?: "<br>" );
3333
}

system/i18n/formbuilder/item-types/select.properties

+1
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ title=Select Box
22
iconclass=fa-list-alt
33

44
field.multiple.title=Allow multiple choice?
5+
field.includeEmptyOption.title=Include empty option?
56

67
tab.values.title=Values / auto population
78
tab.values.description=Choose either a Data manager object as the source of the values, or manually enter values in the Values and Labels inputs.

0 commit comments

Comments
 (0)