- Issue created by @jrockowitz
- @jrockowitz opened merge request.
- Status changed to Needs review
over 1 year ago 7:34pm 31 May 2023 - πΊπΈUnited States apmsooner
This makes sense and simplifies. Let me do my further due diligence and i'll get it merged. Thanks for the patch!
- πΊπΈUnited States jrockowitz Brooklyn, NY
@apmsooner You could create a hidden test module that sets via config a custom field with all the available datatypes. The test module could be used for quickly getting an instance of a custom field available and may be used for automated tests.
- πΊπΈUnited States apmsooner
@jrockowitz, tested your patch and modified it as theres some differences between decimal/float around scale so elected to keep them separate. I revisited the core field types for both of these and matched the logic more closely to account for missing/non-numeric properties. Tested this out in variety of scenarios and works now as expected without error. Agree a test module will be helpful and will work toward that but for now this will get us past the error. Let me know if this works for you or have further input?
-
apmsooner β
committed bece9d64 on 1.0.x authored by
jrockowitz β
Issue #3363964 by jrockowitz, apmsooner: Warning: Undefined variable $...
-
apmsooner β
committed bece9d64 on 1.0.x authored by
jrockowitz β
- Status changed to Fixed
over 1 year ago 8:54pm 14 June 2023 - πΊπΈUnited States jrockowitz Brooklyn, NY
Thank you for improving the patch and merging it.
Automatically closed - issue fixed for 2 weeks with no activity.