Add MatchPhrase and MatchPhrasePrefix classes

[ThePixelDeveloper]

Came up to using these features and realise Elastica didn't have the classes for them.

Features defined in ES here:

1. match_phrase
2. match_phrase_prefix

Worth adding new classes for these?

[ruflin]

Good question. In the documentation there are 2 ways of doing it: Either through the model or through the type. So in case we do additional classes, I would suggest that it is only a mapper for the Match query which calls the functions. What is your take on this?

[ThePixelDeveloper]

I forgot about the 2nd way of doing it, but it sounds fine to me.

[ruflin]

We should only choose the second one, in case it simplifies things. Can you open a pull request with one of the two?

[yellow1912]

Any update on this? I'm looking for a way to use match_phrase_prefix as well

[ruflin]

@yellow1912 There is not pull request for it yet. Do you have time to contribute it?

[yellow1912]

Im not familiar with the standards set but I will fork and try.

[ruflin]

Cool. Let me know if you need some help.

[Jmoati]

To use the second way, just do:

$match = new Query\Match();
$match->setFieldQuery($field, $phrase);
$match->setFieldType($field, 'phrase');

Do you really want classes proxy for that ?
I can do it, if usefull.

[ruflin]

@Jmoati The main reason for the proxy method is probably as most engineers who use Elastica just expect it as there are objects for most of the queries. If you have time to open a pull request, that would be nice.

[yehosef]

I think the library is most useful when it matches the documented way of doing it - and if there are two ways of doing the same thing I would prefer that be replicated in the library (obviously with a proxy method).

[olee]

Also there is no method to set slop without having a class for that.
Currently you need to call

$match->setFieldParam('myField', 'slop', 100);

[ruflin]

@yehosef @olee Does anyone of you have some spare time to open a pull request with the mentioned changes?

[yehosef]

@ruflin - I'll try.

[ruflin]

@yehosef Thx

[Jmoati]

@ruflin Can you please close this issue ?