QRunIO/qqq
2
0
Fork 0
mirror of https://github.com/Kingsrook/qqq.git synced 2025-07-18 13:10:44 +00:00
Code Issues Packages Projects Releases Wiki Activity

Moved getValueAsString call from last commit into earlier check for "%s"

Browse Source
This commit is contained in:
darin.kelkhoff
2023-10-16 08:59:36 -05:00
parent caba27448d
commit b1c287a4e2
1 changed files with 6 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
qqq-backend-core/src/main/java/com/kingsrook/qqq/backend/core/actions/values/QValueFormatter.java
View File

@ -112,6 +112,12 @@ public class QValueFormatter
{ {
return formatLocalTime(lt); return formatLocalTime(lt);
} }
//////////////////////////////////////////////////////////////////////////////////////////
// else, just return the value as a string, rather than going through String.formatted //
// this saves some overhead incurred by String.formatted when called millions of times. //
//////////////////////////////////////////////////////////////////////////////////////////
return (ValueUtils.getValueAsString(value));
} }
//////////////////////////////////////////////////////// ////////////////////////////////////////////////////////
@ -119,15 +125,6 @@ public class QValueFormatter
//////////////////////////////////////////////////////// ////////////////////////////////////////////////////////
if(StringUtils.hasContent(displayFormat)) if(StringUtils.hasContent(displayFormat))
{ {
//////////////////////////////////////////////////////////////////////////////////////////
// if the format is %s (the default), just return value a string //
// this saves some overhead incurred by String.formatted when called millions of times. //
//////////////////////////////////////////////////////////////////////////////////////////
if(displayFormat.equals("%s"))
{
return (ValueUtils.getValueAsString(value));
}
try try
{ {
return (displayFormat.formatted(value)); return (displayFormat.formatted(value));