Apache OpenOffice (AOO) Bugzilla – Issue 77677
Implement Excel 2007 style multi-string value filter
Last modified: 2018-08-18 12:49:32 UTC
We need to implement the Excel 2007 style multi-string value filter in order to import XLSX cleanly. In Excel 2007, when you turn on autofilter, the autofilter list box shows a list of values found in that column with a checkbox next to each value. You can then check only the items you want to show. In the file format, this is given as a list of string items that need to be displayed. We need to a new filter of this type in Calc to correctly import this from XLSX. I have a draft implementation of this. After cleaning it up a bit, I will submit a patch. Kohei
adding jody.
Created attachment 45351 [details] patch for offapi module
Created attachment 45352 [details] patch for sc module
This is just an internal, non-UI part of the implementation. I propose we integrate this first as part of the on-going OOXML importer, and the implement the UI at a later time. The patch should be easily applied in the xmlfilter02 CWS. Kohei
P1 is for build breakers ;-)
Created attachment 45376 [details] updated patch for sc module
Created attachment 45377 [details] This one is the right one. The previous patch doesn't build.
Apart from the subject of file format, published API structs can't be changed. This would break existing components/programs that use the API.
removing jody as I don't think he would appreciates more spam coming in his way. :-)
Created attachment 45973 [details] updated patch to use new UNO API instead of modifying existing API
Created attachment 45974 [details] forget the last patch. This one is the correct one.
This new patch implements the same functionality as the original patch, but uses new UNO API instead of modifying an existing, published API. One note. Two new UNO APIs provided in this patch (TableMultiStringFilterField and XExtendedSheetFilterDescriptor) are meant to be *un*-published. But because I could not compile SheetFilterDescriptor.idl without making them published, I had to mark them "published". It is likely that the XExtendedSheetFilterDescriptor interface to be further extended in future, as we introduce a new filter type, it is important that this interface will stay unpublished (if that's even possible). Also, because I had to import filters of different types, I designed XExtendedSheetFilterDescriptor interface to follow SQL transactional style API, where you call begin() to initiate transactions and commit() to commit all filter items when done.
Created attachment 46924 [details] set of patches
I've justed posted the latest set of patches for this functionality. It implements proper import and export of the extra multi-string type filter adhering to the change proposed (and later accepted) to the ODF v1.2[1]. Since this work is becoming a little too large to be contained in patch form, I will soon create a CWS for this. But I'd like us to discuss and agree upon the UNO API that this functionality will introduce before going further. [1] http://wiki.oasis-open.org/office/OpenDocument_v1.2_Action_Items#head-4854119a1f3d7df4f22fc453bb50e8c6e5b0acd7
>I will soon create a CWS for this. The better strategy would be to put this change into xmlfilter02, since the OOXML importer code is written with this change in mind. This code is in good shape, and ready to be committed (some testing is of course needed). Since the UI part is not implemented, we don't need to be concerned about the specification or feature mail. That will be handled at a later time. Please advise.
I have created autofilter01 CWS to implement this further. So far, the draft UNO API has been implemented (not complete yet), and the ODF import filter has been modified to allow importing this new multi-selection filter items. autofilter01 CWS information http://eis.services.openoffice.org/EIS2/cws.ShowCWS?Path=SRC680%2Fautofilter01 My plan is to finish up the UNO API, the ODF import/export, then work on the UI.
I'm taking this issue.
started
Spec page: http://wiki.services.openoffice.org/wiki/Calc/Features/multi-selection_filter_extension
cc tbe
I don't understand if this issue refers to this functionality: http://h30187.www3.hp.com/tutorials/viewHowTo/p/courseId/14171/Microsoft_Excel_2007_filter_data.htm But if it is... why is it stalled? It's very useful when working with a lot of data.
>why is it stalled? Well, my initial idea was to wait for Issue 35579 to be resolved & change integrated since the two features touch on the same parts of Calc's code. I made that decision two years ago. But Issue 35579 has been taking a lot longer than I had anticipated, and it's not integrated yet even today.... That's the reason for stall.
Thanks for the update kohei. Perhaps it will be helpful to contact maoyg to know what is the status of the actual patch, or when she plan to have a final version.
I will review the patch for issue 35579 after my Easter vacation. The patch is quite large and had a lot of reviews in the past, but I hope to get this feature integrated in the near future.
@tbe: thank you for volunteering that. Just to make sure one thing, I didn't mean to finger-point anybody for the delay. It sometimes happens; people get busy and some things get pushed aside and be forgotten...
kohei -> also thanks to you :D
At least issue 35579 has been fixed!!! Thanks tbe!! Now, what is the next step? wait for the integration with the main tree? (Sorry for being impatient, but for me is a very exciting feature)
sucotronic: In principal you have to thank gaozm, lvyue and maoyg. I did only the review. Even if you're impatient, if you don't want to build an OOo version yourself, you have to wait until this childworkspace is integrated in the master.
We need to re-use the new DataPilot filter popup window for this, hence the dependency.
*** Issue 107351 has been marked as a duplicate of this issue. ***
By resolving this one, please consider resolution of issue #110133.
I'm adding this comment to all open issues with Issue Type == PATCH. We have 220 such issues, many of them quite old. I apologize for that. We need your help in prioritizing which patches should be integrated into our next release, Apache OpenOffice 4.0. If you have submitted a patch and think it is applicable for AOO 4.0, please respond with a comment to let us know. On the other hand, if the patch is no longer relevant, please let us know that as well. If you have any general questions or want to discuss this further, please send a note to our dev mailing list: dev@openoffice.apache.org Thanks! -Rob
Reset assignee on issues not touched by assignee in more than 1000 days.
This issues is running for 6! years and even while patch was almost ready, it is being ignored. feature was long added is essential to be able to use Excel Work Flows and was also long added to LO. If a document structure is designed with this feature in mind, it lack in makes user productivity drop in a major way causing user to routinely reject AOO as viable alternative. Please consider raising the priority. While it was "nice to have" in 2007, by prevalence of 2007+ Office these days as well as inclusion in LO 3.5+ it has become essential in 2013.