Skip to content

Commit 797f586

Browse files
earlgrey02sdeleuze
authored andcommitted
Polish RouterFunctionDsl
Place the lambda outside the method call. Closes gh-32765
1 parent 8ac39d0 commit 797f586

File tree

1 file changed

+24
-24
lines changed

1 file changed

+24
-24
lines changed

Diff for: spring-webflux/src/main/kotlin/org/springframework/web/reactive/function/server/RouterFunctionDsl.kt

+24-24
Original file line numberDiff line numberDiff line change
@@ -174,7 +174,7 @@ class RouterFunctionDsl internal constructor (private val init: RouterFunctionDs
174174
* @since 5.3
175175
*/
176176
fun GET(predicate: RequestPredicate, f: (ServerRequest) -> Mono<out ServerResponse>) {
177-
builder.GET(predicate, { f(it).cast(ServerResponse::class.java) })
177+
builder.GET(predicate) { f(it).cast(ServerResponse::class.java) }
178178
}
179179

180180
/**
@@ -185,7 +185,7 @@ class RouterFunctionDsl internal constructor (private val init: RouterFunctionDs
185185
* @since 5.2
186186
*/
187187
fun GET(pattern: String, predicate: RequestPredicate, f: (ServerRequest) -> Mono<out ServerResponse>) {
188-
builder.GET(pattern, predicate, HandlerFunction { f(it).cast(ServerResponse::class.java) })
188+
builder.GET(pattern, predicate) { f(it).cast(ServerResponse::class.java) }
189189
}
190190

191191
/**
@@ -219,7 +219,7 @@ class RouterFunctionDsl internal constructor (private val init: RouterFunctionDs
219219
* @since 5.3
220220
*/
221221
fun HEAD(predicate: RequestPredicate, f: (ServerRequest) -> Mono<out ServerResponse>) {
222-
builder.HEAD(predicate, HandlerFunction { f(it).cast(ServerResponse::class.java) })
222+
builder.HEAD(predicate) { f(it).cast(ServerResponse::class.java) }
223223
}
224224

225225
/**
@@ -230,7 +230,7 @@ class RouterFunctionDsl internal constructor (private val init: RouterFunctionDs
230230
* @since 5.2
231231
*/
232232
fun HEAD(pattern: String, predicate: RequestPredicate, f: (ServerRequest) -> Mono<out ServerResponse>) {
233-
builder.HEAD(pattern, predicate, HandlerFunction { f(it).cast(ServerResponse::class.java) })
233+
builder.HEAD(pattern, predicate) { f(it).cast(ServerResponse::class.java) }
234234
}
235235

236236
/**
@@ -264,7 +264,7 @@ class RouterFunctionDsl internal constructor (private val init: RouterFunctionDs
264264
* @since 5.3
265265
*/
266266
fun POST(predicate: RequestPredicate, f: (ServerRequest) -> Mono<out ServerResponse>) {
267-
builder.POST(predicate, HandlerFunction { f(it).cast(ServerResponse::class.java) })
267+
builder.POST(predicate) { f(it).cast(ServerResponse::class.java) }
268268
}
269269

270270
/**
@@ -275,7 +275,7 @@ class RouterFunctionDsl internal constructor (private val init: RouterFunctionDs
275275
* @since 5.2
276276
*/
277277
fun POST(pattern: String, predicate: RequestPredicate, f: (ServerRequest) -> Mono<out ServerResponse>) {
278-
builder.POST(pattern, predicate, HandlerFunction { f(it).cast(ServerResponse::class.java) })
278+
builder.POST(pattern, predicate) { f(it).cast(ServerResponse::class.java) }
279279
}
280280

281281
/**
@@ -309,7 +309,7 @@ class RouterFunctionDsl internal constructor (private val init: RouterFunctionDs
309309
* @since 5.3
310310
*/
311311
fun PUT(predicate: RequestPredicate, f: (ServerRequest) -> Mono<out ServerResponse>) {
312-
builder.PUT(predicate, HandlerFunction { f(it).cast(ServerResponse::class.java) })
312+
builder.PUT(predicate) { f(it).cast(ServerResponse::class.java) }
313313
}
314314

315315
/**
@@ -320,7 +320,7 @@ class RouterFunctionDsl internal constructor (private val init: RouterFunctionDs
320320
* @since 5.2
321321
*/
322322
fun PUT(pattern: String, predicate: RequestPredicate, f: (ServerRequest) -> Mono<out ServerResponse>) {
323-
builder.PUT(pattern, predicate, HandlerFunction { f(it).cast(ServerResponse::class.java) })
323+
builder.PUT(pattern, predicate) { f(it).cast(ServerResponse::class.java) }
324324
}
325325

326326
/**
@@ -354,7 +354,7 @@ class RouterFunctionDsl internal constructor (private val init: RouterFunctionDs
354354
* @since 5.3
355355
*/
356356
fun PATCH(predicate: RequestPredicate, f: (ServerRequest) -> Mono<out ServerResponse>) {
357-
builder.PATCH(predicate, HandlerFunction { f(it).cast(ServerResponse::class.java) })
357+
builder.PATCH(predicate) { f(it).cast(ServerResponse::class.java) }
358358
}
359359

360360
/**
@@ -365,7 +365,7 @@ class RouterFunctionDsl internal constructor (private val init: RouterFunctionDs
365365
* @since 5.2
366366
*/
367367
fun PATCH(pattern: String, predicate: RequestPredicate, f: (ServerRequest) -> Mono<out ServerResponse>) {
368-
builder.PATCH(pattern, predicate, HandlerFunction { f(it).cast(ServerResponse::class.java) })
368+
builder.PATCH(pattern, predicate, { f(it).cast(ServerResponse::class.java) })
369369
}
370370

371371
/**
@@ -401,7 +401,7 @@ class RouterFunctionDsl internal constructor (private val init: RouterFunctionDs
401401
* @since 5.3
402402
*/
403403
fun DELETE(predicate: RequestPredicate, f: (ServerRequest) -> Mono<out ServerResponse>) {
404-
builder.DELETE(predicate, HandlerFunction { f(it).cast(ServerResponse::class.java) })
404+
builder.DELETE(predicate) { f(it).cast(ServerResponse::class.java) }
405405
}
406406

407407
/**
@@ -412,7 +412,7 @@ class RouterFunctionDsl internal constructor (private val init: RouterFunctionDs
412412
* @since 5.2
413413
*/
414414
fun DELETE(pattern: String, predicate: RequestPredicate, f: (ServerRequest) -> Mono<out ServerResponse>) {
415-
builder.DELETE(pattern, predicate, HandlerFunction { f(it).cast(ServerResponse::class.java) })
415+
builder.DELETE(pattern, predicate) { f(it).cast(ServerResponse::class.java) }
416416
}
417417

418418
/**
@@ -448,7 +448,7 @@ class RouterFunctionDsl internal constructor (private val init: RouterFunctionDs
448448
* @since 5.3
449449
*/
450450
fun OPTIONS(predicate: RequestPredicate, f: (ServerRequest) -> Mono<out ServerResponse>) {
451-
builder.OPTIONS(predicate, HandlerFunction { f(it).cast(ServerResponse::class.java) })
451+
builder.OPTIONS(predicate) { f(it).cast(ServerResponse::class.java) }
452452
}
453453

454454
/**
@@ -459,7 +459,7 @@ class RouterFunctionDsl internal constructor (private val init: RouterFunctionDs
459459
* @since 5.2
460460
*/
461461
fun OPTIONS(pattern: String, predicate: RequestPredicate, f: (ServerRequest) -> Mono<out ServerResponse>) {
462-
builder.OPTIONS(pattern, predicate, HandlerFunction { f(it).cast(ServerResponse::class.java) })
462+
builder.OPTIONS(pattern, predicate) { f(it).cast(ServerResponse::class.java) }
463463
}
464464

465465
/**
@@ -476,7 +476,7 @@ class RouterFunctionDsl internal constructor (private val init: RouterFunctionDs
476476
* @see RouterFunctions.route
477477
*/
478478
fun accept(mediaType: MediaType, f: (ServerRequest) -> Mono<out ServerResponse>) {
479-
builder.add(RouterFunctions.route(RequestPredicates.accept(mediaType), HandlerFunction { f(it).cast(ServerResponse::class.java) }))
479+
builder.add(RouterFunctions.route(RequestPredicates.accept(mediaType)) { f(it).cast(ServerResponse::class.java) })
480480
}
481481

482482
/**
@@ -493,7 +493,7 @@ class RouterFunctionDsl internal constructor (private val init: RouterFunctionDs
493493
* @see RouterFunctions.route
494494
*/
495495
fun contentType(mediaTypes: MediaType, f: (ServerRequest) -> Mono<out ServerResponse>) {
496-
builder.add(RouterFunctions.route(RequestPredicates.contentType(mediaTypes), HandlerFunction { f(it).cast(ServerResponse::class.java) }))
496+
builder.add(RouterFunctions.route(RequestPredicates.contentType(mediaTypes)) { f(it).cast(ServerResponse::class.java) })
497497
}
498498

499499
/**
@@ -510,7 +510,7 @@ class RouterFunctionDsl internal constructor (private val init: RouterFunctionDs
510510
* @see RouterFunctions.route
511511
*/
512512
fun headers(headersPredicate: (ServerRequest.Headers) -> Boolean, f: (ServerRequest) -> Mono<out ServerResponse>) {
513-
builder.add(RouterFunctions.route(RequestPredicates.headers(headersPredicate), HandlerFunction { f(it).cast(ServerResponse::class.java) }))
513+
builder.add(RouterFunctions.route(RequestPredicates.headers(headersPredicate)) { f(it).cast(ServerResponse::class.java) })
514514
}
515515

516516
/**
@@ -526,7 +526,7 @@ class RouterFunctionDsl internal constructor (private val init: RouterFunctionDs
526526
* @see RouterFunctions.route
527527
*/
528528
fun method(httpMethod: HttpMethod, f: (ServerRequest) -> Mono<out ServerResponse>) {
529-
builder.add(RouterFunctions.route(RequestPredicates.method(httpMethod), HandlerFunction { f(it).cast(ServerResponse::class.java) }))
529+
builder.add(RouterFunctions.route(RequestPredicates.method(httpMethod)) { f(it).cast(ServerResponse::class.java) })
530530
}
531531

532532
/**
@@ -541,7 +541,7 @@ class RouterFunctionDsl internal constructor (private val init: RouterFunctionDs
541541
* @see RouterFunctions.route
542542
*/
543543
fun path(pattern: String, f: (ServerRequest) -> Mono<out ServerResponse>) {
544-
builder.add(RouterFunctions.route(RequestPredicates.path(pattern), HandlerFunction { f(it).cast(ServerResponse::class.java) }))
544+
builder.add(RouterFunctions.route(RequestPredicates.path(pattern)) { f(it).cast(ServerResponse::class.java) })
545545
}
546546

547547
/**
@@ -555,7 +555,7 @@ class RouterFunctionDsl internal constructor (private val init: RouterFunctionDs
555555
* @see RouterFunctions.route
556556
*/
557557
fun pathExtension(extension: String, f: (ServerRequest) -> Mono<out ServerResponse>) {
558-
builder.add(RouterFunctions.route(RequestPredicates.pathExtension(extension), HandlerFunction { f(it).cast(ServerResponse::class.java) }))
558+
builder.add(RouterFunctions.route(RequestPredicates.pathExtension(extension)) { f(it).cast(ServerResponse::class.java) })
559559
}
560560

561561
/**
@@ -570,7 +570,7 @@ class RouterFunctionDsl internal constructor (private val init: RouterFunctionDs
570570
* @see RouterFunctions.route
571571
*/
572572
fun pathExtension(predicate: (String?) -> Boolean, f: (ServerRequest) -> Mono<out ServerResponse>) {
573-
builder.add(RouterFunctions.route(RequestPredicates.pathExtension(predicate), HandlerFunction { f(it).cast(ServerResponse::class.java) }))
573+
builder.add(RouterFunctions.route(RequestPredicates.pathExtension(predicate)) { f(it).cast(ServerResponse::class.java) })
574574
}
575575

576576
/**
@@ -586,7 +586,7 @@ class RouterFunctionDsl internal constructor (private val init: RouterFunctionDs
586586
* @see RouterFunctions.route
587587
*/
588588
fun queryParam(name: String, predicate: (String) -> Boolean, f: (ServerRequest) -> Mono<out ServerResponse>) {
589-
builder.add(RouterFunctions.route(RequestPredicates.queryParam(name, predicate), HandlerFunction { f(it).cast(ServerResponse::class.java) }))
589+
builder.add(RouterFunctions.route(RequestPredicates.queryParam(name, predicate)) { f(it).cast(ServerResponse::class.java) })
590590
}
591591

592592
/**
@@ -605,7 +605,7 @@ class RouterFunctionDsl internal constructor (private val init: RouterFunctionDs
605605
* @see RouterFunctions.route
606606
*/
607607
operator fun RequestPredicate.invoke(f: (ServerRequest) -> Mono<out ServerResponse>) {
608-
builder.add(RouterFunctions.route(this, HandlerFunction { f(it).cast(ServerResponse::class.java) }))
608+
builder.add(RouterFunctions.route(this) { f(it).cast(ServerResponse::class.java) })
609609
}
610610

611611
/**
@@ -614,7 +614,7 @@ class RouterFunctionDsl internal constructor (private val init: RouterFunctionDs
614614
* @see RouterFunctions.route
615615
*/
616616
operator fun String.invoke(f: (ServerRequest) -> Mono<out ServerResponse>) {
617-
builder.add(RouterFunctions.route(RequestPredicates.path(this), HandlerFunction { f(it).cast(ServerResponse::class.java) }))
617+
builder.add(RouterFunctions.route(RequestPredicates.path(this)) { f(it).cast(ServerResponse::class.java) })
618618
}
619619

620620
/**

0 commit comments

Comments
 (0)