Correct spelling on index plural...

Author: cjsaylor

README.md

@@ -6,7 +6,7 @@ A library to enable you to safely rotate indexes with no downtime to end users.
 
 ### Why would I use this?
 
-In many situations, Elasticsearch is used as an ephemeral datastore used to take structured or relational data, and make it fast to search on that data. Often this is achieved via scheduled jobs that read data from a permanent datastore (such as MySQL or Postgres) and translate it into an Elasticsearch index.
+In many situations, Elasticsearch is used as an ephemeral datastore used to take structured or relational data and make it fast to search on that data. Often this is achieved via scheduled jobs that read data from a permanent datastore (such as MySQL or Postgres) and translate it into an Elasticsearch index.
 
 In many cases, rebuilding an index requires a clean slate so that the entire index is rebuilt. How do you do this without interrupting end users searching on that index? The answer is a rotating index.
 
@@ -51,7 +51,7 @@ $newlyBuiltIndexName = $this->buildIndex($client);
 $indexRotator->copyPrimaryIndexToSecondary();
 $indexRotator->setPrimaryIndex($newlyBuiltIndexName);
 // optionally remove the old index right now
-$indexRotator->deleteSecondaryIndexes();
+$indexRotator->deleteSecondaryIndices();
 ```
 
 #### All together
@@ -85,7 +85,7 @@ class MySearchIndex {
 		$indexRotator->copyPrimaryIndexToSecondary();
 		$indexRotator->setPrimaryIndex($newlyBuiltIndexName);
 		// optionally remove the old index right now
-		$indexRotator->deleteSecondaryIndexes();
+		$indexRotator->deleteSecondaryIndices();
 	}
 
 	private function buildIndex(\Elasticsearch\Client $client) {

src/IndexRotator.php

@@ -160,7 +160,7 @@ public function copyPrimaryIndexToSecondary($retryCount = 0)
 	 * @param string $olderThan
 	 * @return array
 	 */
-	public function getSecondaryIndexes(\DateTime $olderThan = null)
+	public function getSecondaryIndices(\DateTime $olderThan = null)
 	{
 		if ($olderThan === null) {
 			$olderThan = new \DateTime();
@@ -213,10 +213,10 @@ public function getSecondaryIndexes(\DateTime $olderThan = null)
 	 * @param \DateTime $olderThan
 	 * @return array Results of the bulk operation.
 	 */
-	public function deleteSecondaryIndexes(\DateTime $olderThan = null)
+	public function deleteSecondaryIndices(\DateTime $olderThan = null)
 	{
 		$results = [];
-		foreach ($this->getSecondaryIndexes($olderThan) as $indexToDelete) {
+		foreach ($this->getSecondaryIndices($olderThan) as $indexToDelete) {
 			if ($this->engine->indices()->exists(['index' => $indexToDelete])) {
 				$results[$indexToDelete] = $this->engine->indices()->delete(['index' => $indexToDelete]);
 				$this->logger->debug('Deleted secondary index.', compact('indexToDelete'));

tests/IndexRotatorTest.php

@@ -94,15 +94,15 @@ public function testCopyPrimaryIndexToSecondary()
 	 * @dataProvider secondaryIndexConditionProvider
 	 */
 	public function testGetSecondaryIndices($olderThan, $expectedIndices) {
-		$results = $this->indexRotator->getSecondaryIndexes($olderThan);
+		$results = $this->indexRotator->getSecondaryIndices($olderThan);
 		$this->assertEmpty(array_diff($results, $expectedIndices));
 	}
 
 	/**
 	 * @dataProvider secondaryIndexConditionProvider
 	 */
 	public function testDeleteSecondaryIndices($olderThan, $expectedToDelete) {
-		$results = $this->indexRotator->deleteSecondaryIndexes($olderThan);
+		$results = $this->indexRotator->deleteSecondaryIndices($olderThan);
 		$this->assertEmpty(array_diff(array_keys($results), $expectedToDelete));
 		foreach ($results as $result) {
 			$this->assertEquals(['acknowledged' => true], $result);