From 7053ca01e6ae270c88ae7f559ee3c04d4bb2ad6f Mon Sep 17 00:00:00 2001
From: John Hann <john@e-numera.com>
Date: Thu, 9 Feb 2012 14:16:38 -0500
Subject: [PATCH] wire/dom/render as a plugin

---
 dom/render.js        | 46 ++++++++++++++++++++---
 test/dom-render.html | 88 +++++++++++++++++++++++++++++++++-----------
 2 files changed, 106 insertions(+), 28 deletions(-)

diff --git a/dom/render.js b/dom/render.js
index da4f8a6..4d49e42 100644
--- a/dom/render.js
+++ b/dom/render.js
@@ -11,9 +11,9 @@
  * http://www.opensource.org/licenses/mit-license.php
  */
 
-define(['../domReady'], function(domReady) {
+define(['when', '../domReady'], function(when, domReady) {
 
-	var parentTypes, parseTemplateRx, getFirstTagNameRx;
+	var parentTypes, parseTemplateRx, getFirstTagNameRx, undef;
 
 	parentTypes = {
 		'td': 'tr',
@@ -53,14 +53,48 @@ define(['../domReady'], function(domReady) {
 		return node;
 	}
 
-	render.wire$plugin = {
-		factories: {
-			render: render
-		}
+	render.wire$plugin = function (ready, destroyed, options) {
+		return {
+			factories: {
+				render: domRenderFactory
+			}
+		};
 	};
 
 	return render;
 
+
+	/**
+	 * Creates rendered dom trees for the "render" factory.
+	 * @param resolver
+	 * @param spec
+	 * @param wire
+	 */
+	function domRenderFactory(resolver, spec, wire) {
+		var parentRef, options;
+
+		options = spec.render;
+
+		domReady(function() {
+
+			var futureTemplate, futureMixin, futureRoot;
+
+			// get args from spec
+			futureTemplate = options.template ? wire(options.template) : '';
+			if (options.mixin) {
+				futureMixin = wire(options.mixin);
+			}
+			if (options.at) {
+				futureRoot = wire(options.at);
+			}
+
+			when.all([futureTemplate, futureMixin, futureRoot], function (args) {
+				return render.apply(undef, args);
+			}).then(resolver.resolve, resolver.reject);
+
+		});
+	}
+
 	/**
 	 * Finds the first html element in a string, extracts its tag name,
 	 * and looks up the natural parent element tag name for this element.
diff --git a/test/dom-render.html b/test/dom-render.html
index e0f6e05..ae21811 100644
--- a/test/dom-render.html
+++ b/test/dom-render.html
@@ -9,13 +9,18 @@
 
     <script type="text/javascript">
 
-		var simpleText = 'this is a simple template';
+		var simpleText = 'this is a simple template',
+			mixin = {
+				foo: 'foo',
+				bar: 'bar'
+			};
 
 		define('template-simple', '<div>' + simpleText + '</div>');
 		define('template-with-root-attributes', '<div class="foo bar" role="menuitem" aria-haspopup="true">this element has attributes</div>');
 		define('template-with-children', '<div>top-level element <code>child element</code></div>');
-		define('template-with-strict-parent', '<li>this is a list item</li>')
+		define('template-with-strict-parent', '<li>this is a list item</li>');
 		define('template-with-complexity', '<div class="foo bar" role="menuitem" aria-haspopup="true">top-level element<span>child element</span></div>');
+		define('template-from-plugin', '<div>top-level element from plugin<code>child element</code> with mixin foo=${foo}</div>');
 
         require(['wire'], function(wire) {
 
@@ -23,6 +28,7 @@
 
 			wired = wire({
 				plugins: [
+					{ module: 'wire/debug' },
 					{ module: 'wire/dom' },
 					{ module: 'wire/dom/render' }
 				],
@@ -56,15 +62,22 @@
 						]
 					}
 				},
-				node5: {
+				node4: {
 					create: {
 						module: 'wire/dom/render',
 						args: [
 							{ module: 'template-with-strict-parent' },
 							{},
-							{ $ref: 'dom!node5' }
+							{ $ref: 'dom!node4' }
 						]
 					}
+				},
+				node5: {
+					render: {
+						template: { module: 'template-from-plugin' },
+						mixin: mixin,
+						at: { $ref: 'dom!node5' }
+					}
 				}
 			});
 
@@ -91,7 +104,7 @@
                     wired.then(
                         function(context) {
 							// test doesn't need to do much. either we get here or we don't!
-							dohd.callback('ol' == context.node5.tagName);
+							dohd.callback('ol' == context.node4.tagName);
                         },
                         function(e) {
                             dohd.errback(e);
@@ -140,16 +153,6 @@
 						}
 					);
 					return dohd;
-
-					function hasSameItems(arr1, arr2) {
-						var i, sorted
-						if (!arr1 || !arr2 || arr1.length != arr2.length) return false;
-						sorted = [arr1.sort(), arr2.sort()];
-						for (i = 0; i < sorted[0].length; i++) {
-							if (sorted[0] != sorted[1]) return false;
-						}
-						return true;
-					}
 				}
             ]);
 
@@ -166,19 +169,59 @@
 						}
 					);
 					return dohd;
+				}
+
+			]);
 
-					function hasChildWithTagName(node, tagName) {
-						for (var i = 0; i < node.childNodes; i++) {
-							if (node.childNodes[i].tagName == tagName) return true;
+			doh.register('template-from-plugin', [
+				function hasContent(doh) {
+					var dohd = new doh.Deferred();
+					dfds.push(dohd);
+					wired.then(
+						function(context) {
+							dohd.callback(hasChildWithTagName(context.node5.innerHTML, 'code'));
+						},
+						function(e) {
+							dohd.errback(e);
 						}
-						return false;
-					}
+					);
+					return dohd;
+				},
+				function replacedToken(doh) {
+					var dohd = new doh.Deferred();
+					dfds.push(dohd);
+					wired.then(
+						function(context) {
+							dohd.callback(/foo=foo/.test(context.node5.innerHTML));
+						},
+						function(e) {
+							dohd.errback(e);
+						}
+					);
+					return dohd;
 				}
-
 			]);
 
             doh.run();
         });
+
+		function hasSameItems(arr1, arr2) {
+			var i, sorted
+			if (!arr1 || !arr2 || arr1.length != arr2.length) return false;
+			sorted = [arr1.sort(), arr2.sort()];
+			for (i = 0; i < sorted[0].length; i++) {
+				if (sorted[0] != sorted[1]) return false;
+			}
+			return true;
+		}
+
+		function hasChildWithTagName(node, tagName) {
+			for (var i = 0; i < node.childNodes; i++) {
+				if (node.childNodes[i].tagName == tagName) return true;
+			}
+			return false;
+		}
+
     </script>
 </head>
 <body>
@@ -186,7 +229,8 @@
     <p id="node1" class="orig1"></p>
     <p id="node2" class="orig2"></p>
     <p id="node3" class="orig3"></p>
-    <ol id="node5" class="orig5"></ol>
+    <ol id="node4" class="orig4"></ol>
+	<p id="node5" class="orig5"></p>
 </div>
 </body>
 </html>
\ No newline at end of file