QTableView move rows and update model

[blackout69]

Hi all,
With the following code, I can move the position of the lines in my QTableView

void Gestione::show_item()
{
  model = new QSqlTableModel(this);
  model->setTable("item");
  model->setHeaderData(1, Qt::Horizontal, tr("Number"));
  model->setHeaderData(2, Qt::Horizontal, tr("Name"));
  model->setHeaderData(3, Qt::Horizontal, tr("Email"));
  if (model->select())
    {
      ui->view_relatori->setModel(model);
      ui->view_relatori->resizeColumnsToContents();
      ui->view_relatori->verticalHeader()->setSectionsMovable(true);
      ui->view_relatori->verticalHeader()->setDragDropOverwriteMode(true);
      ui->view_relatori->verticalHeader()->setDragEnabled(true);
      ui->view_relatori->verticalHeader()->setDragDropMode(QAbstractItemView::InternalMove);
    }
}

Once I have ordered the list to my liking, I need to save the new order in the database table.
I tried to read the data with a for loop, but I read from the model and not from the view

void Gestione::read_data()
{
  QStringList value_id;
  QStringList value_number;
  QVector<QStringList> current_order;
  current_order.clear();
  for(int i = 0; i < model->rowCount(); i++)
    {
      QSqlRecord record = model->record(i);
      value_id.append(record.value(0).toString());
      value_turno.append(record.value(1).toString());
      row = ui->view->verticalHeader()->visualIndex(i);
    }
  current_order.append(value_id_rel); // 00 List id_rel[]
  current_order.append(value_turno);  // 01 List Number[]
}

I searched for a solution, but found nothing.
Is there a method to save the reordered view on the model?
Thank you in advance
blackout69

[SGaist]

Hi,

Do you mean you want to be able to change the order of the database fields based on the visual changes you applied to the QTableView ?

[blackout69]

@sgaist said in QTableView move rows and update model:

Do you mean you want to be able to change the order of the database fields based on the visual changes you applied to the QTableView ?

Hi,
Yes. The number field in the table serves as a "work shift". By changing the order in the view, I would like to update the table number field based on the new order. So that by applying a filter to the number field, I can always have the correct order in the view.

blackout69

[JonB]

@blackout69
Since you are reordering via moving headers, you should perhaps read the answer in https://stackoverflow.com/questions/41807590/reading-qtableview-row-contents-with-the-same-order-as-are-seen-not-stored. Or, I think @VRonin's answer at https://forum.qt.io/topic/75393/reading-qtableview-row-contents-with-the-same-order-as-are-seen-not-stored/9 addresses your situation.

Unless I am missing something (I am interested now), it does seems surprisingly complex to read table view row/columns in visual order instead of model order?

[VRonin]

If you don't use the header to hide individual rows, you could try

QObject::connect(ui->view_relatori->verticalHeader(),&QHeaderView::sectionMoved,[=](int logicalIndex, int oldVisualIndex, int newVisualIndex)->void{
Q_ASSERT(logicalIndex==oldVisualIndex);
const QModelIndex numberIndex =model->index(1,logicalIndex); /*I use 1 here because of model->setHeaderData(1, Qt::Horizontal, tr("Number")); it looks strange, however, you have nothing at index 0*/
Q_ASSERT(logicalIndex==numberIndex.data().toInt());
model->setData(numberIndex,newVisualIndex);
});

[blackout69]

@jonb
reading based on the view would be great, but I don't know how. Could you tell me how?
blackout69

[blackout69]

@vronin

I get this when I move a row

ASSERT: "logicalIndex==numberIndex.data().toInt()" in file gestione.cpp, line 843
Invalid parameter passed to C runtime function.

/I use 1 here because of model->setHeaderData(1, Qt::Horizontal, tr("Number")); it looks strange, however, you have nothing at index 0/
My index 0 is the table id
blackout69

[VRonin]

@blackout69 said in QTableView move rows and update model:

My index 0 is the table id

could you qDebug() << logicalIndex <<"!=" << numberIndex.data().toInt(); and post the result?

[blackout69]

@VRonin

QObject::connect(ui->view_relatori->verticalHeader(),&QHeaderView::sectionMoved,[=](int logicalIndex, int oldVisualIndex, int newVisualIndex)->void{
Q_ASSERT(logicalIndex == oldVisualIndex);
const QModelIndex numberIndex = relatori_model->index(rel_id_rel,logicalIndex); /*I use 1 here because of model->setHeaderData(1, Qt::Horizontal, tr("Number")); it looks strange, however, you have nothing at index 0*/
qDebug() << logicalIndex <<"!=" << numberIndex.data().toInt();
Q_ASSERT(logicalIndex == numberIndex.data().toInt());
relatori_model->setData(numberIndex,newVisualIndex);
});

5 != 0
ASSERT: "logicalIndex == numberIndex.data().toInt()" in file gestione.cpp, line 844
Invalid parameter passed to C runtime function.
Invalid parameter passed to C runtime function.

blackout69

[VRonin]

@blackout69 said in QTableView move rows and update model:

The number field in the table serves as a "work shift"

Looks like it contains a 0 though. what is in that column

[blackout69]

@vronin said in QTableView move rows and update model:

Looks like it contains a 0 though. what is in that column

In that column there is the table id. So only integers starting from 1
The table is thus formed:
id_rel int(11)
turno decimal(3,1)
name varchar(40)
email varchar(60)

blackout69

[blackout69]

@vronin
Perfect so it works.
Thanks for your help.

      QObject::connect(ui->view_relatori->verticalHeader(), &QHeaderView::sectionMoved, [=](int logicalIndex, int oldVisualIndex, int newVisualIndex)->void{
      Q_ASSERT(logicalIndex == oldVisualIndex);
      QModelIndex numberIndex = relatori_model->index(logicalIndex, rel_turno);
      Q_ASSERT(logicalIndex == numberIndex.data().toInt() - 1);
      relatori_model->setData(numberIndex, newVisualIndex + 1);
      relatori_model->submitAll();
      });

blackout69