diff options
author | Julien Nabet <serval2412@yahoo.fr> | 2018-09-06 20:52:26 +0200 |
---|---|---|
committer | Julien Nabet <serval2412@yahoo.fr> | 2018-09-07 12:00:24 +0200 |
commit | 3448ca93eb46365d05387f7f3512c706bdb987c4 (patch) | |
tree | 74590bd47cd64611210a235323b912a790b82438 | |
parent | c9d15d30b66102aff3f3e10bb8243edf2ab77334 (diff) |
tdf#75341: fix condition to approve row (form)
Considering rColInfo.nNullable != ColumnValue::NO_NULLS is ok to approve row
isn't sufficient in the case the field is "Input required" in form.
Change-Id: I27c57fe8ce5afac97eb0650f93703333c85f1421
Reviewed-on: https://gerrit.libreoffice.org/60109
(cherry picked from commit dbb444e4ed7c19a11733ce8438bbb6546d42f852)
Reviewed-on: https://gerrit.libreoffice.org/60125
Reviewed-by: Lionel Elie Mamane <lionel@mamane.lu>
Tested-by: Jenkins
-rw-r--r-- | svx/source/form/formcontroller.cxx | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/svx/source/form/formcontroller.cxx b/svx/source/form/formcontroller.cxx index c302f4dee524..881f1ec84967 100644 --- a/svx/source/form/formcontroller.cxx +++ b/svx/source/form/formcontroller.cxx @@ -3723,8 +3723,6 @@ sal_Bool SAL_CALL FormController::approveRowChange(const RowChangeEvent& _rEvent for ( size_t col = 0; col < colCount; ++col ) { const ColumnInfo& rColInfo = m_pColumnInfoCache->getColumnInfo( col ); - if ( rColInfo.nNullable != ColumnValue::NO_NULLS ) - continue; if ( rColInfo.bAutoIncrement ) continue; @@ -3732,8 +3730,11 @@ sal_Bool SAL_CALL FormController::approveRowChange(const RowChangeEvent& _rEvent if ( rColInfo.bReadOnly ) continue; - if ( !rColInfo.xFirstControlWithInputRequired.is() && !rColInfo.xFirstGridWithInputRequiredColumn.is() ) + if ( !rColInfo.xFirstControlWithInputRequired.is() && !rColInfo.xFirstGridWithInputRequiredColumn.is() + && rColInfo.nNullable != ColumnValue::NO_NULLS ) + { continue; + } // TODO: in case of binary fields, this "getString" below is extremely expensive if ( !rColInfo.xColumn->getString().isEmpty() || !rColInfo.xColumn->wasNull() ) |