happy-hamster / backend

RESTful Spring Boot API
https://api.happyhamster.org
Apache License 2.0
2 stars 0 forks source link

TINF-243: Mergen der Suche auf Develop #149

Closed r-franzke closed 4 years ago

jstsddg commented 4 years ago

Path: /v2/locations/search/Bonn

java.lang.NullPointerException: null
    at de.sakpaas.backend.service.LocationService.findByCoordinates(LocationService.java:60) ~[classes/:na]
    at de.sakpaas.backend.service.SearchService.getByCoordinates(SearchService.java:178) ~[classes/:na]
    at de.sakpaas.backend.service.SearchService.search(SearchService.java:80) ~[classes/:na]
    at de.sakpaas.backend.v2.controller.LocationController.searchForLocations(LocationController.java:236) ~[classes/:na]
    at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[na:na]
    at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) ~[na:na]
    at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[na:na]
    at java.base/java.lang.reflect.Method.invoke(Method.java:566) ~[na:na]
    at org.springframework.web.method.support.InvocableHandlerMethod.doInvoke(InvocableHandlerMethod.java:190) ~[spring-web-5.2.4.RELEASE.jar:5.2.4.RELEASE]

Wenn man keine Koordinaten angibt xD

Mavv3006 commented 4 years ago

Ich dachte der wird aufgefangen 🤔

jstsddg commented 4 years ago

Path: /v2/locations/search/Bonn?latitude=40&longitude=8

{
    "timestamp": "2020-04-30T11:04:02.729+0000",
    "status": 400,
    "error": "Bad Request",
    "path": "/v2/locations/search/Bonn",
    "context": {
        "textId": "INCOMPLETE_COORDINATES",
        "parameters": [],
        "defaultMessage": "Latitude or Longitude not provided."
    }
}
Mavv3006 commented 4 years ago

Nach was habt ihr gesucht?

Mavv3006 commented 4 years ago

Ich kann meine Changes nicht pushen

r-franzke commented 4 years ago

hoffentlich sind alle Bugs beseitigt :D

Zumindest die die ich gefunden habe

Mavv3006 commented 4 years ago

dann is ja gut