Apache OpenOffice (AOO) Bugzilla – Full Text Issue Listing |
Summary: | Compare Document doesn't compare tables | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | Writer | Reporter: | Unknown <non-migrated> | ||||||||||
Component: | code | Assignee: | AOO issues mailing list <issues> | ||||||||||
Status: | CONFIRMED --- | QA Contact: | |||||||||||
Severity: | Trivial | ||||||||||||
Priority: | P3 | CC: | axa1981, cmoulin, cno, gleppert, issues, jian.li, kami911, thomas.lendo | ||||||||||
Version: | OOo 1.1 RC5 | Keywords: | oooqa | ||||||||||
Target Milestone: | --- | ||||||||||||
Hardware: | PC | ||||||||||||
OS: | Windows XP | ||||||||||||
Issue Type: | DEFECT | Latest Confirmation in: | --- | ||||||||||
Developer Difficulty: | --- | ||||||||||||
Attachments: |
|
Description
Unknown
2003-07-03 11:59:38 UTC
I can Reproduce the problem on OpenOffice 1.1 RC5(default Install, US), Win XP Pro Sp1. (And MS Office XP Sp2). But I am not sure it is bugs or feature change to new reassigend to jw set target to OOo2.0 reassigend to dvo can you please take a look at this issue. i set the target to OOo2.0 because es told me that you are currently working on redlining issues for the 2.0. please call back or change the target, if this is not fixable to OOo2.0 *** Issue 28325 has been marked as a duplicate of this issue. *** No chance to make a reasonable implementation until OO 2.0. Target OO later now. I was able to replicate the problem on OOo 1.1.4 using Windows XP (service pack 2). The problem seems to only exist with all table compares only, even if the difference is in a totally different cell, but if the 2 documents have different non-table text, it works fine. Comparing the contents of tables still seems to be a problem with OO 2.0.3. Changes are only reflected if complete cells are deleted or inserted, but not if only the contents of a cell was changed. We are using open office (2.3) for comparing documents and this issue is a real headache for us. Seems to be doable... Fixed in CWS swqbf105 doccomp.cxx Great that this has been picked up! Would it be possible to include the fix also in the 2.3.1 code line? ->Cor Technically possible, of course. But I don't think that this issue fulfills the criteria for 2.3.1 Great news that this has been fixed - thank you It would be great if it could go in 2.3.1 Many thanks Anthony Ready for QA. BTW: OOo is not able to compare a document with itself, i.e. open a document "xy.odt", change something, compare with "xy.odt" again => nothing will be compared! What I've fixed is the problem of _table_ comparison between _two_ documents. Sorry to say that, but I've to remove this issue from CWS swqbf105 because of the occurrence of new crashes. I will create a new CWS for this issues where I am planning to solve all problems we get if we enable table comparison. Reassigned to myself again :-( ama->lijian: I tried to enable doc comparison for table content but run into trouble even with your fix for issue 73682 Created attachment 49820 [details]
My patch to enable table comparison
Created attachment 49821 [details]
Document with table to comparison
Created attachment 49822 [details]
Document with table to comparison
Created attachment 49823 [details]
Document with table to comparison
ama->lijian: I checked your patch from issue 73682 together with my patch (see attachment doccomp.diff). It works fine e.g. if I compare doccomp01c.odt with doccomp01b.odt. But if I compare doccomp01a.odt with doccomp01b.odt, it does not look like expected. And if I accept the first deletion and try to undo this action afterwards, I'll run into trouble (C++ exception etc.). So it looks like my patch needs some improvement. If you are interested in this issue, your help would be very well appreciated! lijian->ama: I would love to. :-) Hope I can help. Sorry to say: New target OOo3.0. I set the target for swredline02 to OOo3.0 because I don't think that we should integrate this CWS in the current version into OOo2.4. (Too much side-effects, too less time to solve all problems) New target OOo3.0 Retarget to OOo3.1 There are too many open questions to finish in 3.1. *** Issue 96416 has been marked as a duplicate of this issue. *** I can reproduce the problem on OOo-dev3.2.0 DEV300m44. (excuse me for my bad english) Can reproduce bug with OpenOffice Writer 3.0.0 under Windows XP and 3.0.1 under Slackware 12.1. May be OpenOffice can say something like "Comparing documents with tables is unsupported yet" if it find any tables when comparing documents? It is temporary "solution" before bug will be fixed. IMHO it will be much better than deceiving user. Now user ask OpenOffice to compare documents, see that OpenOffice did not report any differences for tables and can think that tables really equivalent. This is bad. Sincere warning will be better. Hi sergey feo, Our friends of DeltaXML (also present at latest OOoCon) have made a fine extension as (temporary) solution: http://extensions.services.openoffice.org/project/DeltaXMLODTCompare Dear cornouws, thanks ever so much for this hint! I didn't know of this extension yet, and in fact it does exactly what I need! Not only tables are perfectly compared now, but also changes in images are shown. Also, the comparison algorithm for normal text is better than the one of the built-in comparison feature, which often shows whole text blocks as changed instead of the individual word that was changed. So thanks a lot also to the DeltaXML people! I think this issue is resolved by this extension. :-) Thanks again, phil The mentioned extension does NOT really solve the problem. It introduces a new and only English interface that doesn't fit very well in OOo, the comparison method requires to save the result in a new file(!), and most OOo users don't know about extensions. The extension is only a time-consuming workaround for some people. Hope AMA can spare time to create a new patch in the future. :) *** Issue 105525 has been marked as a duplicate of this issue. *** Also affected, voting. One awkward workaround appears be to paste your tables into spreadsheets (OO.o Spreadsheet) and use its comparison feature. However, you need to save out one of the sheet documents before you can run the comparison, which is annoying. So, two suggestions from a user: -If doing this "right" is somewhat difficult, can it be band-aided by comparing a checksum or hash for the entire table and doing something to indicate a mismatch [for the entire table as an object]? This would be way better than failing silently. (Presumably the hard part is not matching tables between documents by name.) Getting a "false difference" if only "metadata" (like the table name) differs would still beat getting a "false match" when it fails silently. -If the functionality in Sheet works, maybe someone wants to patch its comparison function to support comparing Sheet1, 2, 3 etc. in one spreadsheet "document". That would take a couple steps out of the workaround and make it almost-acceptable in the meantime. Or perhaps there's robust code in there that can be reused? added myself to CC. So far, this issue is scheduled for 3.3, at least according to the CWS where it is in. Will we see it in 3.3? |