Skip to content

Commit 4484642

Browse files
committed
Add RandomJokeDTO and update random getmapping to it. To not include ID
1 parent 7c3fa8a commit 4484642

File tree

5 files changed

+9
-8
lines changed

5 files changed

+9
-8
lines changed

Dockerfile

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
FROM openjdk
2-
VOLUME /tmp
32
COPY build/libs/DeveloperJokesApp-0.0.1-SNAPSHOT.jar app.jar
43
EXPOSE 8080
54

src/main/kotlin/com/salt/developerjokes/api/JokeController.kt

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ package com.salt.developerjokes.api
33
import com.salt.developerjokes.api.model.jokes.IncomingJokeDTO
44
import com.salt.developerjokes.api.model.jokes.JokeDTO
55
import com.salt.developerjokes.api.model.jokes.JokeListDTO
6+
import com.salt.developerjokes.api.model.jokes.RandomJokeDTO
67
import org.springframework.beans.factory.annotation.Autowired
78
import org.springframework.http.ResponseEntity
89
import org.springframework.web.bind.annotation.*
@@ -21,7 +22,7 @@ class JokeController(@Autowired val service: JokeService) {
2122
}
2223

2324
@GetMapping("/random")
24-
fun getRandomJoke(@RequestParam language: String?): ResponseEntity<JokeDTO> {
25+
fun getRandomJoke(@RequestParam language: String?): ResponseEntity<RandomJokeDTO> {
2526
if (language == null || language == "") {
2627
return ResponseEntity.ok(service.getRandomJoke())
2728
}

src/main/kotlin/com/salt/developerjokes/api/JokeService.kt

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,6 @@
11
package com.salt.developerjokes.api
22

3-
import com.salt.developerjokes.api.model.jokes.IncomingJokeDTO
4-
import com.salt.developerjokes.api.model.jokes.Joke
5-
import com.salt.developerjokes.api.model.jokes.JokeDTO
6-
import com.salt.developerjokes.api.model.jokes.JokeListDTO
3+
import com.salt.developerjokes.api.model.jokes.*
74
import com.salt.developerjokes.api.repository.JokeRepoDAO
85
import org.springframework.beans.factory.annotation.Autowired
96
import org.springframework.http.HttpStatusCode
@@ -12,12 +9,12 @@ import org.springframework.web.server.ResponseStatusException
129

1310
@Service
1411
class JokeService(@Autowired private val repo: JokeRepoDAO) {
15-
fun getRandomJoke(): JokeDTO = repo.getRandomJoke()?.toDTO() ?: throw ResponseStatusException(
12+
fun getRandomJoke(): RandomJokeDTO = repo.getRandomJoke()?.toRandomDTO() ?: throw ResponseStatusException(
1613
HttpStatusCode.valueOf(404),
1714
"No Jokes in the database"
1815
)
1916

20-
fun getRandomJoke(language: String): JokeDTO = repo.getRandomJoke(language)?.toDTO() ?: throw ResponseStatusException(
17+
fun getRandomJoke(language: String): RandomJokeDTO = repo.getRandomJoke(language)?.toRandomDTO() ?: throw ResponseStatusException(
2118
HttpStatusCode.valueOf(404),
2219
"No Jokes for that language"
2320
)

src/main/kotlin/com/salt/developerjokes/api/model/jokes/Joke.kt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@ class Joke(
1717
constructor(oldJoke: Joke, newJoke: IncomingJokeDTO) : this(newJoke.text, oldJoke.id, newJoke.language)
1818

1919
fun toDTO(): JokeDTO = JokeDTO(this.text, this.id.toString())
20+
fun toRandomDTO() : RandomJokeDTO = RandomJokeDTO(this.text)
2021
override fun equals(other: Any?): Boolean {
2122
if (this === other) return true
2223
if (javaClass != other?.javaClass) return false
Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
package com.salt.developerjokes.api.model.jokes
2+
3+
data class RandomJokeDTO(val text : String)

0 commit comments

Comments
 (0)