[FIXED] PropertyGridItemEnum is not case sensitve

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

Hello,

CXTPPropertyGridItemEnum is not case sensitive (tested with version 13.4.2 and 15.1.1). The following code is taken from the GridSample. I have just added some more constraints and the TRACE statements:

        CXTPPropertyGridItem* pItem = pStandard->AddChildItem(new CXTPPropertyGridItemEnum(_T("Enum item"), 2));
        pItem->GetConstraints()->AddConstraint(_T("Windows 98"), 1);
        pItem->GetConstraints()->AddConstraint(_T("Windows 2000"), 2);
        pItem->GetConstraints()->AddConstraint(_T("Windows XP"), 3);
        pItem->GetConstraints()->AddConstraint(_T("Windows xp"), 4);
        pItem->GetConstraints()->AddConstraint(_T("windows xp"), 5);
        pItem->GetConstraints()->AddConstraint(_T("windows XP"), 6);

        TRACE1("Constraint: %d\n", pItem->GetConstraints()->FindConstraint(_T("Windows XP")));
        TRACE1("Constraint: %d\n", pItem->GetConstraints()->FindConstraint(_T("Windows xp")));
        TRACE1("Constraint: %d\n", pItem->GetConstraints()->FindConstraint(_T("windows xp")));
        TRACE1("Constraint: %d\n", pItem->GetConstraints()->FindConstraint(_T("windows XP")));

All FindConstraint statements return 2 (the index of "Windows XP").
The fix for the bug seems to be simple. Just replace the "CompareNoCase" in the CXTPPropertyGridItemConstraints::FindConstraint method with "Compare". It would be nice if CodeJock can fix this in the next version.

Best regards
   Holger Farr.