Skip to content

Apply station profile to WSJTX contacts#978

Closed
aa5sh wants to merge 1 commit intofoldynl:masterfrom
aa5sh:WSJT-StationProfile
Closed

Apply station profile to WSJTX contacts#978
aa5sh wants to merge 1 commit intofoldynl:masterfrom
aa5sh:WSJT-StationProfile

Conversation

@aa5sh
Copy link
Copy Markdown

@aa5sh aa5sh commented Mar 23, 2026

This will fill in missing Station Profile information on contacts originating from the WSJTX UDP Receiver. This allows for before information logging of missing details and for station profile filtering. It will not overwrite information but update any blank fields if the information is in the current Station Profile in QLog.

Populate QSqlRecord fields from the current StationProfile when receiving WSJTX UDP contacts. Adds applyStationProfile() (declaration and implementation) and calls it from both raw and ADIF insertContact paths, plus includes StationProfile.h. Also fixes StationProfile::getContactInnerJoin to use the correct "my_sota_ref" field name. This ensures missing station metadata (operator, QTH, awards, zones, DXCC, etc.) is filled from the active profile for correct filtering and display.
@foldynl
Copy link
Copy Markdown
Owner

foldynl commented Apr 6, 2026

@aa5sh
Now I’m not sure, but I just tested it and it works without the patch. It should be handled by the function NewContactWidget::addAddlFields, which is called from NewContactWidget::saveExternalContact, and that is invoked when logging all QSOs from external sources, including WSJTX.

Under what circumstances was it not being populated without this patch?

@aa5sh
Copy link
Copy Markdown
Author

aa5sh commented Apr 6, 2026 via email

@aa5sh
Copy link
Copy Markdown
Author

aa5sh commented Apr 6, 2026

Ok I'll close this one out. I know I tested several times a few weeks ago and there was inconsistencies in matching the station profile for uploads. But seems to work ok now. If I determine the cause I will revisit.

@aa5sh aa5sh closed this Apr 6, 2026
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