Skip to content

Commit 4452a19

Browse files
author
Satvik Kumar
committed
Operations for merging and splitting lists
1 parent 33566eb commit 4452a19

File tree

6 files changed

+318
-1
lines changed

6 files changed

+318
-1
lines changed

‎webodf/lib/manifest.json‎

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -447,6 +447,9 @@
447447
"ops.OpInsertText": [
448448
"ops.OdtDocument"
449449
],
450+
"ops.OpMergeList": [
451+
"ops.OdtDocument"
452+
],
450453
"ops.OpMergeParagraph": [
451454
"odf.CollapsingRules",
452455
"ops.OdtDocument"
@@ -485,6 +488,10 @@
485488
"ops.OpSetParagraphStyle": [
486489
"ops.OdtDocument"
487490
],
491+
"ops.OpSplitList": [
492+
"odf.CollapsingRules",
493+
"ops.OdtDocument"
494+
],
488495
"ops.OpSplitParagraph": [
489496
"ops.OdtDocument"
490497
],
@@ -512,6 +519,7 @@
512519
"ops.OpInsertImage",
513520
"ops.OpInsertTable",
514521
"ops.OpInsertText",
522+
"ops.OpMergeList",
515523
"ops.OpMergeParagraph",
516524
"ops.OpMoveCursor",
517525
"ops.OpRemoveAnnotation",
@@ -524,6 +532,7 @@
524532
"ops.OpRemoveText",
525533
"ops.OpSetBlob",
526534
"ops.OpSetParagraphStyle",
535+
"ops.OpSplitList",
527536
"ops.OpSplitParagraph",
528537
"ops.OpUpdateMember",
529538
"ops.OpUpdateMetadata",

‎webodf/lib/ops/OpMergeList.js‎

Lines changed: 110 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,110 @@
1+
/**
2+
* Copyright (C) 2010-2014 KO GmbH <[email protected]>
3+
*
4+
* @licstart
5+
* This file is part of WebODF.
6+
*
7+
* WebODF is free software: you can redistribute it and/or modify it
8+
* under the terms of the GNU Affero General Public License (GNU AGPL)
9+
* as published by the Free Software Foundation, either version 3 of
10+
* the License, or (at your option) any later version.
11+
*
12+
* WebODF is distributed in the hope that it will be useful, but
13+
* WITHOUT ANY WARRANTY; without even the implied warranty of
14+
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
15+
* GNU Affero General Public License for more details.
16+
*
17+
* You should have received a copy of the GNU Affero General Public License
18+
* along with WebODF. If not, see <http://www.gnu.org/licenses/>.
19+
* @licend
20+
*
21+
* @source: http://www.webodf.org/
22+
* @source: https://github.com/kogmbh/WebODF/
23+
*/
24+
25+
/*global ops, runtime, odf, core */
26+
27+
/**
28+
*
29+
* @constructor
30+
* @implements ops.Operation
31+
*/
32+
ops.OpMergeList = function OpMergeList() {
33+
"use strict";
34+
35+
var memberid,
36+
timestamp,
37+
/**@type{!number}*/
38+
sourceStartPosition,
39+
/**@type{!number}*/
40+
destinationStartPosition,
41+
odfUtils = odf.OdfUtils;
42+
43+
/**
44+
* @param {!ops.OpMergeList.InitSpec} data
45+
*/
46+
this.init = function (data) {
47+
memberid = data.memberid;
48+
timestamp = data.timestamp;
49+
sourceStartPosition = data.sourceStartPosition;
50+
destinationStartPosition = data.destinationStartPosition;
51+
};
52+
53+
this.isEdit = true;
54+
this.group = undefined;
55+
56+
/**
57+
* @return {!ops.OpMergeList.Spec}
58+
*/
59+
this.spec = function () {
60+
return {
61+
optype: "MergeList",
62+
memberid: memberid,
63+
timestamp: timestamp,
64+
sourceStartPosition: sourceStartPosition,
65+
destinationStartPosition: destinationStartPosition
66+
};
67+
};
68+
69+
/**
70+
* @param {!ops.Document} document
71+
*/
72+
this.execute = function (document) {
73+
var odtDocument = /**@type{ops.OdtDocument}*/(document),
74+
rootNode = odtDocument.getRootNode(),
75+
sourceDomPosition = odtDocument.convertCursorStepToDomPoint(sourceStartPosition),
76+
destinationDomPosition = odtDocument.convertCursorStepToDomPoint(destinationStartPosition),
77+
sourceList,
78+
destinationList,
79+
childListItem;
80+
81+
sourceList = odfUtils.getTopLevelListElement(sourceDomPosition.node, rootNode);
82+
destinationList = odfUtils.getTopLevelListElement(destinationDomPosition.node, rootNode);
83+
childListItem = sourceList.firstElementChild;
84+
85+
while (childListItem) {
86+
destinationList.appendChild(childListItem);
87+
childListItem = sourceList.firstElementChild;
88+
}
89+
90+
sourceList.parentNode.removeChild(sourceList);
91+
return true;
92+
};
93+
};
94+
95+
/**@typedef{{
96+
optype: !string,
97+
memberid: !string,
98+
timestamp: !number,
99+
sourceStartPosition: !number,
100+
destinationStartPosition: !number
101+
}}*/
102+
ops.OpMergeList.Spec;
103+
104+
/**@typedef{{
105+
memberid: !string,
106+
timestamp:(!number|undefined),
107+
sourceStartPosition: !number,
108+
destinationStartPosition: !number
109+
}}*/
110+
ops.OpMergeList.InitSpec;

‎webodf/lib/ops/OpSplitList.js‎

Lines changed: 132 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,132 @@
1+
/**
2+
* Copyright (C) 2010-2014 KO GmbH <[email protected]>
3+
*
4+
* @licstart
5+
* This file is part of WebODF.
6+
*
7+
* WebODF is free software: you can redistribute it and/or modify it
8+
* under the terms of the GNU Affero General Public License (GNU AGPL)
9+
* as published by the Free Software Foundation, either version 3 of
10+
* the License, or (at your option) any later version.
11+
*
12+
* WebODF is distributed in the hope that it will be useful, but
13+
* WITHOUT ANY WARRANTY; without even the implied warranty of
14+
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
15+
* GNU Affero General Public License for more details.
16+
*
17+
* You should have received a copy of the GNU Affero General Public License
18+
* along with WebODF. If not, see <http://www.gnu.org/licenses/>.
19+
* @licend
20+
*
21+
* @source: http://www.webodf.org/
22+
* @source: https://github.com/kogmbh/WebODF/
23+
*/
24+
25+
/*global ops, runtime, odf, core, Range*/
26+
27+
/**
28+
*
29+
* @constructor
30+
* @implements ops.Operation
31+
*/
32+
ops.OpSplitList = function OpSplitList() {
33+
"use strict";
34+
35+
var memberid,
36+
timestamp,
37+
/**@type{!number}*/
38+
sourceStartPosition,
39+
/**@type{!number}*/
40+
splitPosition,
41+
odfUtils = odf.OdfUtils,
42+
/**@const*/
43+
xmlns = odf.Namespaces.xmlns;
44+
45+
/**
46+
* @param {!ops.OpSplitList.InitSpec} data
47+
*/
48+
this.init = function (data) {
49+
memberid = data.memberid;
50+
timestamp = data.timestamp;
51+
sourceStartPosition = data.sourceStartPosition;
52+
splitPosition = data.splitPosition;
53+
};
54+
55+
this.isEdit = true;
56+
this.group = undefined;
57+
58+
/**
59+
* @return {!ops.OpSplitList.Spec}
60+
*/
61+
this.spec = function () {
62+
return {
63+
optype: "SplitList",
64+
memberid: memberid,
65+
timestamp: timestamp,
66+
sourceStartPosition: sourceStartPosition,
67+
splitPosition: splitPosition
68+
};
69+
};
70+
71+
/**
72+
* @param {!ops.Document} document
73+
*/
74+
this.execute = function (document) {
75+
var odtDocument = /**@type{ops.OdtDocument}*/(document),
76+
ownerDocument = odtDocument.getDOMDocument(),
77+
rootNode = odtDocument.getRootNode(),
78+
collapseRules = new odf.CollapsingRules(rootNode),
79+
sourceDomPosition = odtDocument.convertCursorStepToDomPoint(sourceStartPosition),
80+
splitDomPosition = odtDocument.convertCursorStepToDomPoint(splitPosition),
81+
sourceParagraph = /**@type{!Element}*/(odfUtils.getParagraphElement(sourceDomPosition.node, sourceDomPosition.offset)),
82+
splitParagraph = /**@type{!Element}*/(odfUtils.getParagraphElement(splitDomPosition.node, splitDomPosition.offset)),
83+
sourceList = odfUtils.getTopLevelListElement(sourceParagraph, rootNode),
84+
destinationList,
85+
splitPositionParentList,
86+
range = ownerDocument.createRange(),
87+
fragment;
88+
89+
if (!sourceList || !splitParagraph) {
90+
return false;
91+
}
92+
93+
destinationList = sourceList.cloneNode(false);
94+
destinationList.removeAttributeNS(xmlns, "id");
95+
96+
// create a range starting at before the list item element we split at and
97+
// ending at just after the last list item in the list
98+
range.setStartBefore(splitParagraph.parentNode);
99+
range.setEndAfter(sourceList.lastElementChild);
100+
101+
// extract the range to get all list items from the split position to the end of the list
102+
// then collapse any empty nodes at the parent text:list element of the paragraph at the split position
103+
splitPositionParentList = /**@type{!Node}*/(splitParagraph.parentNode.parentNode);
104+
fragment = range.extractContents();
105+
106+
// don't collapse nodes if its the top level list
107+
if (splitPositionParentList !== sourceList) {
108+
collapseRules.mergeChildrenIntoParent(splitPositionParentList);
109+
}
110+
111+
destinationList.appendChild(fragment);
112+
sourceList.parentNode.insertBefore(destinationList, sourceList.nextElementSibling);
113+
return true;
114+
};
115+
};
116+
117+
/**@typedef{{
118+
optype: !string,
119+
memberid: !string,
120+
timestamp: !number,
121+
sourceStartPosition: !number,
122+
splitPosition: !number
123+
}}*/
124+
ops.OpSplitList.Spec;
125+
126+
/**@typedef{{
127+
memberid: !string,
128+
timestamp:(number|undefined),
129+
sourceStartPosition: !number,
130+
splitPosition: !number
131+
}}*/
132+
ops.OpSplitList.InitSpec;

‎webodf/lib/ops/OperationFactory.js‎

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,9 @@ ops.OperationFactory = function OperationFactory() {
103103
RemoveHyperlink: construct(ops.OpRemoveHyperlink),
104104
AddList: construct(ops.OpAddList),
105105
RemoveList: construct(ops.OpRemoveList),
106-
AddListStyle: construct(ops.OpAddListStyle)
106+
AddListStyle: construct(ops.OpAddListStyle),
107+
MergeList: construct(ops.OpMergeList),
108+
SplitList: construct(ops.OpSplitList)
107109
};
108110
}
109111

‎webodf/tests/ops/operationtests.xml‎

Lines changed: 62 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2409,4 +2409,66 @@
24092409
</office:text>
24102410
</after>
24112411
</test>
2412+
<test name="MergeList_Simple">
2413+
<before>
2414+
<office:text>
2415+
<text:list>
2416+
<text:list-item>
2417+
<text:p>SampleText1</text:p>
2418+
</text:list-item>
2419+
</text:list>
2420+
<text:list>
2421+
<text:list-item>
2422+
<text:p>SampleText2</text:p>
2423+
</text:list-item>
2424+
</text:list>
2425+
</office:text>
2426+
</before>
2427+
<ops>
2428+
<op optype="MergeList" destinationStartPosition="0" sourceStartPosition="12"/>
2429+
</ops>
2430+
<after>
2431+
<office:text>
2432+
<text:list>
2433+
<text:list-item>
2434+
<text:p>SampleText1</text:p>
2435+
</text:list-item>
2436+
<text:list-item>
2437+
<text:p>SampleText2</text:p>
2438+
</text:list-item>
2439+
</text:list>
2440+
</office:text>
2441+
</after>
2442+
</test>
2443+
<test name="SplitList_Simple">
2444+
<before>
2445+
<office:text>
2446+
<text:list>
2447+
<text:list-item>
2448+
<text:p>SampleText1</text:p>
2449+
</text:list-item>
2450+
<text:list-item>
2451+
<text:p>SampleText2</text:p>
2452+
</text:list-item>
2453+
</text:list>
2454+
</office:text>
2455+
</before>
2456+
<ops>
2457+
<op optype="SplitList" sourceStartPosition="0" splitPosition="12"/>
2458+
</ops>
2459+
<after>
2460+
<office:text>
2461+
<text:list>
2462+
<text:list-item>
2463+
<text:p>SampleText1</text:p>
2464+
</text:list-item>
2465+
</text:list>
2466+
<text:list>
2467+
<text:list-item>
2468+
<text:p>SampleText2</text:p>
2469+
</text:list-item>
2470+
</text:list>
2471+
</office:text>
2472+
</after>
2473+
</test>
24122474
</tests>

‎webodf/tools/karma.conf.js‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -93,6 +93,7 @@ module.exports = function (config) {
9393
'lib/ops/OpInsertImage.js',
9494
'lib/ops/OpInsertTable.js',
9595
'lib/ops/OpInsertText.js',
96+
'lib/ops/OpMergeList.js',
9697
'lib/odf/CollapsingRules.js',
9798
'lib/ops/OpMergeParagraph.js',
9899
'lib/ops/OpMoveCursor.js',
@@ -106,6 +107,7 @@ module.exports = function (config) {
106107
'lib/ops/OpRemoveText.js',
107108
'lib/ops/OpSetBlob.js',
108109
'lib/ops/OpSetParagraphStyle.js',
110+
'lib/ops/OpSplitList.js',
109111
'lib/ops/OpSplitParagraph.js',
110112
'lib/ops/OpUpdateMember.js',
111113
'lib/ops/OpUpdateMetadata.js',

0 commit comments

Comments
 (0)