Skip to content

Default to Yes when exporting a callsign KLog considers invalid - #1139

Merged
ea4k merged 1 commit into
ea4k:masterfrom
yl3gbc:export-invalid-call-default
Sep 10, 2026
Merged

ea4k merged 1 commit into
ea4k:masterfrom
yl3gbc:export-invalid-call-default

Conversation

@yl3gbc

@yl3gbc yl3gbc commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

showInvalidCallMessage() asks whether to export a callsign the validator rejected, with No as the default button. Since the dialog itself says the call may well be valid and wrongly flagged, defaulting to No means an accidental Enter or Escape silently drops a real QSO from the export.

The QSO stays in the log and is marked as sent, so nothing looks wrong, but it never reaches LoTW or Club Log. I found this when my log showed one QSO more than the exported ADIF.

Example: 4JT3DJ (Azerbaijan), 35,245 lookups on QRZ, QSL via Club Log
OQRS and the bureau: https://www.qrz.com/db/4JT3DJ

Defaulting to Yes keeps the warning but makes the safe answer the default one.

showInvalidCallMessage() asks whether to export a callsign the validator
rejected, with No as the default button. Since the dialog itself says the
call may well be valid and wrongly flagged, defaulting to No means an
accidental Enter or Escape silently drops a real QSO from the export.

The QSO stays in the log and is marked as sent, so nothing looks wrong,
but it never reaches LoTW or Club Log. I found this when my log showed
one QSO more than the exported ADIF.

Example: 4JT3DJ (Azerbaijan), 35,245 lookups on QRZ, QSL via Club Log
OQRS and the bureau: https://www.qrz.com/db/4JT3DJ

Defaulting to Yes keeps the warning but makes the safe answer the
default one.
@sonarqubecloud

sonarqubecloud Bot commented Sep 9, 2026

Copy link
Copy Markdown

@ea4k
ea4k merged commit 6819f98 into ea4k:master Sep 10, 2026
6 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants