Extend(ed) API Endpoint /publications/search/:search_params

[tbsvttr]

adding advanced search for reaction criteria
adding advanced search for sample criteria
adding advanced search for analysis criteria

http://193.196.38.77/swagger#/retrieve_id_list
https://docs.google.com/spreadsheets/d/1q6L_vbWorxLsvSVz7KCHMiNItZkyF4TwuhwzAwxh0pw/edit#gid=0

[tbsvttr]

@Respicite Which ones are still open?

[Respicite]

Type	Field	Operator	Status
Publication	Published Date	> = <	DONE
Publication	Authors		DONE
Publication	Contributor		DONE
Publication	Embargo#
Publication	DOI		?
Publication	ORCID
Publication	Chemotion-ID		?
Reaction	Description	LIKE	DONE
Reaction	Short-RInChIKey	LIKE
Reaction	Yield	> = <	DONE
Reaction	Temperature	> = <	REMOVED
Sample	Formula	LIKE
Sample	InChIKey	LIKE
Sample	SMILES	LIKE
Sample	X-Vial	LIKE
Sample	Description	LIKE	DONE
Analysis	Instrument
Analysis	Ontologies	LIKE
Analysis	Description	LIKE	DONE

@tbsvttr

[Respicite]

@tbsvttr Though do we need a search by chemotion id / doi? We have an dedicated endpoint for them after all.

[Respicite]

Also another question:
For yield and datum we have > = < but instead of having one field where you can define one such filter wouldn't it be better to have two fields for each to define the upper and lower bounds? (I mean we could also allow to dynamically add as many filters as wanted for each but that would make it much more complex / potentially taking a lot longer for the server to execute the query while being nearly no improvement for the user) @tbsvttr

[tbsvttr]

@Respicite We do not do temperature, because of the problematic constraints we both found out to which Paggy agreed.

[tbsvttr]

@Respicite I will ask about Chemotion ID and DOI. The only thing that would make sense for them is LIKE, because EQ is, as you say, implemented via another endpoint. Do not do anything here before I have an answer.

[tbsvttr]

@Respicite Good work with the > = < search. I will ask about the upper and lower bound stuff. At the moment, leave it as it is and create a PR for it.

[Respicite]

@tbsvttr #99

[tbsvttr]

@Respicite Published Date and Yield done, aren't they?

[Respicite]

@tbsvttr Yes they are since last month, we also had a talk about it around three weeks ago and IIRC everything was fine, or is something missing / should be changed?