Make increaseInterest transactional

Moves the logic inside service layer.

src/main/kotlin/hu/kirdev/schpincer/service/CircleService.kt

@@ -41,6 +41,30 @@ open class CircleService {
  repo.save(circleEntity)
  }
 
+ private fun internalIncreaseInterestAndSave(circleEntity: CircleEntity): Long {
+ val newInterest = circleEntity.interestCounter;
+ repo.save(circleEntity)
+ return newInterest
+ }
+
+ @Transactional(readOnly = false)
+ open fun increaseInterest(id: Long): Long? {
+ val circleEntity = getOne(id)
+ if circleEntity == null {
+ return null // TODO throw error?
+ }
+ return internalIncreaseInterestAndSave(circleEntity)
+ }
+
+ @Transactional(readOnly = false)
+ open fun increaseInterestByAlias(alias: String): Long? {
+ val circleEntity = findByAlias(id)
+ if circleEntity == null {
+ return null // TODO throw error?
+ }
+ return internalIncreaseInterestAndSave(circleEntity)
+ }
+
  @Transactional(readOnly = true)
  open fun getOne(id: Long): CircleEntity? {
  return repo.getOne(id)

src/main/kotlin/hu/kirdev/schpincer/web/ApiController.kt

@@ -342,19 +342,17 @@ open class ApiController(
  @PostMapping("/provider/{circle}/increaseInterest")
  @ResponseBody
  fun circleIncreaseInterest(@PathVariable circle: String): ResponseEntity<String> {
- val circleEntity: CircleEntity?
+ var result: Long?
  if (circle.matches("^\\d+$".toRegex())) {
  val id = circle.toLong()
- circleEntity = circles.getOne(id)
+ result = circles.increaseInterest(id)
  } else {
- circleEntity = circles.findByAlias(circle)
+ result = circles.increaseInterestByAlias(circle)
  }
- if (circleEntity == null) {
+ if result != null {
  return responseOf("Invalid Circle")
  }
- val newInterest = ++circleEntity.interestCounter
- circles.save(circleEntity)
- return responseOf(""+newInterest)
+ return responseOf(""+result)
  }
 
 // @PostMapping("/easteregg/trashpanda/{feedback}")