this is my test case, seems working well, but not ...
# luceeg
gsr
11/23/2022, 6:55 PMthis is my test case, seems working well, but not sure why in my code it is displaying the 1 instead of 01
https://trycf.com/gist/d9ff9aa9ef685b342883747d52bd581c/acf2021?theme=monokai
r
Rodney
11/23/2022, 7:03 PMThe DateFormat is only applying to the FromDate, and only if the FromDate doesn't exist, and not the StartDate. The code looks overly complicated. What are you trying to accomplish?
g
gsr
11/23/2022, 7:16 PMi am trying to extract the values using the mid and the createdate, i am not sure if it can be simplified but i am ok with it if it can be
m
Matt Jones
11/23/2022, 7:21 PMmid is not a good way to work with dates. year() month() day() or dateFormat() would be better choices
Matt Jones
11/23/2022, 7:24 PMalso, you could use parseDateTime() over the createDate() in your scenario also
Matt Jones
11/23/2022, 7:30 PM<cfdump var="#year( parseDateTime( startDate ) )#"><cfdump var="#parseDateTime( startDate ).year()#">g
gsr
11/23/2022, 7:31 PMchanged my gist exactly as i have in code
gsr
11/23/2022, 7:31 PMgsr
11/23/2022, 7:37 PMcan i have the code in your way you would write, its a legacy codeand i am not a fan of mid too, i rarely use it
r
Rodney
11/23/2022, 7:40 PMThere is no need to
<cfset StartDateIn = createdate(mid(FromDate,7,4),mid(FromDate,1,2),mid(FromDate,4,2))><cfdump var="#Year(FromDate)#>Rodney
11/23/2022, 7:45 PMWell, it's a string representation of a date and if you want an actual date object, use parseDateTime() as Matt suggested.
Rodney
11/23/2022, 7:48 PM<cfset fromDate = lsParseDateTime(fromDate)><cfdump var="#fromDate#"><cfdump var="#year(fromDate)#">g
gavinbaumanis
11/25/2022, 10:08 AMThe default is not necessarily what you are using though.
If the argument is passed in - it may well not be in dd/mm/yyyy format.
Check your form / arguments / api - whatever it is that the cfparam is the "default" for / ensuring the source is validating the date/time in the expected format.
g
gsr
11/25/2022, 1:51 PMThe form is passing the 1/01/2022
How can I fix when it reaches this page should I force the mask using whatever format it expects again so it ends up in better date format where that function already written should be able to handle it
gsr
11/25/2022, 1:52 PMI know it’s patching but full context is not known to me also so trying to make it work even some invalid data is passed
m
Matt Jones
11/25/2022, 4:06 PMparseDateTime() handles your concern, it converts a string into a date object, and doesn't require the data it receives to be in a strict pattern, then you can use the functions to grab the part you want. i would try catch around the parseDateTime portion, and if you get to the catch, just set to now() or whatever your default is.