SQLHelperFunctions — wrong Format masks in toISO

Open

#67 opened on Oct 16, 2025 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty: 1/5
Estimated time: Under an hour
Newbie friendliness: 55/100

Issue type: Bug
Clarity: Clearly specified
Activity status: Stale
Tech stack: sql, visualbasic
Domain: databases

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.

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)

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

Expected

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

Steps to Reproduce

Print toISO(Now) and compare minutes field.

Actual

Minutes are incorrect.

Proposed Fix

Use mm for month and nn for minutes.

Dominant language: VBA
Stars: 90
Forks: 18
PR merge metrics: No merged PRs in 30d

Contributor guide

No contributing guide indexed for this repository

First steps

Read the whole issue, then the project's contributing guide.
Comment on the issue to say you are picking it up — it saves two people doing the same work.
Fork the repository and make your change on a branch.
Open a pull request that references the issue number.

Similar issues

[Documentation Request] vLLM kv_load_failure_policy doesn't apply to load failures in L2 adapters Open

Difficulty 2/5 1-3 hours Newbie friendliness 65/100

LMCache/LMCache#5326 · Opened Sep 23, 2026
[Bug]: Event executions make the originating request panic (nil JobInsertResult in queue log middleware) when the OTLP log exporter is enabled Open

Difficulty 2/5 1-3 hours Newbie friendliness 65/100

zitadel/zitadel#12811 · Opened Sep 23, 2026
DATABASE_URL=pglite:memory: creates a directory named 'memory:' instead of an in-memory database Open

Difficulty 2/5 1-3 hours Newbie friendliness 75/100

BuilderIO/agent-native#5730 · 1 comment · Opened Sep 23, 2026
Request headers and POST/PUT bodies silently corrupted once the wasm heap passes 2 GiB (signed pointers in http_wasm.cc) Open

Difficulty 2/5 1-3 hours Newbie friendliness 65/100

duckdb/duckdb-wasm#2258 · Opened Sep 23, 2026
user-feedback: document idempotent migrations with auth-dependent triggers Open

Difficulty 2/5 1-3 hours Newbie friendliness 70/100

supabase/agent-skills#602 · Opened Sep 23, 2026

More Databases issues