Skip to content

Commit 584cbd0

Browse files
committed
Fix test failure
1 parent dddce25 commit 584cbd0

File tree

2 files changed

+7
-3
lines changed

2 files changed

+7
-3
lines changed

src/gui/layout/qgslayoutnewitempropertiesdialog.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ void QgsLayoutItemPropertiesDialog::setItemPosition( QgsLayoutPoint position )
6161
position = mLayout->convertFromLayoutUnits( mLayout->pageCollection()->positionOnPage( layoutPoint ), position.units() );
6262

6363
mPageSpin->setValue( page + 1 );
64-
mPosUnitsComboBox->setUnit( position.units() );
64+
whileBlocking( mPosUnitsComboBox )->setUnit( position.units() );
6565
mXPosSpin->setValue( position.x() );
6666
mYPosSpin->setValue( position.y() );
6767
}
@@ -80,7 +80,7 @@ void QgsLayoutItemPropertiesDialog::setItemSize( QgsLayoutSize size )
8080
{
8181
mWidthSpin->setValue( size.width() );
8282
mHeightSpin->setValue( size.height() );
83-
mSizeUnitsComboBox->setUnit( size.units() );
83+
whileBlocking( mSizeUnitsComboBox )->setUnit( size.units() );
8484
}
8585

8686
QgsLayoutSize QgsLayoutItemPropertiesDialog::itemSize() const

tests/src/python/test_qgslayoutitempropertiesdialog.py

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414

1515
import qgis # NOQA
1616

17-
from qgis.core import QgsUnitTypes, QgsLayoutSize, QgsLayoutPoint, QgsLayoutItem
17+
from qgis.core import QgsUnitTypes, QgsLayoutSize, QgsLayoutPoint, QgsLayoutItem, QgsProject, QgsLayout
1818
from qgis.gui import QgsLayoutItemPropertiesDialog
1919

2020
from qgis.testing import start_app, unittest
@@ -28,6 +28,10 @@ def testGettersSetters(self):
2828
""" test dialog getters/setters """
2929
dlg = qgis.gui.QgsLayoutItemPropertiesDialog()
3030

31+
l = QgsLayout(QgsProject.instance())
32+
l.initializeDefaults()
33+
dlg.setLayout(l)
34+
3135
dlg.setItemPosition(QgsLayoutPoint(5, 6, QgsUnitTypes.LayoutPixels))
3236
self.assertEqual(dlg.itemPosition().x(), 5.0)
3337
self.assertEqual(dlg.itemPosition().y(), 6.0)

0 commit comments

Comments
 (0)