diff options
| author | Mike Kaganski <mike.kaganski@collabora.com> | 2022-08-24 10:49:47 +0200 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2022-08-26 11:42:28 +0200 |
| commit | 9fc6b68a075201b52ba1b70f60e04e6200dcf35b (patch) | |
| tree | 06fad8cdfdf47de1e78729aa051a6cf376c97016 /svx | |
| parent | ed95bc0d2f0d8ee8f04ac84bcb4e41d780878501 (diff) | |
tdf#150577: Revert "Micro-optim in FormController::approveRowChange"
This reverts commit e005ab5d40d358adb75a64e140d46f4bf605647d.
XColumn::wasNull is documented [1] to require a previous call to a get*
method. The reverted change internally queried the *previous* column's
wasNull.
[1] https://api.libreoffice.org/docs/idl/ref/interfacecom_1_1sun_1_1star_1_1sdb_1_1XColumn.html#a0d482c20564f9119052b1962f830190e
Change-Id: I3e988358764fa6935e8eb3e66340a0869c157d02
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138550
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
(cherry picked from commit 59f30175bfc557aa7c752ab0b45af9d34215d4dc)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138821
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'svx')
| -rw-r--r-- | svx/source/form/formcontroller.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/svx/source/form/formcontroller.cxx b/svx/source/form/formcontroller.cxx index d38b14d67367..d53b589394d3 100644 --- a/svx/source/form/formcontroller.cxx +++ b/svx/source/form/formcontroller.cxx @@ -3647,7 +3647,7 @@ sal_Bool SAL_CALL FormController::approveRowChange(const RowChangeEvent& _rEvent } // TODO: in case of binary fields, this "getString" below is extremely expensive - if ( !rColInfo.xColumn->wasNull() || !rColInfo.xColumn->getString().isEmpty() ) + if ( !rColInfo.xColumn->getString().isEmpty() || !rColInfo.xColumn->wasNull() ) continue; OUString sMessage( SvxResId( RID_ERR_FIELDREQUIRED ) ); |