r98786 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:< r98785‎ | r98786 | r98787 >
Date:17:56, 3 October 2011
Author:tparscal
Status:deferred
Tags:
Comment:
Added basic model classes
Modified paths:
  • /trunk/parsers/wikidom/lib/hype/models/es.DocumentModel.js (modified) (history)
  • /trunk/parsers/wikidom/lib/hype/models/es.ListItemModel.js (added) (history)
  • /trunk/parsers/wikidom/lib/hype/models/es.ListModel.js (added) (history)
  • /trunk/parsers/wikidom/lib/hype/models/es.ParagraphModel.js (modified) (history)
  • /trunk/parsers/wikidom/lib/hype/models/es.TableCellModel.js (added) (history)
  • /trunk/parsers/wikidom/lib/hype/models/es.TableModel.js (added) (history)
  • /trunk/parsers/wikidom/lib/hype/models/es.TableRowModel.js (added) (history)
  • /trunk/parsers/wikidom/tests/hype/es.DocumentModel.test.js (modified) (history)
  • /trunk/parsers/wikidom/tests/hype/index.html (modified) (history)

Diff [purge]

Index: trunk/parsers/wikidom/tests/hype/index.html
——@@ -13,12 +13,17 @@
1414 <script src="../../lib/jquery.js"></script>
1515 <script src="../../lib/qunit.js"></script>
1616 <script src="../../lib/hype/es.js"></script>
17 - <script src="../../lib/synth/bases/es.AggregateArray.js"></script>
1817 <script src="../../lib/hype/bases/es.EventEmitter.js"></script>
1918 <script src="../../lib/hype/bases/es.ModelNode.js"></script>
2019 <script src="../../lib/hype/bases/es.ViewNode.js"></script>
2120 <script src="../../lib/hype/bases/es.DocumentModelNode.js"></script>
2221 <script src="../../lib/hype/models/es.DocumentModel.js"></script>
 22+ <script src="../../lib/hype/models/es.ListItemModel.js"></script>
 23+ <script src="../../lib/hype/models/es.ListModel.js"></script>
 24+ <script src="../../lib/hype/models/es.ParagraphModel.js"></script>
 25+ <script src="../../lib/hype/models/es.TableCellModel.js"></script>
 26+ <script src="../../lib/hype/models/es.TableModel.js"></script>
 27+ <script src="../../lib/hype/models/es.TableRowModel.js"></script>
2328 <script src="es.ModelNode.test.js"></script>
2429 <script src="es.DocumentModel.test.js"></script>
2530 </body>
Index: trunk/parsers/wikidom/tests/hype/es.DocumentModel.test.js
——@@ -44,10 +44,10 @@
4545 'type': 'table',
4646 'children': [
4747 {
48 - 'type': 'row',
 48+ 'type': 'tableRow',
4949 'children': [
5050 {
51 - 'type': 'cell',
 51+ 'type': 'tableCell',
5252 'children': [
5353 {
5454 'type': 'paragraph',
——@@ -132,9 +132,9 @@
133133 // 5 - Beginning of table
134134 { 'type': 'table' },
135135 // 6 - Beginning of row
136 - { 'type': 'row' },
 136+ { 'type': 'tableRow' },
137137 // 7 - Beginning of cell
138 - { 'type': 'cell' },
 138+ { 'type': 'tableCell' },
139139 // 8 - Beginning of paragraph
140140 { 'type': 'paragraph' },
141141 // 9 - Plain content
——@@ -164,9 +164,9 @@
165165 // 21 - End of list
166166 { 'type': '/list' },
167167 // 22 - End of cell
168 - { 'type': '/cell' },
 168+ { 'type': '/tableCell' },
169169 // 23 - End of row
170 - { 'type': '/row' },
 170+ { 'type': '/tableRow' },
171171 // 24 - End of table
172172 { 'type': '/table' },
173173 // 25 - Beginning of paragraph
——@@ -177,7 +177,7 @@
178178 { 'type': '/paragraph' }
179179 ];
180180
181 -test( 'es.ModelNode', function() {
 181+test( 'es.DocumentModel', function() {
182182 var documentModel = es.DocumentModel.newFromPlainObject( obj );
183183
184184 deepEqual( documentModel.getData(), data, 'Flattening plain objects results in correct data' );
Index: trunk/parsers/wikidom/lib/hype/models/es.DocumentModel.js
——@@ -18,6 +18,13 @@
1919 this.attributes = $.isPlainObject( attributes ) ? attributes : {};
2020 };
2121
 22+/* Static Members */
 23+
 24+/**
 25+ * Mapping of symbolic names and node model constructors.
 26+ */
 27+es.DocumentModel.nodeModels = {};
 28+
2229 /* Static Methods */
2330
2431 /**
Index: trunk/parsers/wikidom/lib/hype/models/es.TableRowModel.js
——@@ -0,0 +1,18 @@
 2+/**
 3+ * Creates an es.TableRowModel object.
 4+ *
 5+ * @class
 6+ * @constructor
 7+ */
 8+es.TableRowModel = function( length ) {
 9+ // Inheritance
 10+ es.DocumentModelNode.call( this, length );
 11+};
 12+
 13+/* Registration */
 14+
 15+es.DocumentModel.nodeModels.tableRow = es.TableRowModel;
 16+
 17+/* Inheritance */
 18+
 19+es.extend( es.TableRowModel, es.DocumentModelNode );
Index: trunk/parsers/wikidom/lib/hype/models/es.ParagraphModel.js
——@@ -9,6 +9,10 @@
1010 es.DocumentModelNode.call( this, length );
1111 };
1212
 13+/* Registration */
 14+
 15+es.DocumentModel.nodeModels.paragraph = es.ParagraphModel;
 16+
1317 /* Inheritance */
1418
1519 es.extend( es.ParagraphModel, es.DocumentModelNode );
Index: trunk/parsers/wikidom/lib/hype/models/es.TableCellModel.js
——@@ -0,0 +1,18 @@
 2+/**
 3+ * Creates an es.TableCellModel object.
 4+ *
 5+ * @class
 6+ * @constructor
 7+ */
 8+es.TableCellModel = function( length ) {
 9+ // Inheritance
 10+ es.DocumentModelNode.call( this, length );
 11+};
 12+
 13+/* Registration */
 14+
 15+es.DocumentModel.nodeModels.tableCell = es.TableCellModel;
 16+
 17+/* Inheritance */
 18+
 19+es.extend( es.TableCellModel, es.DocumentModelNode );
Index: trunk/parsers/wikidom/lib/hype/models/es.TableModel.js
——@@ -0,0 +1,18 @@
 2+/**
 3+ * Creates an es.TableModel object.
 4+ *
 5+ * @class
 6+ * @constructor
 7+ */
 8+es.TableModel = function( length ) {
 9+ // Inheritance
 10+ es.DocumentModelNode.call( this, length );
 11+};
 12+
 13+/* Registration */
 14+
 15+es.DocumentModel.nodeModels.table = es.TableModel;
 16+
 17+/* Inheritance */
 18+
 19+es.extend( es.TableModel, es.DocumentModelNode );
Index: trunk/parsers/wikidom/lib/hype/models/es.ListItemModel.js
——@@ -0,0 +1,18 @@
 2+/**
 3+ * Creates an es.ListItemModel object.
 4+ *
 5+ * @class
 6+ * @constructor
 7+ */
 8+es.ListItemModel = function( length ) {
 9+ // Inheritance
 10+ es.DocumentModelNode.call( this, length );
 11+};
 12+
 13+/* Registration */
 14+
 15+es.DocumentModel.nodeModels.listItem = es.ListItemModel;
 16+
 17+/* Inheritance */
 18+
 19+es.extend( es.ListItemModel, es.DocumentModelNode );
Index: trunk/parsers/wikidom/lib/hype/models/es.ListModel.js
——@@ -0,0 +1,18 @@
 2+/**
 3+ * Creates an es.ListModel object.
 4+ *
 5+ * @class
 6+ * @constructor
 7+ */
 8+es.ListModel = function( length ) {
 9+ // Inheritance
 10+ es.DocumentModelNode.call( this, length );
 11+};
 12+
 13+/* Registration */
 14+
 15+es.DocumentModel.nodeModels.list = es.listModel;
 16+
 17+/* Inheritance */
 18+
 19+es.extend( es.ListModel, es.DocumentModelNode );

Status & tagging log