[ASP-28550] Docx with un-accepted changes #72
Conversation
| @@ -74,6 +56,27 @@ void testHeaderParagraph() throws IOException { | |||
| } | |||
| } | |||
|
|
|||
| @Test | |||
| void testUnAcceptedChanges() throws IOException { | |||
| try (XWPFDocument xml = XWPFTestDataSamples.openSampleDocument("UnAcceptedChangesTest.docx")) { | |||
There was a problem hiding this comment.
Еще нужны тесты на remove аля
testRemoveRuns
Вот сюда нужно добавить новый вид ранов
testRemoveAndInsertRunsWithOtherIRunElement
Мб еще есть тесты в этом классе где мы вариантивно создаем разные раны - туда нужно тоже добавить новый ран
| this.trackChange = trackChange; | ||
| } | ||
|
|
||
| public CTRunTrackChange getTrackChange() { |
There was a problem hiding this comment.
Тут и в методах ниже this не обяз. писать для доступа к проперти
|
Еще нужно такую же логику реализовать в XWPFSDTContentRun. Обрати внимание на метод buildRunsInOrderFromXml |
| @@ -146,7 +146,7 @@ private void buildRunsInOrderFromXml(XmlObject object) { | |||
| } | |||
| if (o instanceof CTRunTrackChange) { | |||
| for (CTR r : ((CTRunTrackChange) o).getRArray()) { | |||
| XWPFRun cr = new XWPFRun(r, this); | |||
| XWPFRun cr = new XWPFTrackChangeRun((CTRunTrackChange) o, r, this); | |||
There was a problem hiding this comment.
раз уж мы тут вносим измнения, то может добавим сюда вот эти вот методы?
https://github.com/pdffiller/dadadocs-scripting-template-engine/pull/475/files#diff-52316947a9ce35960fd4534be4ce3cd1b30dc8d78081839e75a67628a04ce86dR26
airvavilov
changed the title
No description provided.