Skip to content

Commit

Permalink
Index mapping field of type 'string' has been removed
Browse files Browse the repository at this point in the history
  • Loading branch information
p365labs committed Aug 17, 2017
1 parent 2418e03 commit e07c520
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 7 deletions.
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ All notable changes to this project will be documented in this file based on the
- Replace IndexAlreadyExistsException with [ResourceAlreadyExistsException](https://github.com/elastic/elasticsearch/pull/21494) [#1350](https://github.com/ruflin/Elastica/pull/1350)
- in order to delete an index you should not delete by its alias now you should delete using the [concrete index name](https://github.com/elastic/elasticsearch/blob/6.0/core/src/test/java/org/elasticsearch/aliases/IndexAliasesIT.java#L445) [#1348](https://github.com/ruflin/Elastica/pull/1348)
- Removed ```optimize``` from Index class as it has been deprecated in ES 2.1 and removed in [ES 5.x+](https://www.elastic.co/guide/en/elasticsearch/reference/2.4/indices-optimize.html) use forcemerge [#1351](https://github.com/ruflin/Elastica/pull/1350)
- Index mapping field of type [*'string'*](https://www.elastic.co/guide/en/elasticsearch/reference/5.5/string.html) has been removed from Elasticsearch 6.0 codebase [#1353](https://github.com/ruflin/Elastica/pull/1353)

### Bugfixes
- Enforce [Content-Type requirement on the layer Rest](https://github.com/elastic/elasticsearch/pull/23146), a [PR on Elastica #1301](https://github.com/ruflin/Elastica/issues/1301) solved it (it has been implemented only in the HTTP Transport), but it was not implemented in the Guzzle Transport. [#1349](https://github.com/ruflin/Elastica/pull/1349)
Expand All @@ -24,6 +25,7 @@ All notable changes to this project will be documented in this file based on the

### Deprecated


## [5.3.0](https://github.com/ruflin/Elastica/compare/5.2.1...5.3.0)

### Backward Compatibility Breaks
Expand Down
12 changes: 5 additions & 7 deletions test/Elastica/TypeTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -509,21 +509,19 @@ public function testDeleteByQueryWithQueryAndOptions()
*/
public function testGetDocumentWithFieldsSelection()
{
$this->markTestSkipped('ES6 update: No handler for type [string] declared on field [name]');

$index = $this->_createIndex();

$type = new Type($index, 'test');
$mapping = new Mapping();
$mapping->setProperties([
'name' => [
'type' => 'string',
'store' => 'yes', ],
'type' => 'text',
'store' => true, ],
'email' => [
'type' => 'string',
'store' => 'yes', ],
'type' => 'text',
'store' => true, ],
'country' => [
'type' => 'string',
'type' => 'text',
],
]);

Expand Down

0 comments on commit e07c520

Please sign in to comment.