Gallica : validation XML test 11

library/Class/WebService/OAI/Request/ListIdentifiers.php

@@ -48,12 +48,17 @@ class Class_WebService_OAI_Request_ListIdentifiers {
 		$this->_until = $this->_params['until'];
 
 		$this->_resumptionToken = $this->_params['resumptionToken'];
+
 		if ($this->_resumptionToken 
 				&& ($token = Class_WebService_OAI_ResumptionToken::find($this->_resumptionToken))) {
 			$this->_set = $token->getParam('set');
 			$this->_from = $token->getParam('from');
 			$this->_until = $token->getParam('until');
 		}
+		if (!strtotime($this->_until))
+				$this->_until = null;
+		if (!strtotime($this->_from))
+			$this->_from = null;
 
 		$this->_catalogue = $this->getCatalogueFromSetSpec($this->_set);
 	}

tests/application/modules/opac/controllers/OAIControllerListIdentifiersTest.php

@@ -298,6 +298,16 @@ class OAIControllerListIdentifiersInvalidParamsTest extends AbstractControllerTe
 		$this->_xpath->assertXPath($this->_response->getBody(),
 															 '//oai:error[@code="badResumptionToken"]');
 	}
+
+	/** @test */
+	public function withBadFromAndUntilResponseShouldNotContainsAttributes() {
+		$this->dispatch('/opac/oai/request?verb=ListIdentifiers&metadataPrefix=oai_dc&set=really_wrong_set&from=some_random_date&until=some_random_date');
+		$this->_xpath->assertXPath($this->_response->getBody(), 
+																	'//oai:request[not(@from)][not(@until)]');
+		
+	}
+
+
 }