Skip to content

Commit

Permalink
Refactor AudioLoaderRestHandler
Browse files Browse the repository at this point in the history
  • Loading branch information
davidffa committed Sep 5, 2021
1 parent c09e3be commit 25f8f0e
Showing 1 changed file with 6 additions and 10 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -46,10 +46,7 @@ class AudioLoaderRestHandler(
}

private fun log(request: HttpServletRequest) {
val path = request.servletPath
val method = request.method

log.info("$method $path")
log.info("${request.method} ${request.servletPath}")
}

private fun trackToJSON(audioTrack: AudioTrack): JSONObject {
Expand All @@ -63,10 +60,9 @@ class AudioLoaderRestHandler(
.put("uri", trackInfo.uri)
.put("isStream", trackInfo.isStream)
.put("isSeekable", audioTrack.isSeekable)
.put("source", audioTrack.sourceManager.sourceName)
.put("source", audioTrack.sourceManager?.sourceName)
.put("position", audioTrack.position)
.put("thumbnail", trackInfo.artworkUrl)
.put("sourceName", audioTrack.sourceManager?.sourceName)
}

private fun encodeLoadResult(result: LoadResult): JSONObject {
Expand Down Expand Up @@ -115,7 +111,7 @@ class AudioLoaderRestHandler(
return AudioLoader(audioPlayerManager).load(identifier)
.thenApply(this::encodeLoadResult)
.thenApply {
ResponseEntity<String>(it.toString(), HttpStatus.OK)
ResponseEntity(it.toString(), HttpStatus.OK)
}
}

Expand All @@ -127,7 +123,7 @@ class AudioLoaderRestHandler(

val audioTrack = Util.decodeAudioTrack(audioPlayerManager, track)

return ResponseEntity<String>(trackToJSON(audioTrack).toString(), HttpStatus.OK)
return ResponseEntity(trackToJSON(audioTrack).toString(), HttpStatus.OK)
}

@PostMapping(value = ["/decodetracks"], consumes = ["application/json"], produces = ["application/json"])
Expand All @@ -139,7 +135,7 @@ class AudioLoaderRestHandler(
val requestJSON = JSONArray(body)
val responseJSON = JSONArray()

requestJSON.forEachIndexed { i, _ ->
for (i in 0 until requestJSON.length()) {
val track = requestJSON.getString(i)
val audioTrack = Util.decodeAudioTrack(audioPlayerManager, track)

Expand All @@ -151,6 +147,6 @@ class AudioLoaderRestHandler(
responseJSON.put(trackJSON)
}

return ResponseEntity<String>(responseJSON.toString(), HttpStatus.OK)
return ResponseEntity(responseJSON.toString(), HttpStatus.OK)
}
}

0 comments on commit 25f8f0e

Please sign in to comment.