Merge pull request zendframework/zendframework#5936 from god107/zf4267

Fix for issue zendframework/zendframework#4267

src/Reader/Reader.php

@@ -299,6 +299,11 @@ public static function importRemoteFeed($uri, Http\ClientInterface $client)
      */
     public static function importString($string)
     {
+        $trimmed = trim($string);
+        if (!is_string($string) || empty($trimmed)) {
+            throw new Exception\InvalidArgumentException('Only non empty strings are allowed as input');
+        }
+
         $libxmlErrflag = libxml_use_internal_errors(true);
         $oldValue = libxml_disable_entity_loader(true);
         $dom = new DOMDocument;

test/Reader/ReaderTest.php

@@ -302,6 +302,13 @@ public function testImportRemoteFeedMethodPerformsAsExpected()
         $this->assertEquals(Reader\Reader::TYPE_RSS_20, $type);
     }
 
+    public function testImportStringMethodThrowProperExceptionOnEmptyString()
+    {
+        $this->setExpectedException('Zend\Feed\Reader\Exception\InvalidArgumentException');
+        $string = ' ';
+        $feed = Reader\Reader::importString($string);
+    }
+
     protected function _getTempDirectory()
     {
         $tmpdir = array();