diff options
| author | Mike Kaganski <mike.kaganski@collabora.com> | 2022-08-24 10:49:47 +0200 |
|---|---|---|
| committer | Andras Timar <andras.timar@collabora.com> | 2022-08-28 14:00:19 +0200 |
| commit | dc94c82f67369219b69a0daa34c055eea28dfb5c (patch) | |
| tree | e530e86a9f43e7f09e6b104870ec26e3ba0fc56c /svx/source | |
| parent | dd816077a3dc8fa29590ed7b926012a96f25b6bf (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/+/138822
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'svx/source')
| -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 50e0d4f592d9..d5687a15ba03 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 ) ); |