Merge pull request #50 from bopoda/master_issue48

Directive with '+' not handled correctly, escape '+', resolve #48

RobotsTxtValidator.php

@@ -192,6 +192,7 @@ private static function prepareRegexpRule($ruleValue)
 			'/\$./' => '\$',
 			'/\?/' => '\?',
 			'/\./' => '\.',
+			'/\+/' => '\+',
 			'/\*/' => '.*',
 		);
 

phpunit.xml

@@ -23,6 +23,7 @@
 			<file>tests/t1gorTest.php</file>
 			<file>tests/MultipleUserAgentsRulesTest.php</file>
 			<file>tests/Issue13Test.php</file>
+			<file>tests/Issue48Test.php</file>
 		</testsuite>
 	</testsuites>
 </phpunit>

tests/Issue48Test.php

@@ -0,0 +1,26 @@
+
+<?php
+
+class Issue48Test extends \PHPUnit\Framework\TestCase
+{
+	/**
+	 * Load library
+	 */
+	public static function setUpBeforeClass()
+	{
+		require_once(realpath(__DIR__.'/../RobotsTxtParser.php'));
+		require_once(realpath(__DIR__.'/../RobotsTxtValidator.php'));
+	}
+
+	public function testIsUrlAllowWithPlus()
+	{
+		$robotsTxtContentIssue = "
+User-agent: *
+Disallow: *telecommande++*
+";
+		$robotsTxtParser = new RobotsTxtParser($robotsTxtContentIssue);
+		$robotsTxtValidator = new RobotsTxtValidator($robotsTxtParser->getRules());
+
+		$this->assertTrue($robotsTxtValidator->isUrlAllow('/telecommandes-box-decodeur.html'));
+	}
+}