Cannot submit value of a spinbox [SOLVED]

Mr. Kibu

Hello!

If I call "mapper->submit" the data of a mapped spinbox is not written into the model, but the string of an lineedit does.

In the class "mainwindow" I connect to the database (sqlite) and create a QSqlRelationalTableModel and show the data in a QTableView. Than I have a funktion where I open the data of one row in a dialog. In the funktion where I open the dialog, I pass the model to the dialog:

void MainWindow::editBeleg()
{
     //####get the value of the key-field
    QModelIndexList indexList = ui->tabBelege->selectionModel()->selectedIndexes();
    int row;
    foreach (QModelIndex index, indexList) {
        row = index.row();
    }

   int _selBelegKey = ui->tabBelege->model()->data(ui->tabBelege->model()->index(row,0)).toInt();

    frmBeleg newBeleg(_selBelegKey,modBelege, this);// <<<<<<<<modBelege is the model to pass!!!!
    newBeleg.exec();

In the dialog I adopt the variables like this:

frmBeleg::frmBeleg(int BelegID, QSqlRelationalTableModel *BelegMod, QWidget *parent) :
    QDialog(parent),
    ui(new Ui::frmBeleg)
{

    ui->setupUi(this);

In the dialog I can edit the data. If I edit the data of the spinbox by hand and submit the data to the model, it works fine and the data is in the database.
If I set the value of a spinbox in a event (for example: ui->spinbox->setValue(4)), the data is NOT submited to the model. If I do that with a lineedit, everything works.

How can I set the data of a spinbox from a event???

Thanks for help!!

Franz

SGaist

Hi,

Are you calling setValue before you setup the mapper ?

Mr. Kibu

Yes, submit is called in a submit-function.

SGaist

It's not what I'm asking; when are you calling ui->spinBox->setValue(4) ? Before or after setting up the mapper ?

Mr. Kibu

Ok, I set the mapper in the constructor, set the spinbox in a event-function and submit the data in a submit-function. So I think that I set the mapper befor.

SGaist

What event function ?

Mr. Kibu

void frmBeleg::on_txtAnm1_editingFinished()
{
    ui->txtBelegNr->setValue(4);//<<<< this is the spinbox that not work
    ui->txtAnm2->setText("tessst");//<<<this is a combobox; after submit is the value in the database
}```

SGaist

Can you show how you setup the mapper ?

Mr. Kibu

Here is the constructor of the dialog (frmBeleg):

frmBeleg::frmBeleg(int BelegID, QSqlRelationalTableModel *BelegMod, QWidget *parent) :
    QDialog(parent),
    ui(new Ui::frmBeleg)
{
    ui->setupUi(this);

    mapper=new QDataWidgetMapper(this);
    mapper->setModel(BelegMod);
    //mapper->setSubmitPolicy(QDataWidgetMapper::ManualSubmit);
    mapper->setItemDelegate(new QSqlRelationalDelegate(mapper));

    mapper->addMapping(ui->txtBelegNr,2);//<<<<<    the spinbox
    mapper->addMapping(ui->txtAnm1,14);//<<<<<<<   the combo

    if (BelegID != -1) {
            for (int row = 0; row < BelegMod->rowCount(); ++row) {
                QSqlRecord record = BelegMod->record(row);
                if (record.value(0).toInt() == BelegID) {//Spalte des Key-Feldes
                    mapper->setCurrentIndex(row);
                    break;
                }
            }
        }
        else {
            mapper->toFirst();
            }

Mr. Kibu

Addition:

It is no problem to use the passed model from the mainwindow in the constuctor of the dialog (frmBeleg).
When I use the model (BelegMod) in a function, than I have no error at buildung the application but the programm crashes when I use the function in the programm.

For example I tried:

void frmBeleg::on_btnSave_clicked()
{
    mapper->submit();

    if (BelegMod->submitAll())
        BelegMod->database().commit();
    else
        BelegMod->database().rollback();

    frmBeleg::close();
}

Or:

void frmBeleg::on_btnNextBelegNr_clicked()
{
    QSqlQueryModel modMaxBelNr;
    modMaxBelNr.setQuery("SELECT max(BelegNr) FROM Belege");
    int nextBelegNr = modMaxBelNr.data(modMaxBelNr.index(0, 0)).toInt();
    BelegMod->setData(BelegMod->index(0,2), nextBelegNr+1);
}

How can I use the passed model in a function?

Thank's

Franz

SGaist

Are you initializing BelegMod properly in the constructor ?

Mr. Kibu

I hope that I have understand your question:

I am initializing the model "modBelege" in the function "on_actionBelege_triggered" like this:

void MainWindow::on_actionBelege_triggered()
{
    ui->stackedWidget->setCurrentIndex(0);

    modBelege = new QSqlRelationalTableModel(this, cn::db());
    modBelege->setEditStrategy(QSqlRelationalTableModel::OnFieldChange);
    modBelege->setTable("belege");

    //modBelege->setRelation(19, QSqlRelation("Zahlarten","ZahlartKey","Zahlart"));
    modBelege->setRelation(8, QSqlRelation("kontakte","AdrKey","Name"));
    modBelege->select();
    ......

Then I pass the model (modBeleg) to the dialog:

void MainWindow::editBeleg()
{
    //####search the row of the tableview "tabBelege"
    QModelIndexList indexList = ui->tabBelege->selectionModel()->selectedIndexes();
    int row;
    foreach (QModelIndex index, indexList) {
        row = index.row();
    }

    //####get Key-value in column 0

    int _selBelegKey = ui->tabBelege->model()->data(ui->tabBelege->model()->index(row,0)).toInt();

    //#### open dialog:
    frmBeleg newBeleg(_selBelegKey,modBelege, this);
    newBeleg.exec();
}

Hope it helps!

SGaist

In the frmBeleg constructor, do you assign modBelege to BelegMod ?

Mr. Kibu

I have only this code in frmBeleg:

frmBeleg::frmBeleg(int BelegID, QSqlRelationalTableModel *BelegMod, QWidget *parent) :
    QDialog(parent),
    ui(new Ui::frmBeleg)
{
    ui->setupUi(this);
    ...

SGaist

From your code it seems that you don't initialize your class member that is also named BeleMod

Mr. Kibu

And how can I do this? And is it better to rename BelegMod?

SGaist

frmBeleg::frmBeleg(int BelegID, QSqlRelationalTableModel *BelegMod, QWidget *parent) :
    QDialog(parent),
    ui(new Ui::frmBeleg),
    BelegMod(BelegMod)
{

But yes,

frmBeleg::frmBeleg(int BelegID, QSqlRelationalTableModel *model, QWidget *parent) :
    QDialog(parent),
    ui(new Ui::frmBeleg),
    BelegMod(model)
{

is clearer.

One thing I recommend: use a clear naming scheme e.g. camel case with starting letter lowercased for variables and camel case for classes. It will make your code easier to read (following Qt's pattern is good idea)

Mr. Kibu

Hi SGaist!

It works. Thank you for your help!

märssi

Franz