Beakerboy / Beakerboy/VBA-SQL-Library

SQLHelperFunctions — wrong Format masks in toISO

Open
#67 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
VBA
Stars
90
Forks
18
PR merge metrics
No merged PRs in 30d

Description

SQLHelperFunctions: correct toISO Format masks (mm vs nn)
- Component: `src/Modules/SQLHelperFunctions.bas` (top)

### Problem
- Uses month/minute masks incorrectly; minutes display as month.

### Snippet (actual)
```vb
Public Function toISO(dt) As String
toISO = Format(dt, "YYYY-MM-DD") & "T" & Format(dt, "HH:MM:SS")
End Function
```

### Expected
```vb
Public Function toISO(dt) As String
toISO = Format(dt, "yyyy-mm-dd") & "T" & Format(dt, "hh:nn:ss")
End Function
```

### Steps to Reproduce
1) Print `toISO(Now)` and compare minutes field.

### Actual
- Minutes are incorrect.

### Proposed Fix
- Use `mm` for month and `nn` for minutes.

---

Contributor guide

No contributing guide indexed for this repository

Research direction

Open src/Modules/SQLHelperFunctions.bas and inspect the toISO function at the top. Reproduce the issue with Print toISO(Now), then verify the output's month and minute fields use the expected masks and match ISO-style output.

Written by the indexing model from the issue text.

Assessment

Tech stack
sql, visualbasic
Domain
databases
Issue type
Bug
Difficulty
1/5
Estimated time
Under an hour
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.