search

mariarahat / bungeni-editor

Automatically exported from code.google.com/p/bungeni-editor
2 stars 0 forks source link

BungeniODFDOM api to retrieve changes by user does not match update edits #70

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Update edits use the following xml pattern :

        <text:changed-region text:id="ct-1412927444">
                    <text:deletion>
                        <office:change-info>
                            <dc:creator>Ashok Hariharan</dc:creator>
                            <dc:date>2010-03-05T16:02:00</dc:date>
                        </office:change-info>
                        <text:p text:style-name="Standard">te</text:p>
                    </text:deletion>
                    <text:insertion>
                        <office:change-info office:chg-author="Flavio Zeni"
                            office:chg-date-time="2010-03-05T16:01:00"/>
                    </text:insertion>
                </text:changed-region>

essentailly a deletion and insertion element packaged into a changed-region
container. the current API matches changed regions only by dc:creator and
does not check for office:change-info/office:chg-author.

Original issue reported on code.google.com by ashok.ha...@gmail.com on 8 Mar 2010 at 7:28

GoogleCodeExporter commented 9 years ago

Original comment by ashok.ha...@gmail.com on 8 Mar 2010 at 7:44

GoogleCodeExporter commented 9 years ago 
Changed Xpath to match changed regions 
//text:changed-region[(descendant::text:insertion/office:change-info[@office:chg
-author='Ashok
Hariharan']) or (descendant::dc:creator='Ashok Hariharan')]

instead of 

//text:changed-region[(descendant::dc:creator='Ashok Hariharan')]

Original comment by ashok.ha...@gmail.com on 8 Mar 2010 at 7:45

GoogleCodeExporter commented 9 years ago 
getChange() also needs to be fixed for the replace-change pattern

Original comment by ashok.ha...@gmail.com on 8 Mar 2010 at 10:57

GoogleCodeExporter commented 9 years ago 
getChange() was updated - Unit tests need to updated and tested for this 
specific api

Original comment by ashok.ha...@gmail.com on 9 Mar 2010 at 6:06

GoogleCodeExporter commented 9 years ago

Original comment by ashok.ha...@gmail.com on 4 May 2010 at 10:29