Hi,
when the satellite mode is enabled for new qso and you enter a frequency in the "New QSO" form in the field "Frequency" a comma , in the number is always replaced by a point. just enter a frequency number with a comma, and the comma is asap switched to a point.
when you now enter a frequency in the RX frequency comma and point both
is possible.
comma frequency leads to a NULL in the Insert Statement to cqrlog_main.
i think the function "cmdFreqChange" is only called for the normal frequency,
but as i see, never called for the rx_frequency.
so the function "CheckFreq", which replaces comma by point is never called for the rx_frequency, which leads to a NULL in the rxfreq field in cqrlog_main.
cqrlog is 2.3.0 (104).
the workaround actual is to enter the rx frequencies always with dot .
bye Andreas, DL7OAP
HI!
There is actually a bug in subroutine that checks comma to dot.
It is not "multi purpose" and refers only to frequency field of New QSO:
function TfrmNewQSO.CheckFreq(freq : String) : String;
begin
if (Pos(',',cmbFreq.Text) > 0) then
begin
freq := cmbFreq.Text;
freq[Pos(',',cmbFreq.Text)] := FormatSettings.DecimalSeparator;
Result := freq
end;
Result := freq
end;
After fixing it to format:
function TfrmNewQSO.CheckFreq(freq : String) : String;
begin
if (Pos(',',freq) > 0) then
begin
freq[Pos(',',freq)] := FormatSettings.DecimalSeparator;
Result := freq
end;
Result := freq
end;
it can be called also from satellite rx frequency box "On exit" function:
procedure TfrmNewQSO.edtRXFreqExit(Sender: TObject);
begin
edtRXFreq.Text := CheckFreq(edtRXFreq.Text);
end;
That fixes the problem. (and will be test version 105)
--
Saku
OH1KH
Hi Saku,
did a test with version dev 105 and satellite rx_frequence is now correct stored, even when you use comma.
thank you very much :-)
Andreas, DL7OAP
Hi!
Perhaps we could expand CheckFreq so that if there is no comma or dot it would try to guess the place of dot in frequency from number of digits and comparing most significant digits to known bands.
Then it would be even better.
--
Saku
OH1KH