A modern programming language that makes developers happier.

kotlinlang

You have created list `props`, updated it, but never used it. Seems like a bug? :slightly_smiling_face:

I guess you can create local function that will get param by name from context and will nullify it if it is empty.

Something like this (I didn’t test):
```fun getParam(name: String): String? {
    val value = ctx.formParam(name)
    return if (value == "") null else value 
}```

Or you can put this function into context class or make it extension function, whatever makes sense in your case.

oh yeah, i could’ve just made the list and ran it with that. Thanks for that function as well! That will make things easier. Just a question, is my indentation for the book properties acceptable, or discouraged

```
  <http://app.post|app.post>("/new-book-manual") manual@{ ctx -&gt;
    val title = ctx getParam "title"?: return@manual
    val author = ctx getParam "author"?: return@manual
    val edition = ctx getParam "edition"
    val publisher = ctx getParam "publisher"
    val subjects = ctx getParam "subjects"
    val binding = ctx getParam "binding"
    val publishDate = ctx getParam "publish-date"
    val overview = ctx getParam "overview"
  
    BookRepo.add(Book(title, author,
                      edition, publisher,
                      subjects, binding,
                      publishDate, overview))
    
    ctx.html("Book has been added!")
  }
```
I simplified it even more with an infix function

Nice! :slightly_smiling_face:
Your infix function gave me an idea: What if you tried operator overloading with extension functions to access params like an array? <https://kotlinlang.org/docs/reference/operator-overloading.html#indexed>

I'm thinking about a variation like this:
```operator fun CtxType.get(name: String): String? {
  val value = this.formParam(name)
  return if (value == "") null else value
}```

so you can get the params like this:
```val title = ctx["title"] ?: return@manual
val author = ctx["author"] ?: return@manual
val edition = ctx["edition"]
val publisher = ctx["publisher"]
\\ etc```

Haven't tried it myself, just wanted to share my idea with you in case you like it :slightly_smiling_face:

I like operator overloading more than infix function too.
Now I think you can put `if (value == "") null else value` to separate extension function too :slightly_smiling_face:

`?: return` is missing on sending back an error html

What I’d do with <#C0A974TJ9|ktor> (probably javalin has similar mechanism) is define an `IncompleteDataException` type, `?: throw IncompleteDataException(&lt;paramName&gt;)`  and install a generic exception handler to return a standard html page. You can reuse it in other routes, of course.

You also might want to return a `Bad Request` status code along with HTML, not `200 OK` as you do now