fixing osx_cocoa

[wxWidgets.git] / src / msw / datectrl.cpp

diff --git a/src/msw/datectrl.cpp b/src/msw/datectrl.cpp
index 0b291ecdd41b8a4bda6fe9f3fdf4939fd1925f8a..c544e322511acb93dd7535b7f235906a190efd23 100644 (file)
--- a/src/msw/datectrl.cpp
+++ b/src/msw/datectrl.cpp
@@ -124,7 +124,7 @@ WXDWORD wxDatePickerCtrl::MSWGetStyle(long style, WXDWORD *exstyle) const
 
 wxSize wxDatePickerCtrl::DoGetBestSize() const
 {
 
 wxSize wxDatePickerCtrl::DoGetBestSize() const
 {

-    wxClientDC dc(wx_const_cast(wxDatePickerCtrl *, this));
+    wxClientDC dc(const_cast<wxDatePickerCtrl *>(this));

 
     // we can't use FormatDate() here as the CRT doesn't always use the same
     // format as the date picker control
 
     // we can't use FormatDate() here as the CRT doesn't always use the same
     // format as the date picker control


@@ -148,7 +148,7 @@ wxSize wxDatePickerCtrl::DoGetBestSize() const
         const DWORD rc = ::GetLastError();
         if ( rc != ERROR_INSUFFICIENT_BUFFER )
         {
         const DWORD rc = ::GetLastError();
         if ( rc != ERROR_INSUFFICIENT_BUFFER )
         {

-            wxLogApiError(_T("GetDateFormat"), rc);
+            wxLogApiError(wxT("GetDateFormat"), rc);

 
             // fall back on wxDateTime, what else to do?
             s = wxDateTime::Today().FormatDate();
 
             // fall back on wxDateTime, what else to do?
             s = wxDateTime::Today().FormatDate();


@@ -160,7 +160,7 @@ wxSize wxDatePickerCtrl::DoGetBestSize() const
     // representation of todays date because the control must accommodate any
     // date and while the widths of all digits are usually about the same, the
     // width of the month string varies a lot, so try to account for it
     // representation of todays date because the control must accommodate any
     // date and while the widths of all digits are usually about the same, the
     // width of the month string varies a lot, so try to account for it

-    s += _T("WW");
+    s += wxT("WW");

 
     int x, y;
     dc.GetTextExtent(s, &x, &y);
 
     int x, y;
     dc.GetTextExtent(s, &x, &y);


@@ -184,16 +184,45 @@ wxSize wxDatePickerCtrl::DoGetBestSize() const
 void wxDatePickerCtrl::SetValue(const wxDateTime& dt)
 {
     wxCHECK_RET( dt.IsValid() || HasFlag(wxDP_ALLOWNONE),
 void wxDatePickerCtrl::SetValue(const wxDateTime& dt)
 {
     wxCHECK_RET( dt.IsValid() || HasFlag(wxDP_ALLOWNONE),

-                    _T("this control requires a valid date") );
+                    wxT("this control requires a valid date") );

 
     SYSTEMTIME st;
     if ( dt.IsValid() )
 
     SYSTEMTIME st;
     if ( dt.IsValid() )

+    {
+        // Don't try setting the date if it's out of range: calendar control
+        // under XP (and presumably all the other pre-Vista Windows versions)
+        // doesn't return false from DateTime_SetSystemtime() in this case but
+        // doesn't actually change the date, so we can't update our m_date
+        // unconditionally and would need to check whether it was changed
+        // before doing it. It looks simpler to just check whether it's in
+        // range here instead.
+        //
+        // If we ever drop support for XP we could rely on the return value of
+        // DateTime_SetSystemtime() but this probably won't happen in near
+        // future.
+        wxDateTime dtStart, dtEnd;
+        GetRange(&dtStart, &dtEnd);
+        if ( (dtStart.IsValid() && dt < dtStart) ||
+                (dtEnd.IsValid() && dt > dtEnd) )
+        {
+            // Fail silently, some existing code relies on SetValue() with an
+            // out of range value simply doing nothing -- so don't.
+            return;
+        }
+

         dt.GetAsMSWSysTime(&st);
         dt.GetAsMSWSysTime(&st);

+    }
+

     if ( !DateTime_SetSystemtime(GetHwnd(),
                                  dt.IsValid() ? GDT_VALID : GDT_NONE,
                                  &st) )
     {
     if ( !DateTime_SetSystemtime(GetHwnd(),
                                  dt.IsValid() ? GDT_VALID : GDT_NONE,
                                  &st) )
     {

-        wxLogDebug(_T("DateTime_SetSystemtime() failed"));
+        // The only expected failure is when the date is out of range but we
+        // already checked for this above.
+        wxFAIL_MSG( wxT("Setting the calendar date unexpectedly failed.") );
+
+        // In any case, skip updating m_date below.
+        return;

     }
 
     // we need to keep only the date part, times don't make sense for this
     }
 
     // we need to keep only the date part, times don't make sense for this


@@ -205,18 +234,18 @@ void wxDatePickerCtrl::SetValue(const wxDateTime& dt)
 
 wxDateTime wxDatePickerCtrl::GetValue() const
 {
 
 wxDateTime wxDatePickerCtrl::GetValue() const
 {

-#ifdef __WXDEBUG__
+#if wxDEBUG_LEVEL

     wxDateTime dt;
     SYSTEMTIME st;
     if ( DateTime_GetSystemtime(GetHwnd(), &st) == GDT_VALID )
     {
     wxDateTime dt;
     SYSTEMTIME st;
     if ( DateTime_GetSystemtime(GetHwnd(), &st) == GDT_VALID )
     {

-        dt.SetFromMSWSysTime(st);
+        dt.SetFromMSWSysDate(st);

     }
 
     wxASSERT_MSG( m_date.IsValid() == dt.IsValid() &&
                     (!dt.IsValid() || dt == m_date),
     }
 
     wxASSERT_MSG( m_date.IsValid() == dt.IsValid() &&
                     (!dt.IsValid() || dt == m_date),

-                  _T("bug in wxDatePickerCtrl: m_date not in sync") );
-#endif // __WXDEBUG__
+                  wxT("bug in wxDatePickerCtrl: m_date not in sync") );
+#endif // wxDEBUG_LEVEL

 
     return m_date;
 }
 
     return m_date;
 }


@@ -240,7 +269,7 @@ void wxDatePickerCtrl::SetRange(const wxDateTime& dt1, const wxDateTime& dt2)
 
     if ( !DateTime_SetRange(GetHwnd(), flags, st) )
     {
 
     if ( !DateTime_SetRange(GetHwnd(), flags, st) )
     {

-        wxLogDebug(_T("DateTime_SetRange() failed"));
+        wxLogDebug(wxT("DateTime_SetRange() failed"));

     }
 }
 
     }
 }
 


@@ -252,7 +281,7 @@ bool wxDatePickerCtrl::GetRange(wxDateTime *dt1, wxDateTime *dt2) const
     if ( dt1 )
     {
         if ( flags & GDTR_MIN )
     if ( dt1 )
     {
         if ( flags & GDTR_MIN )

-            dt1->SetFromMSWSysTime(st[0]);
+            dt1->SetFromMSWSysDate(st[0]);

         else
             *dt1 = wxDefaultDateTime;
     }
         else
             *dt1 = wxDefaultDateTime;
     }


@@ -260,7 +289,7 @@ bool wxDatePickerCtrl::GetRange(wxDateTime *dt1, wxDateTime *dt2) const
     if ( dt2 )
     {
         if ( flags & GDTR_MAX )
     if ( dt2 )
     {
         if ( flags & GDTR_MAX )

-            dt2->SetFromMSWSysTime(st[1]);
+            dt2->SetFromMSWSysDate(st[1]);

         else
             *dt2 = wxDefaultDateTime;
     }
         else
             *dt2 = wxDefaultDateTime;
     }


@@ -283,7 +312,7 @@ wxDatePickerCtrl::MSWOnNotify(int idCtrl, WXLPARAM lParam, WXLPARAM *result)
             NMDATETIMECHANGE *dtch = (NMDATETIMECHANGE *)hdr;
             wxDateTime dt;
             if ( dtch->dwFlags == GDT_VALID )
             NMDATETIMECHANGE *dtch = (NMDATETIMECHANGE *)hdr;
             wxDateTime dt;
             if ( dtch->dwFlags == GDT_VALID )

-                dt.SetFromMSWSysTime(dtch->st);
+                dt.SetFromMSWSysDate(dtch->st);

 
             // filter out duplicate DTN_DATETIMECHANGE events which the native
             // control sends us when using wxDP_DROPDOWN style
 
             // filter out duplicate DTN_DATETIMECHANGE events which the native
             // control sends us when using wxDP_DROPDOWN style