CXTPPropertyGridItemDate doubt

Message Topic Search Topic Options Post Reply Create New Topic Printable Version Translate Topic

   XTP 13.1.

I don't like the initialization of "m_strFormat".
void CXTPPropertyGridItemDate::Init(const COleDateTime& oleDate)
{
    m_nFlags = xtpGridItemHasComboButton | xtpGridItemHasEdit;
    m_strFormat = _T("%d/%m/%Y");
}

That's not the desired format on many platforms! Instead, you usually want to format like this:
date.Format(VAR_DATEVALUEONLY)

This is not possible with your current implementation. I suggest you make this the default.

Author	Message Topic Search Topic Options Post Reply Create New Topic Printable Version Translate Topic
znakeeye Members Profile Send Private Message Find Members Posts Add to Buddy List Senior Member Joined: 26 July 2006 Status: Offline Points: 1672	Post Options Post Reply Quote znakeeye Report Post Thanks(0) Quote Reply Topic: CXTPPropertyGridItemDate doubt Posted: 05 August 2009 at 9:37am
	XTP 13.1. I don't like the initialization of "m_strFormat". void CXTPPropertyGridItemDate::Init(const COleDateTime& oleDate) { m_nFlags = xtpGridItemHasComboButton \| xtpGridItemHasEdit; m_strFormat = _T("%d/%m/%Y"); } That's not the desired format on many platforms! Instead, you usually want to format like this: date.Format(VAR_DATEVALUEONLY) This is not possible with your current implementation. I suggest you make this the default.
	PokerMemento - http://www.pokermemento.com/

znakeeye Members Profile Send Private Message Find Members Posts Add to Buddy List Senior Member Joined: 26 July 2006 Status: Offline Points: 1672	Post Options Post Reply Quote znakeeye Report Post Thanks(0) Quote Reply Posted: 11 August 2009 at 6:54am
	It's a bug! return oleDate.Format(); should be return oleDate.Format(VAR_DATEVALUEONLY); Otherwise the formatting is erroneous. E.g. with my Swedish setting the date should be formatted as YYYY-mm-dd, but with your code I keep getting YYYY/mm/dd.
	PokerMemento - http://www.pokermemento.com/