Changes in table with combo boxes

[VRonin]

@koahnig said in Changes in table with combo boxes:

the compiler complains about addItem use.

you can still copy the entire content of additem in that block or exploit the combo and call comboEditor->addItem instead. also, probably it's worth preventing empty (or even whitespace only) strings to be added

The blank lines are weird. you could check what is going on by inquiring m_comboProxy->rowCount()

[koahnig]

@VRonin

Code of setEditorData

{
    auto comboEditor =qobject_cast<QComboBox*>(editor);
    Q_ASSERT(comboEditor);
    const QString& textToFind = index.data(Qt::EditRole).toString();
    int itemIdx = comboEditor->findText(textToFind);
    if(itemIdx<0){
        comboEditor->addItem(textToFind);
        itemIdx  = comboEditor->findText(textToFind);
        qDebug() << "addItem " << textToFind << " " << itemIdx << " " << m_comboProxy->rowCount() << " " << m_comboProxy->columnCount();
    }
    Q_ASSERT ( itemIdx >= 0 );
    comboEditor->setCurrentIndex(itemIdx);
    qDebug() << "in combo box " << comboEditor->currentText() << " " << itemIdx << " " << m_comboProxy->rowCount() << " " << m_comboProxy->columnCount();;
}

Double click on 4, 2
Typing "Testing" <enter>
Double click on 6, 2
Typing "erf"
Double click 8, 2
Typing "Zufall"

output window:

Starting r:\build-AnotherComboBoxTest-Desktop_Qt_5_9_1_MinGW_32bit-Debug\debug\AnotherComboBoxTest.exe...
addItem  ""   0   2   1
in combo box  ""   0   2   1
in combo box  "testing"   2   3   1
in combo box  "testing"   2   3   1
in combo box  ""   0   3   1
in combo box  "erf"   1   4   1
in combo box  "erf"   1   4   1
in combo box  ""   0   4   1
in combo box  ""   0   5   1
in combo box  ""   0   5   1

That's the outcome

[VRonin]

Yep, that's what I meant with:

@VRonin said in Changes in table with combo boxes:

also, probably it's worth preventing empty (or even whitespace only) strings

void ComboBoxDelegate::setEditorData(QWidget *editor, const QModelIndex &index) const
{
    if (index.data(Qt::EditRole).isNull())
        qobject_cast<QComboBox*>(editor)->setCurrentIndex(-1);
    else
{
    auto comboEditor =qobject_cast<QComboBox*>(editor);
    Q_ASSERT(comboEditor);
    const QString& textToFind = index.data(Qt::EditRole).toString();
    //make sure you are not just searching whitespace
const QRegularExpression realTextRegExp("\S+");
if(!realTextRegExp.match(textToFind).hasMatch()){
comboEditor->setCurrentIndex(-1);
return;
}
int itemIdx = comboEditor->findText(textToFind);
    if(itemIdx<0){
        comboEditor->addItem(textToFind);
        itemIdx  = comboEditor->findText(textToFind);
        qDebug() << "addItem " << textToFind << " " << itemIdx << " " << m_comboProxy->rowCount() << " " << m_comboProxy->columnCount();
    }
    Q_ASSERT ( itemIdx >= 0 );
    comboEditor->setCurrentIndex(itemIdx);
    qDebug() << "in combo box " << comboEditor->currentText() << " " << itemIdx << " " << m_comboProxy->rowCount() << " " << m_comboProxy->columnCount();
}
}

[koahnig]

@VRonin

I am sorry to correct, but it doesn't help either.

The problem is with the index coming into setEditorData

Here is my latest code:

void ComboBoxDelegate::setEditorData(QWidget *editor, const QModelIndex &index) const
{
    if (index.data(Qt::EditRole).isNull())
        qobject_cast<QComboBox*>(editor)->setCurrentIndex(-1);
    else
    {
        auto comboEditor =qobject_cast<QComboBox*>(editor);
        Q_ASSERT(comboEditor);
        qDebug() << " index " << index;
        const QString& textToFind = index.data(Qt::EditRole).toString();
        qDebug() << "textToFind before reg exp " << textToFind;
        //make sure you are not just searching whitespace
        const QRegularExpression realTextRegExp("\\S+");
        if( ! realTextRegExp.match(textToFind).hasMatch()){
            qDebug() << "textToFind " << textToFind;
            comboEditor->setCurrentIndex(-1);
            return;
        }
        int itemIdx = comboEditor->findText(textToFind);
        if(itemIdx<0){
            comboEditor->addItem(textToFind);
            itemIdx  = comboEditor->findText(textToFind);
            qDebug() << "addItem " << textToFind << " " << itemIdx << " " << m_comboProxy->rowCount() << " " << m_comboProxy->columnCount();
        }
        Q_ASSERT ( itemIdx >= 0 );
        comboEditor->setCurrentIndex(itemIdx);
        qDebug() << "in combo box " << comboEditor->currentText() << " " << itemIdx << " " << m_comboProxy->rowCount() << " " << m_comboProxy->columnCount();
    }
}

That is the latest output:

Starting r:\build-AnotherComboBoxTest-Desktop_Qt_5_9_1_MinGW_32bit-Debug\debug\AnotherComboBoxTest.exe...
no delegate found
addItem  "test 1 1"   E:\Source\AnotherComboBoxTest\MyTableWidget.cpp 28
addItem  "test 1 2"   E:\Source\AnotherComboBoxTest\MyTableWidget.cpp 28
 index  QModelIndex(3,1,0x0,QTableModel(0x175fb6a8))
textToFind before reg exp  "testing"
in combo box  "testing"   1   2   1
 index  QModelIndex(3,1,0x0,QTableModel(0x175fb6a8))
textToFind before reg exp  "testing"
in combo box  "testing"   1   2   1
 index  QModelIndex(5,1,0x0,QTableModel(0x175fb6a8))
textToFind before reg exp  "erf"
in combo box  "erf"   0   3   1
 index  QModelIndex(5,1,0x0,QTableModel(0x175fb6a8))
textToFind before reg exp  "erf"
in combo box  "erf"   0   3   1
 index  QModelIndex(7,1,0x0,QTableModel(0x175fb6a8))
textToFind before reg exp  ""
textToFind  ""
 index  QModelIndex(7,1,0x0,QTableModel(0x175fb6a8))
textToFind before reg exp  ""
textToFind  ""

In additiion the update problems there are also the initial input for this test "test 1 1" not covered anymore after some input directly to the table.

[VRonin]

Isn't that the correct output? what were you expecting?

[koahnig]

@VRonin

Nope, I did the same as with the previous tests. Only lower case letters this time. The input was:

• Double click on 4, 2
• Typing "testing" <enter>
• Double click on 6, 2
• Typing "erf" <enter>
• Double click 8, 2
• Typing "zufall" <enter>

[koahnig]

We are back to the start :(

#define USE_COMBO_PROXY
QWidget* ComboBoxDelegate::createEditor(QWidget *parent, const QStyleOptionViewItem &, const QModelIndex &) const
{
    QComboBox* result = new QComboBox(parent);
    result->setEditable( true );
    result->setInsertPolicy( QComboBox::InsertAlphabetically );

#ifdef USE_COMBO_PROXY
//    m_comboProxy->sort( 0 );
    result->setModel(m_comboProxy);
#else
    result->setModel(m_comboModel);
#endif

    return result;
}

Commenting out m_comboProxy->sort( 0 ) or by-passing the whole proxy model do work correctly.

[VRonin]

Sorry but now I'm lost. could you describe what the expected output was?

[VRonin]

for anyone ending in this post from google, you should add m_comboProxy->setDynamicSortFilter(false); in the constructor of the delegate

[koahnig]

@VRonin

Yes, this does the trick.
After introducing the line in the constructor the test example is working.

Once again, many thanks for your help.