Merge pull request #1423 from NatLibFi/fix-datetime-to-string-in-cont…

…ainshtml fix conversion of DateTime to string in getContainsHtml
NatLibFi · Mar 22, 2023 · 7d7b1bb · 7d7b1bb
2 parents a59f251 + bc490ba
commit 7d7b1bb
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/model/ConceptPropertyValueLiteral.php b/model/ConceptPropertyValueLiteral.php
@@ -57,7 +57,7 @@ public function getType()
     }
 
     public function getContainsHtml() {
-        return preg_match("/\/[a-z]*>/i", $this->literal->getValue()) != 0;
+        return preg_match("/\/[a-z]*>/i", $this->getLabel()) != 0;
     }
 
     public function getLabel()

diff --git a/tests/ConceptPropertyValueLiteralTest.php b/tests/ConceptPropertyValueLiteralTest.php
@@ -173,6 +173,17 @@ public function testGetContainsHtmlWhenThereIsOnlyAOpeningTag() {
     $this->assertFalse($lit->getContainsHtml());
   }
 
+  /**
+   * @covers ConceptPropertyValueLiteral::getContainsHtml
+   */
+  public function testGetContainsHtmlWhenValueIsDate() {
+    $litmock = $this->getMockBuilder('EasyRdf\Literal\Date')->disableOriginalConstructor()->getMock();
+    $litmock->method('getValue')->will($this->returnValue(DateTime::createFromFormat('Y-m-d', '2013-10-14')));
+    $resmock = $this->getMockBuilder('EasyRdf\Resource')->disableOriginalConstructor()->getMock();
+    $lit = new ConceptPropertyValueLiteral($this->model, $this->vocab, $resmock, $litmock, 'skosmos:testType');
+    $this->assertFalse($lit->getContainsHtml());
+  }
+
   /**
    * @covers ConceptPropertyValueLiteral::getContainsHtml
    */