DateTimePicker error debugging

[Steve Mayfield]

Try moving CDialog::OnOK() to the end of your OnOK routine...

void CDialogSd::OnOK()
{
CTime time;
m_date.GetTime(time);
day.Format("%d",time.GetDay());
month.Format("%d",time.GetMonth());
year.Format("%d",time.GetYear());
CDialog::OnOK();
}

Steve

[Jose Luis Sogorb]

Same error.

[Steve Mayfield]

I assume that you have already set your CDateTimeCtrl with a value in your dialog using the SetTime method. I use virtually the same sequence in a filtering dialog and it works perfectly. Steve

[Jose Luis Sogorb]

No, the value in the dialog appears automatically(actual date), so I thought it was not needed to init with a value. How do you do it?

[Steve Mayfield]

Is it possible that your control is a 'time only' control (check the dialog properties Styles/Format)? According to the docs, you will get an assert trying to access the date members in debug mode. Otherwise you need to set the style in the Create function. Steve

[Jose Luis Sogorb]

No, I have it setted as 'short date' style. Can you give me an example code as you have initialized it? Thanks

[Steve Mayfield]

in the View Class:
...
CDateTimeDlg dlg;
CTime timeNewDateTime;

dlg.m_DateTime = CTime::GetCurrentTime();
if(dlg.DoModal() == IDOK)
timeNewDateTime = dlg.m_DateTime;

======================
in the CDateTimeDlg class:
in the .h file:
...
public:
CDateTimeDlg(CWnd *pParent = NULL); // standard constructor
CTime m_DateTime;
// Dialog Data
//{{AFX_DATA(CDateTimeDlg)
enum { IDD = IDD_DATETIME_DLG };
CDateTimeCtrl m_cDateTime;
//}}AFX_DATA

in the .cpp file:
...
void CDateTimeDlg::DoDataExchange(CDataExchange* pDX)
{
CDialog::DoDataExchange(pDX);
//{{AFX_DATA_MAP(CDateTimeDlg)
DDX_Control(pDX, IDC_DATETIME, m_cDateTime);
//}}AFX_DATA_MAP
}

BEGIN_MESSAGE_MAP(CDateTimeDlg, CDialog)
//{{AFX_MSG_MAP(CDateTimeDlg)
ON_NOTIFY(DTN_DATETIMECHANGE, IDC_DATETIME, OnDatetimechangeDateTime)
//}}AFX_MSG_MAP
END_MESSAGE_MAP()
...
BOOL CDateTimeDlg::OnInitDialog()
{
CDialog::OnInitDialog();

m_cDateTime.SetFormat("MM/dd/yyyy HH:mm:ss");
m_cDateTime.SetTime(&m_DateTime);
return TRUE; // return TRUE unless you set the focus to a control
// EXCEPTION: OCX Property Pages should return FALSE
}

void CDateTimeDlg::OnDatetimechangeDateTime(NMHDR *pNMHDR, LRESULT *pResult)
{
m_cDateTime.GetTime(m_DateTime);
*pResult = 0;
}

Steve

[Jose Luis Sogorb]

Thank you, very much. I'll try it.

[Jose Luis Sogorb]

It works fine in debug mode, also. Thanks!!!!

[Steve Mayfield]

Happy to have helped you solve your problem. Steve