Apache OpenOffice (AOO) Bugzilla – Full Text Issue Listing |
Summary: | Implement Excel 2007 style multi-string value filter | ||
---|---|---|---|
Product: | Calc | Reporter: | kyoshida |
Component: | code | Assignee: | AOO issues mailing list <issues> |
Status: | ACCEPTED --- | QA Contact: | |
Severity: | Trivial | ||
Priority: | P3 | CC: | issues, kamataki, kami911, leiw, niklas.nebel, online, openoffice, pagalmes.lists, pescetti, thomas.benisch, weigel |
Version: | 680m211 | ||
Target Milestone: | 4.0.0 | ||
Hardware: | All | ||
OS: | All | ||
URL: | http://wiki.services.openoffice.org/wiki/Calc/Features/multi-selection_filter_extension | ||
Issue Type: | PATCH | Latest Confirmation in: | --- |
Developer Difficulty: | --- | ||
Issue Depends on: | 110133, 27745, 35579, 100619 | ||
Issue Blocks: | |||
Attachments: |
Description
kyoshida
2007-05-22 14:33:56 UTC
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 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. |