From de64aaf96791b0e4adc5e1d35b6234cd96ea64f4 Mon Sep 17 00:00:00 2001
From: Kevin Schroeder <kschroeder@mirageworks.com>
Date: Thu, 25 May 2017 11:24:20 -0500
Subject: [PATCH] Updated to better handle permittedValues in <select> elements

---
 lib/View/Controllers/View.php | 2 +-
 tests/View/ViewTest.php       | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/lib/View/Controllers/View.php b/lib/View/Controllers/View.php
index 8bd32a6..39a619b 100644
--- a/lib/View/Controllers/View.php
+++ b/lib/View/Controllers/View.php
@@ -137,7 +137,7 @@ protected function getSourceData(\SimpleXMLElement $node)
         } else if (isset($node->permittedValues)) {
             $source = [];
             foreach ($node->permittedValues->children() as $value) {
-                $source[] = (string)$value;
+                $source[(string)$value] = (string)$value;
             }
             return $source;
         }
diff --git a/tests/View/ViewTest.php b/tests/View/ViewTest.php
index c4889e6..b0b6e24 100644
--- a/tests/View/ViewTest.php
+++ b/tests/View/ViewTest.php
@@ -73,8 +73,8 @@ public function testViewRendering()
         self::assertXpathExists($simpleXml, '//input[@type="text" and @name="section1_test_element"]'); // we do this separately to test if the element exists apart from whether it has a value.
         self::assertXpathExists($simpleXml, '//input[@type="text" and @name="section1_test_element" and @value="Element Value"]');
         self::assertXpathExists($simpleXml, '//select[@name="section1_test_permitted"]');
-        self::assertXpathExists($simpleXml, '//select[@name="section1_test_permitted"]/option[.="yes"]');
-        self::assertXpathExists($simpleXml, '//select[@name="section1_test_permitted"]/option[.="no"]');
+        self::assertXpathExists($simpleXml, '//select[@name="section1_test_permitted"]/option[.="yes" and @value="yes"]');
+        self::assertXpathExists($simpleXml, '//select[@name="section1_test_permitted"]/option[.="no" and @value="no"]');
     }
 
     public function getViewConfiguration(array $params = [], $method = 'GET')