AronGahagan / cpt-dev

Code repository for the ClearPlan Toolbar
https://www.ClearPlanConsulting.com
3 stars 1 forks source link

cptResourceDemand - Australian date issue #355

Closed AronGahagan closed 8 months ago

AronGahagan commented 9 months ago

Problem:

Australian client reports date format issues. Where else does this issue exist?

Solution:

TBD - need to figure out checking against regional settings vs. only the Project Setting for DateFormat.

Todo:

AronGahagan commented 9 months ago

Application.LocaleName = "en-us" ; "en-au" add a setting on the Settings form? default to user's locale? how many formats? where to change? see Windows Locales

AronGahagan commented 8 months ago

yikes:

git grep -c -E 'DateFormat|FormatDate|Format.*yy|mm\/dd\/yyyy' | awk -F: '{ sum += $2 } END { print sum }'

Admin/cptAdmin_bas.bas:1
Backbone/cptBackbone_bas.bas:1
Calendar/cptFiscal_bas.bas:1
Calendar/cptFiscal_frm.frm:3
Core/cptCore_bas.bas:2
CostRateTables/cptCostRateTables_bas.bas:1
CustomFields/cptDataDictionary_bas.bas:1
Integration/cptIMSCobraExport_bas.bas:2
Integration/cptIMSCobraExport_frm.frm:5
Integration/cptIMSCobraExport_frm.frx:1
Integration/cptIntegration_frm.frm:2
Metrics/cptDECM_bas.bas:4
Metrics/cptDECM_frm.frm:1
Metrics/cptMetrics_bas.bas:52
ResourceDemand/cptResourceDemand_bas.bas:11
Status/cptAgeDates_bas.bas:14
Status/cptAgeDates_frm.frm:1
Status/cptSmartDuration_bas.bas:1
Status/cptSmartDuration_frm.frm:1
Status/cptStatusSheetImport_bas.bas:61
Status/cptStatusSheet_bas.bas:34
Status/cptStatusSheet_frm.frm:16
Status/cptTaskHistory_bas.bas:12
Text/cptText_bas.bas:1
Trace/cptCriticalPathTools_bas.bas:4
Trace/cptCriticalPath_bas.bas:8
Trace/cptNetworkBrowser_bas.bas:13
cptSetup_bas.bas:43

git grep -c -E 'DateFormat|FormatDate|Format.*yy|mm\/dd\/yy' | awk -F: '{ sum += $2 } END { print sum }'
297
AronGahagan commented 8 months ago

I think if we eliminate all hard-coded date formats for the named vbDateTimeFormats that should do the trick: vbGeneralDate; vbLongDate; vbShortDate; vbLongTime; vbShortTime. I think we can rely on the user's Locale Settings to make the formats. Need to confirm folder path/status dates and file name assumptions, etc.