Conversation
…gurator-field-value-does-not-show-when-editing'.
| var sourceIdField = presideObjectService.getIdField( args.sourceObject ); | ||
| var sortOrderField = presideObjectService.getObjectAttribute( targetObject, "datamanagerSortField", "sort_order" ); | ||
| var hasSortOrder = StructKeyExists( presideObjectService.getObjectProperties( targetObject ), sortOrderField ); | ||
| var isEdit = isEmptyString( args.savedData[ "datecreated" ] ?: "" ); |
There was a problem hiding this comment.
This naming seems the wrong way around? isEdit should be = !isEmptyString( ... )? and then the logic below would be if ( isEdit || isTrue( sourceProperty.cloneable...?
I wonder should it all be simpler and just use if ( !StructIsEmpty( args.savedData ) )? i.e. if we have saved data, whether cloning or editing, then we should prepopulate?
I'm curious that this would not have been picked up previously re the bug in general. Wonder if @sebduggan has any thoughts here?
There was a problem hiding this comment.
The issue was only introduced with the previous bugfix from last month...
There was a problem hiding this comment.
I am afraid we cannot use !StructIsEmpty(args.savedData) because the cloned args.savedData is not empty, but it is missing datecreated and datemodified.
3 participants
No description provided.