sc/qa/uitest/autofilter/autofilter.py | 15 ++++++++++++ sc/qa/uitest/data/autofilter/tdf137626.xlsx |binary sc/source/core/data/column3.cxx | 8 ++++++ sc/source/filter/oox/autofilterbuffer.cxx | 33 +++++++++++++++++++++++++--- sc/source/ui/unoobj/datauno.cxx | 5 ++-- 5 files changed, 56 insertions(+), 5 deletions(-)
New commits: commit 842fcf712864d88d0eef5098574b8728b8a7ee71 Author: Balazs Varga <[email protected]> AuthorDate: Tue Mar 2 22:46:33 2021 +0100 Commit: Gabor Kelemen <[email protected]> CommitDate: Tue Jun 29 09:54:11 2021 +0200 tdf#137626 XLSX import: fix missing datetime filters by convert string representation of the datetime data to ISO 8601 (with blank instead of T) datetime to eliminate locale dependent behaviour when filtering for datetimes. Follow-up of commit 0e751d0cb816197f15a2448ec36c57df17387e40 (tdf#116818 sc,offapi,XLSX import: fix autofiltered date columns). Change-Id: I3a0f41dbbf28a1a60a54fe7b2c8c338516edb079 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/111851 Tested-by: László Németh <[email protected]> Reviewed-by: László Németh <[email protected]> (cherry picked from commit 26032e63abd01c3d5941a2728ef024da290d6b0a) Reviewed-on: https://gerrit.libreoffice.org/c/core/+/118056 Tested-by: Gabor Kelemen <[email protected]> Reviewed-by: Gabor Kelemen <[email protected]> diff --git a/sc/qa/uitest/autofilter/autofilter.py b/sc/qa/uitest/autofilter/autofilter.py index b8f2bb3be83f..18bb9105bb87 100644 --- a/sc/qa/uitest/autofilter/autofilter.py +++ b/sc/qa/uitest/autofilter/autofilter.py @@ -282,5 +282,20 @@ class AutofilterTest(UITestCase): xOkBtn = xFloatWindow.getChild("cancel") xOkBtn.executeAction("CLICK", tuple()) + self.ui_test.close_doc() + + def test_tdf137626(self): + doc = self.ui_test.load_file(get_url_for_data_file("tdf137626.xlsx")) + + xGridWin = self.xUITest.getTopFocusWindow().getChild("grid_window") + + xGridWin.executeAction("LAUNCH", mkPropertyValues({"AUTOFILTER": "", "COL": "1", "ROW": "0"})) + xFloatWindow = self.xUITest.getFloatWindow() + xCheckListMenu = xFloatWindow.getChild("check_list_menu") + xTreeList = xCheckListMenu.getChild("check_list_box") + self.assertEqual(3, len(xTreeList.getChildren())) + xOkBtn = xFloatWindow.getChild("cancel") + xOkBtn.executeAction("CLICK", tuple()) + self.ui_test.close_doc() # vim: set shiftwidth=4 softtabstop=4 expandtab: diff --git a/sc/qa/uitest/data/autofilter/tdf137626.xlsx b/sc/qa/uitest/data/autofilter/tdf137626.xlsx new file mode 100644 index 000000000000..eb5ce4da7b98 Binary files /dev/null and b/sc/qa/uitest/data/autofilter/tdf137626.xlsx differ diff --git a/sc/source/core/data/column3.cxx b/sc/source/core/data/column3.cxx index 69213b7fedbc..8a457be7ea00 100644 --- a/sc/source/core/data/column3.cxx +++ b/sc/source/core/data/column3.cxx @@ -2473,6 +2473,14 @@ class FilterEntriesHandler sal_uInt32 nIndex = pFormatter->GetFormatIndex( NF_DATE_DIN_YYYYMMDD); pFormatter->GetInputLineString( fVal, nIndex, aStr); } + else if (nType == SvNumFormatType::DATETIME) + { + // special case for datetime values. + // Convert string representation to ISO 8601 (with blank instead of T) datetime + // to eliminate locale dependent behaviour later when filtering for datetimes. + sal_uInt32 nIndex = pFormatter->GetFormatIndex(NF_DATETIME_ISO_YYYYMMDD_HHMMSS); + pFormatter->GetInputLineString(fVal, nIndex, aStr); + } // maybe extend ScTypedStrData enum is also an option here mrFilterEntries.push_back(ScTypedStrData(aStr, fVal, ScTypedStrData::Value,bDate)); } diff --git a/sc/source/filter/oox/autofilterbuffer.cxx b/sc/source/filter/oox/autofilterbuffer.cxx index a9ec62c4e655..e09bd084e7f3 100644 --- a/sc/source/filter/oox/autofilterbuffer.cxx +++ b/sc/source/filter/oox/autofilterbuffer.cxx @@ -240,23 +240,50 @@ void DiscreteFilter::importAttribs( sal_Int32 nElement, const AttributeList& rAt // it is just a fallback, we do not need the XML_day as default value, // because if the dateGroupItem exists also XML_dateTimeGrouping exists! sal_uInt16 nToken = rAttribs.getToken(XML_dateTimeGrouping, XML_day); - if( nToken == XML_year || nToken == XML_month || nToken == XML_day ) + if( nToken == XML_year || nToken == XML_month || nToken == XML_day || + nToken == XML_hour || nToken == XML_min || nToken == XML_second ) { aDateValue = rAttribs.getString(XML_year, OUString()); - if( nToken == XML_month || nToken == XML_day ) + if( nToken == XML_month || nToken == XML_day || nToken == XML_hour || + nToken == XML_min || nToken == XML_second ) { OUString aMonthName = rAttribs.getString(XML_month, OUString()); if( aMonthName.getLength() == 1 ) aMonthName = "0" + aMonthName; aDateValue += "-" + aMonthName; - if( nToken == XML_day ) + if( nToken == XML_day || nToken == XML_hour || nToken == XML_min || + nToken == XML_second ) { OUString aDayName = rAttribs.getString(XML_day, OUString()); if( aDayName.getLength() == 1 ) aDayName = "0" + aDayName; aDateValue += "-" + aDayName; + + if( nToken == XML_hour || nToken == XML_min || nToken == XML_second ) + { + OUString aHourName = rAttribs.getString(XML_hour, OUString()); + if( aHourName.getLength() == 1 ) + aHourName = "0" + aHourName; + aDateValue += " " + aHourName; + + if( nToken == XML_min || nToken == XML_second ) + { + OUString aMinName = rAttribs.getString(XML_min, OUString()); + if( aMinName.getLength() == 1 ) + aMinName = "0" + aMinName; + aDateValue += ":" + aMinName; + + if( nToken == XML_second ) + { + OUString aSecName = rAttribs.getString(XML_second, OUString()); + if( aSecName.getLength() == 1 ) + aSecName = "0" + aSecName; + aDateValue += ":" + aSecName; + } + } + } } } } diff --git a/sc/source/ui/unoobj/datauno.cxx b/sc/source/ui/unoobj/datauno.cxx index 8e8e54536037..531cc27eaa0f 100644 --- a/sc/source/ui/unoobj/datauno.cxx +++ b/sc/source/ui/unoobj/datauno.cxx @@ -1135,8 +1135,9 @@ void fillQueryParam( aItem.maString = rPool.intern(aStr); } - // filter all dates starting with the given date filter YYYY or YYYY-MM - if( aItem.meType == ScQueryEntry::ByDate && aItem.maString.getLength() < 10 ) + // filter all dates starting with the given date filter YYYY or YYYY-MM and filter all datetimes + // starting with the given datetime filter YYYY-MM-DD, YYYY-MM-DD HH, or YYYY-MM-DD HH:MM + if( aItem.meType == ScQueryEntry::ByDate && aItem.maString.getLength() < 19 ) { ScFilterEntries aFilterEntries; pDoc->GetFilterEntries(rEntry.nField, rParam.nRow1, rParam.nTab, aFilterEntries); _______________________________________________ Libreoffice-commits mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits
