[PyQT] QDataWidgetMapper and Custom Widget

[Vincent Le Saux]

Could you be more precise or ideally write the piece of code you mentionned because I do not see what you mean.

Cheers,

Vincent

[anda_skoa]

I haven't done any PyQt programming myself, but maybe something like

self.dataMapper.addMapping(self.colorWidget,  1, "color")
self.colorWidget.sigDataChanged.connect(self.dataMapper.submit);

self.dataMapper.addMapping(self.colorWidget,  1, "color")
self.colorWidget.sigDataChanged.connect(self.dataMapper.submit);

To copy to clipboard, switch view to plain text mode 

Cheers,
_

[Vincent Le Saux]

I did not manage to make it working properly. By changing the policy of the dataMapper to Manual, I get plenty of errors. But I may not use the method correctly. I just did something like:

self.dataMapper.submit()

self.dataMapper.submit()

To copy to clipboard, switch view to plain text mode 

but it is not working.

It tried few other things and I noticed something which appear very strange to me. When I change the color attribute of my colorWidget with the QColorDialog, I then write the name of the color in the QLineEdit and modify the icon of the QPushButton. If I change the selected object within my treeView, the change is not stored in the model (we already knew that). However, if I set the focus on the QLineEdit and I press manually enter (to force the widget to emit the returnPressed signal), then the color name is stored into the model (the setData method is called). I tried to reproduce programmaticaly this behaviour by changing the focus on the QLineEdit and by emitting manually the returnPressed signal, but it does not solve my problem. I still have to interact with my widget to force the setData method to be called. Strange isn't it? I start to feel really dumb. This problem should have a simple solution, but I don't find it.

In attachment the last version of the example (it may be clearer for you to directly test the code).

Any help would be highly welcomed!

Thanks in advance for your time.

Cheers,

Vincent

[anda_skoa]

So you connect to a local slot and that one calls submit(), should work as well.

Is the slot being invoked?

it definitely works in C++. I have a color editing widget and connect its signal to the datawidgetmapper's submit() slot and whenever I change color, the model is update accordingly.

#ifndef COLOREDITOR_H
#define COLOREDITOR_H
 
#include <QColor>
#include <QWidget>
 
class QLineEdit;
 
class ColorEditor : public QWidget
{
    Q_OBJECT
    Q_PROPERTY(QColor color READ color WRITE setColor NOTIFY colorChanged)
 
public:
    explicit ColorEditor(QWidget *parent = 0);
 
    QColor color() const;
 
signals:
    void colorChanged(const QColor &color);
 
public slots:
    void setColor(const QColor &color);
 
private:
    QColor m_color;
    QLineEdit *m_lineEdit;
 
private slots:
    void onButtonClicked();
};
 
#endif // COLOREDITOR_H

#ifndef COLOREDITOR_H
#define COLOREDITOR_H

#include <QColor>
#include <QWidget>

class QLineEdit;

class ColorEditor : public QWidget
{
    Q_OBJECT
    Q_PROPERTY(QColor color READ color WRITE setColor NOTIFY colorChanged)

public:
    explicit ColorEditor(QWidget *parent = 0);

    QColor color() const;

signals:
    void colorChanged(const QColor &color);

public slots:
    void setColor(const QColor &color);

private:
    QColor m_color;
    QLineEdit *m_lineEdit;

private slots:
    void onButtonClicked();
};

#endif // COLOREDITOR_H

To copy to clipboard, switch view to plain text mode 

#include "coloreditor.h"
 
#include <QColorDialog>
#include <QHBoxLayout>
#include <QLineEdit>
#include <QPushButton>
 
ColorEditor::ColorEditor(QWidget *parent)
    : QWidget(parent)
{
    QHBoxLayout *hbox = new QHBoxLayout(this);
 
    m_lineEdit = new QLineEdit(this);
    hbox->addWidget(m_lineEdit);
 
    QPushButton *button = new QPushButton("...", this);
    hbox->addWidget(button);
    connect(button, SIGNAL(clicked()), this, SLOT(onButtonClicked()));
}
 
QColor ColorEditor::color() const
{
    return m_color;
}
 
void ColorEditor::setColor(const QColor &color)
{
    if (color == m_color) {
        return;
    }
 
    m_color = color;
    m_lineEdit->setText(m_color.name());
 
    emit colorChanged(color);
}
 
void ColorEditor::onButtonClicked()
{
    const QColor color = QColorDialog::getColor(m_color, this);
    if (!color.isValid()) {
        return;
    }
 
    setColor(color);
}

#include "coloreditor.h"

#include <QColorDialog>
#include <QHBoxLayout>
#include <QLineEdit>
#include <QPushButton>

ColorEditor::ColorEditor(QWidget *parent)
    : QWidget(parent)
{
    QHBoxLayout *hbox = new QHBoxLayout(this);

    m_lineEdit = new QLineEdit(this);
    hbox->addWidget(m_lineEdit);

    QPushButton *button = new QPushButton("...", this);
    hbox->addWidget(button);
    connect(button, SIGNAL(clicked()), this, SLOT(onButtonClicked()));
}

QColor ColorEditor::color() const
{
    return m_color;
}

void ColorEditor::setColor(const QColor &color)
{
    if (color == m_color) {
        return;
    }

    m_color = color;
    m_lineEdit->setText(m_color.name());

    emit colorChanged(color);
}

void ColorEditor::onButtonClicked()
{
    const QColor color = QColorDialog::getColor(m_color, this);
    if (!color.isValid()) {
        return;
    }

    setColor(color);
}

To copy to clipboard, switch view to plain text mode 

ui->tableView->setModel(model);
 
    QDataWidgetMapper *mapper = new QDataWidgetMapper(this);
    mapper->setModel(model);
    mapper->addMapping(ui->name, 0);
    mapper->addMapping(ui->color, 1, "color");
    mapper->toFirst();
 
    connect(ui->color, SIGNAL(colorChanged(QColor)), mapper, SLOT(submit()));

ui->tableView->setModel(model);

    QDataWidgetMapper *mapper = new QDataWidgetMapper(this);
    mapper->setModel(model);
    mapper->addMapping(ui->name, 0);
    mapper->addMapping(ui->color, 1, "color");
    mapper->toFirst();

    connect(ui->color, SIGNAL(colorChanged(QColor)), mapper, SLOT(submit()));

To copy to clipboard, switch view to plain text mode 

Cheers,
_

[Vincent Le Saux]

I finally managed to make it working! Your code is exactly the same as I tried before my last post and I did not manage to make it working because of a recursivity problem (the sigDataChanged signal was directly emitted in the setColor method).

I had to change a little bit the code :

class ColorWidget(QWidget):
    sigDataChanged = pyqtSignal(str)
    def __init__(self,  color=None):
    some python code here
    self.btnColor.clicked.connect(self.changeColor)
 
    def changeColor(self):
        colorInit = QColor()
        colorInit.setNamedColor(self._color)
        color = QColorDialog.getColor(colorInit, self, "Select a color")
        if color.isValid():
            self.ldtColor.setText(color.name())
            self.setColor(color.name())
            self.sigDataChanged.emit(self._color)
 
    def updateIcon(self):
        pixmap = QPixmap(24, 24)
        newcolor = QColor()
        newcolor.setNamedColor(self._color)
        pixmap.fill(newcolor)
        icon = QIcon(pixmap)
        self.btnColor.setIcon(icon)
 
    def getColor(self):
        return self._color
 
    def setColor(self, newColor):
        self._color = newColor
        self.ldtColor.setText(newColor)
        self.updateIcon()
 
    color = pyqtProperty(str, fget=getColor, fset=setColor)

class ColorWidget(QWidget):
    sigDataChanged = pyqtSignal(str)
    def __init__(self,  color=None):
    some python code here
    self.btnColor.clicked.connect(self.changeColor)
       
    def changeColor(self):
        colorInit = QColor()
        colorInit.setNamedColor(self._color)
        color = QColorDialog.getColor(colorInit, self, "Select a color")
        if color.isValid():
            self.ldtColor.setText(color.name())
            self.setColor(color.name())
            self.sigDataChanged.emit(self._color)
    
    def updateIcon(self):
        pixmap = QPixmap(24, 24)
        newcolor = QColor()
        newcolor.setNamedColor(self._color)
        pixmap.fill(newcolor)
        icon = QIcon(pixmap)
        self.btnColor.setIcon(icon)
        
    def getColor(self):
        return self._color
        
    def setColor(self, newColor):
        self._color = newColor
        self.ldtColor.setText(newColor)
        self.updateIcon()

    color = pyqtProperty(str, fget=getColor, fset=setColor)

To copy to clipboard, switch view to plain text mode 

I just emit the sigDataChanged signal after the setColor method has been called. Finally, I update the model by connecting this signal to the submit slot of the QDataWidgetMapper in the appropriate PropertiesEditor class (as you mentionned).

self.colorWidget.sigDataChanged.connect(self.dataMapper.submit)

self.colorWidget.sigDataChanged.connect(self.dataMapper.submit)

To copy to clipboard, switch view to plain text mode 

Problem solved. I close the thread.

Thank you very much anda_skoa for your time and help .

Cheers,

Vincent

[anda_skoa]

I finally managed to make it working! Your code is exactly the same as I tried before my last post and I did not manage to make it working because of a recursivity problem (the sigDataChanged signal was directly emitted in the setColor method).

My guess would be that this is a problem caused by not checking for actual change.

I do that in setColor(), so the signal is only emitted if the color actually changes.

But maybe PyQt behaves differently in some other aspect.

Cheers,
_

[Vincent Le Saux]

You are definitely right! I just check for changes in the setColor method, and the recursivity problem disappears.

PyQt and Qt behaves exactly the same way on this aspect.

Once again, thank you!

Cheers,

Vincent