From b19d129ec2e97d94f48e2acafefe3b056259df57 Mon Sep 17 00:00:00 2001
From: Gabriel Zerbib <gabriel.zerbib@gmail.com>
Date: Sat, 28 Nov 2015 20:33:55 +0200
Subject: [PATCH] doctype

---
 .../example1-getstarted/template-main.xml     |  9 ++----
 src/figdice/classes/ViewElementTag.php        | 31 +++++++++++++++++--
 test/ViewParserTest.php                       | 19 ++++++++++++
 3 files changed, 50 insertions(+), 9 deletions(-)
diff --git a/examples/example1-getstarted/template-main.xml b/examples/example1-getstarted/template-main.xml
index 7bc270f..ee37b62 100644
--- a/examples/example1-getstarted/template-main.xml
+++ b/examples/example1-getstarted/template-main.xml
@@ -1,11 +1,8 @@
-<fig:template xmlns:fig="http://figdice.org">
+<fig:template xmlns:fig="http://figdice.org" fig:doctype="html">
 <!-- Let's make a "fig:*" Root node, so as to mute it automatically from the output. -->
+<!-- the root node can carry the fig:doctype attribute, to instruct the engine to
+     generate a <!DOCTYPE > directive. -->
 
-<!-- Let's use a CDATA sequence in our template, to
-     produce the DOCTYPE element in the output -->
-<![CDATA[
-<!DOCTYPE html>
-]]>
 <html lang="en">
 	<head>
 		<!-- You can use any expression, including 'paths and symbols' in the
diff --git a/src/figdice/classes/ViewElementTag.php b/src/figdice/classes/ViewElementTag.php
index b8f7fde..40e6bf5 100644
--- a/src/figdice/classes/ViewElementTag.php
+++ b/src/figdice/classes/ViewElementTag.php
@@ -155,6 +155,18 @@ public function setAttribute($name, $value) {
 	public function hasAttribute($name) {
 		return array_key_exists($name, $this->attributes);
 	}
+
+	/**
+	 * Indicates whether the current tag carries the specified attribute within
+	 * the fig: namespace (where "fig:" is soft-coded according the xmlns:fig).
+	 * @param $name
+	 * @return bool
+	 */
+	private function hasFigAttribute($name)
+	{
+		return $this->hasAttribute($this->view->figNamespace . $name);
+	}
+
 	public function appendChild(ViewElement & $child) {
 		if(0 < count($this->children)) {
 			$this->children[count($this->children) - 1]->nextSibling = & $child;
@@ -329,6 +341,8 @@ private function renderNoMacro($bypassWalk = false) {
 		//so that its immediate children can run the fig:case attribute form fresh.
 		$this->caseSwitched = false;
 
+
+
 		//================================================================
 		//fig:cond
 		//(fig:condition is deprecated)
@@ -417,7 +431,7 @@ private function renderNoMacro($bypassWalk = false) {
         $this->parent->runtimeAttributes[$this->attributes['name']] = new Flag();
       }
       else {
-        if (isset($this->attributes['value'])) {
+        if ($this->hasAttribute('value')) {
           $value = $this->evaluate($this->attributes['value']);
           if (is_string($value)) {
             $value = htmlspecialchars($value);
@@ -450,7 +464,7 @@ private function renderNoMacro($bypassWalk = false) {
 		//fig:walk
 		//Loop over evaluated dataset.
 		if(! $bypassWalk) {
-			if($this->hasAttribute($this->view->figNamespace . 'walk')) {
+			if($this->hasFigAttribute('walk')) {
 				return $this->fig_walk();
 			}
 		}
@@ -459,7 +473,7 @@ private function renderNoMacro($bypassWalk = false) {
 		//fig:call
 		//A tag with the fig:call directive is necessarily mute.
 		//It is used as a placeholder only for the directive.
-		if(isset($this->attributes[$this->view->figNamespace . 'call'])) {
+		if($this->hasFigAttribute('call')) {
 			return $this->fig_call();
 		}
 
@@ -663,6 +677,14 @@ private function renderNoMacro($bypassWalk = false) {
 			$this->runtimeAttributes = array();
 		}
 
+
+		//================================================================
+		// If we're dealing with rootnode, trean doctype directive
+		// fig:doctype
+		if ((null == $this->parent) && ($this->hasFigAttribute('doctype'))) {
+			$result = '<!doctype ' . $this->getFigAttribute('doctype') . '>' . PHP_EOL . $result;
+		}
+
 		return $result;
 	}
 
@@ -728,6 +750,9 @@ public function getAttribute($name, $default = null) {
 		}
 		return $default;
 	}
+	private function getFigAttribute($name, $default = null) {
+		return $this->getAttribute($this->view->figNamespace . $name, $default);
+	}
 
 	private function fig_mount() {
 		$target = $this->getAttribute('target');
diff --git a/test/ViewParserTest.php b/test/ViewParserTest.php
index f8eb23f..e78ed21 100755
--- a/test/ViewParserTest.php
+++ b/test/ViewParserTest.php
@@ -225,4 +225,23 @@ public function testWalkOnNonCountableObjectRunsOnArrayWithObject()
     $view->mount('obj', $obj);
     $this->assertEquals('<test>12</test>', $view->render());
   }
+
+  public function testDoctype()
+  {
+    $view = new View();
+    $templateSource = <<<ENDXML
+<tpl:template xmlns:tpl="http://figdice.org" tpl:doctype="html">
+<html></html>
+</tpl:template>
+ENDXML;
+    $view->loadString($templateSource);
+
+    $expected = <<<EXPECTED
+<!doctype html>
+<html></html>
+
+EXPECTED;
+
+    $this->assertEquals($expected, $view->render());
+  }
 }
