search

ORFAP / FAPBackend

0 stars 0 forks source link

FindById Methoden für Airline und City/Market #12

Closed Arne2 closed 8 years ago

Arne2 commented 8 years ago

Damit der Crawler die Airlines und Cities mit der Id finden kann, werden findBy Methoden benötigt. Diese bitte bereitstellen...

FabianWilms commented 8 years ago
  1. Gibt es schon findAll(id) und findOne(id) durch das CrudRepository
  2. wofür braucht der crawler das? Oo
Arne2 commented 8 years ago
  1. Es geht nicht um die Id, die im Link steht, sondern die Variable Id in der Klasse Airline bzw. City. Die findOne benutzt die, die im Link steht. Die findById soll nach der Objektvariablen suchen.
  2. Wir müssen uns die Id der Links holen können, damit wir die Ruten speichern können. Wir bekommen im zip mit dem POST die Id(Objektvariable) der Airline/Cities.
peter-mueller commented 8 years ago

Warum gibt es dann 2x eine ID? Sollte eine (die Objektvariable) nicht reichen?

Arne2 commented 8 years ago

Es gibt nicht 2x eine Id in der Entität. Die Id, die man mit findOne verwendet nutzt den hintersten Abschnitt von dem selfLink. Die Id der Entität ist die, nach der ich suchen möchte.

FabianWilms commented 8 years ago

Ach ihr habt die Entitäten so gemacht. Ich hätte einfach für Airline und City das ID-Feld halt als @Id markiert und einfach nicht von BaseEntity abgeleitet. Finde ich so etwas umständlich weil die CrudRepositories das eigentlich alles so handlen könnten. Vor allem könntet ihr dann einfach direkt die Routen erstellen mit der ID die ihr schon von Transtats habt ohne euch erst die vom Backend holen zu müssen.

FabianWilms commented 8 years ago

Also im Prinzip genau das wofür rene gerade nen PR aufgemacht hat xD https://github.com/ORFAP/FAPBackend/pull/14/files

Arne2 commented 8 years ago

Achso wenn er jetzt die Id, die er bekommt hernimmt um den Selflink zu erstellen würde es passen

FabianWilms commented 8 years ago

Ignoriert doch mal den selflink xD

Arne2 commented 8 years ago

Den muss ich aber übergeben, wenn ich das Objekt erstellen will

darenegade commented 8 years ago

Auf Wunsch von Arne und Otto heißt es übrigens nicht mehr City sondern Market. Wenn @FabianHoltkoetter oder @peter-mueller ein Problem damit haben, dann warte ich noch mit dem merge

FabianWilms commented 8 years ago

No problems muss sowieso noch die filter-felder anpassen

Arne2 commented 8 years ago

Ok funktioniert jetzt, dadurch, dass die Id, die ist, die im Selflink ist. Findby Methode nicht länger von nöten.