Skip to:

  1. Skip to Jira Navigation
  2. Skip to Side Navigation
  3. Skip to Main Content

date parsing regression

Description

https://dev.lucee.org/t/is-dateadd-enforcement-of-params-more-strict-in-5-4/12765

<!--- Case 1: Add string (pass) ---> <cftry> <cfset myDate1_plus_90 = dateAdd("d", "2023-07-05", 90) /> <cfdump var="#myDate1_plus_90#" label="case 1 pass" /> <cfcatch> <cfdump var="#cfcatch#" label="case 1 fail" /> </cfcatch> </cftry>

fails in 5.4 and 6.0

https://trycf.com/gist/bd9748302dd7266fb11d98fff4650c73/lucee6-beta?theme=monokai

lucee.runtime.exp.NativeException: lucee.runtime.op.Caster.toDoubleValue(Llucee/runtime/PageContext;Ljava/lang/String;)D at main1688622923150_74f72d4c_032d_14ae_8536_f546b21379f9_cfm7020$cf$f4.call(/obscure_ram_path/main1688622923150-74f72d4c-032d-14ae-8536-f546b21379f9.cfm:5) at lucee.runtime.PageContextImpl._doInclude(PageContextImpl.java:1058)

Environment

None

Activity

Show:

Michael Offner 7 July 2023 at 09:50

we reverted https://luceeserver.atlassian.net/browse/LDEV-4506 for 5.4 but kept it for Lucee 6.

Zac Spitzer 7 July 2023 at 08:54

the argument order is wrong btw

DateAdd( datePart=string, number=number, date=datetime );

https://docs.lucee.org/reference/functions/dateadd.html

with 6.0 just #int("2023-07-05")# blows up

Michael Offner 7 July 2023 at 08:53

Zac Spitzer 6 July 2023 at 09:09

confirmed with 5.4 that https://luceeserver.atlassian.net/browse/LDEV-4506 causes the regression, 6.0 has different issues with 6.0 when reverting that fix

lucee.runtime.exp.NativeException: 'double lucee.runtime.op.Caster.toDoubleValue(lucee.runtime.PageContext, java.lang.String)'

Zac Spitzer 6 July 2023 at 07:48

Fixed

Details

Assignee

Reporter

Priority

Labels

Fix versions

New Issue warning screen

Before you create a new Issue, please post to the mailing list first https://dev.lucee.org

Once the issue has been verified, one of the Lucee team will ask you to file an issue

Parent

Sprint

Created 6 July 2023 at 07:20
Updated 11 July 2023 at 16:53
Resolved 11 July 2023 at 16:53

Flag notifications