Почему при удалении строки в Spring Boot с помощью JPA выводит ошибку?

Question

[Вергилий Меркель]

у меня есть таблица admin_productions с колонками [id, adminproductname, adminproducttotal]
Есть Crud репозиторий с таким кодом:

package com.example.beton.repos;
import com.example.beton.domain.AdminProductions;
import org.springframework.data.repository.CrudRepository;

import java.util.List;
public interface AdminProductRepo extends CrudRepository<AdminProductions, Integer> {
    List<AdminProductions> findByAdminproductname(String adminproductname);
    List<AdminProductions> deleteByAdminproductname(String adminproductname);
}

и такой контроллер: (пытался сделать deleteByAdminproductname() и найти по id, удалить и сохранить)

package com.example.beton.controller;

import com.example.beton.domain.AdminProductions;
import com.example.beton.repos.AdminProductRepo;
import com.example.beton.repos.WarehouseRepo;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.security.access.prepost.PreAuthorize;
import org.springframework.stereotype.Controller;
import org.springframework.web.bind.annotation.GetMapping;
import org.springframework.web.bind.annotation.PostMapping;
import org.springframework.web.bind.annotation.RequestParam;

import java.util.List;
import java.util.Map;

@Controller
@PreAuthorize("hasAuthority('ADMIN')")
public class RemoveController {
    @Autowired
    private AdminProductRepo adminProductRepo;
    @Autowired
    private WarehouseRepo warehouseRepo;

    @GetMapping("/removeproduct")
    public String remPage(Map<String, Object> model){
        Iterable<AdminProductions> adminproductions = adminProductRepo.findAll();
        model.put("adminproductions", adminproductions);
        return "removeproduct";
    }

    @PostMapping("/removeproduct")
    public String removeAdminProduct(@RequestParam String removname, Map<String, Object> model){

        List<AdminProductions> rmadm = adminProductRepo.findByAdminproductname(removname);
        for (AdminProductions admrem : rmadm){
            adminProductRepo.deleteById(admrem.getId());
            adminProductRepo.save(admrem);
        }
        Iterable<AdminProductions> adminproductions = adminProductRepo.findAll();
        model.put("adminproductions", adminproductions);
        return "removeproduct";
    }
}

Выводит такую ошибку

Comments

[Dmitry Roo]

Spring boot еще что-то обязательно в логи пишет. Там ответ.
Логи неплохо бы к вопросу приложить.

[Вергилий Меркель]

Dmitry Roo, debug и terminal заполнен только моими System.out.println))
контроллер

package com.example.beton.controller;

import com.example.beton.domain.AdminProductions;
import com.example.beton.repos.AdminProductRepo;
import com.example.beton.repos.WarehouseRepo;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.security.access.prepost.PreAuthorize;
import org.springframework.stereotype.Controller;
import org.springframework.web.bind.annotation.GetMapping;
import org.springframework.web.bind.annotation.PostMapping;
import org.springframework.web.bind.annotation.RequestParam;

import java.util.List;
import java.util.Map;

//@PreAuthorize("hasAuthority('ADMIN')")
@Controller
public class RemoveController {
    @Autowired
    private AdminProductRepo adminProductRepo;
    @Autowired
    private WarehouseRepo warehouseRepo;

    @GetMapping("/removeproduct")
    public String remPage(Map<String, Object> model){
        Iterable<AdminProductions> adminproductions = adminProductRepo.findAll();
        for (AdminProductions ap : adminproductions){
            System.out.println(ap.getAdminproductname());
        }
        model.put("adminproductions", adminproductions);
        return "removeproduct";
    }
    @PostMapping("/removeproduct")
    public String removeAdminProduct(@RequestParam String removname, Map<String, Object> model){
        System.out.println(removname);

        adminProductRepo.deleteByAdminproductname(removname);
        Iterable<AdminProductions> adminproductions = adminProductRepo.findAll();
        model.put("adminproductions", adminproductions);
        return "removeproduct";
    }
}

логи:

Кольца 1м
Крышка 1,5м
Днище 1,5м
Кольца 2м
ФБС 6
ФБС 6.5

Можно заметить, что с @PostMapping("/removeproduct") System.out.println(removname) не работает

[Вергилий Меркель]

Dmitry Roo,
для начала я забыл написать в форме написать _csrf.token, а во вторых вот логи)

Spring Boot RemoveController error

[сергей кузьмин]

думаю что раз 403 (увеен что это Ошибка 403 доступ запрещен) то попробуйте для начала закомментировать
@PreAuthorize("hasAuthority('ADMIN')")

[сергей кузьмин]

для логов добавьте

private static Logger logger = LoggerFactory.getLogger(RemoveController.class);

ну и

logger.info("message {}" + variable);

и logback.xml в ресурсы

<?xml version="1.0" encoding="UTF-8"?>
<configuration debug="false">
  <property name="LOGDIR" value="./logs"/>
  <property name="FILENAME" value="App"/>
  <property name="PATTERN" value="%d{HH:mm:ss.SSS} [%thread] %-5level %logger{36} - %msg%n"/>
  <appender name="CONSOLE" class="ch.qos.logback.core.ConsoleAppender">
    <encoder class="ch.qos.logback.classic.encoder.PatternLayoutEncoder">
      <pattern>${PATTERN}</pattern>
    </encoder>
  </appender>
  <appender name="FILE" class="ch.qos.logback.core.rolling.RollingFileAppender">
    <rollingPolicy class="ch.qos.logback.core.rolling.TimeBasedRollingPolicy">
      <fileNamePattern>${LOGDIR}/${FILENAME}.%d{yyyy-MM-dd}.log.gz</fileNamePattern>
      <MaxHistory>3</MaxHistory>
    </rollingPolicy>
    <encoder class="ch.qos.logback.classic.encoder.PatternLayoutEncoder">
      <pattern>${PATTERN}</pattern>
    </encoder>
    <triggeringPolicy class="ch.qos.logback.core.rolling.SizeBasedTriggeringPolicy">
      <MaxFileSize>1KB</MaxFileSize>
    </triggeringPolicy>
  </appender>
  <logger name="org.springframework" level="WARN"/>
  <logger name="o.s" level="WARN"/>
  <logger name="org.springframework.jdbc" level="TRACE"/>
  <logger name="java.sql.Connection" level="DEBUG"/>
  <logger name="java.sql.Statement" level="DEBUG"/>
  <logger name="java.sql.PreparedStatement" level="DEBUG"/>
  <root level="DEBUG">
    <appender-ref ref="CONSOLE"/>
    <appender-ref ref="FILE"/>
  </root>
</configuration>

- тут по вкусу под себя доделайте

и поменяйте руты
напр
@PostMapping("/removeadminproduct")

Answer 1

[Орхан Гасанлы]

Здравствуйте!
То что вы видите 403 ошибку (Forbidden) особо ни о чем не говорит.
1) Если у вас подключен Spring Security, то проверьте может ли пользователь совершить нужный вам запрос. @PreAuthorize("hasAuthority('ADMIN')") Пользователь должен быть админом
2) Проверьте правильно ли вы обрабатываете post || get запрос.
3) В этом участке кода тоже ошибка. Вы сперва удаляете admrem, а потом пытаетесь сохранить удаленный admrem.

for (AdminProductions admrem : rmadm){
            adminProductRepo.deleteById(admrem.getId());
            adminProductRepo.save(admrem);
        }

Скорее всего это лишнее
adminProductRepo.save(admrem);

4) Тут тоже может быть проблема. Например, в camelCase.
deleteByAdminproductname()
Потому, что у вас сущность называется AdminProduct и поле name
А структура запроса у вас:
Adminproductname
Тоже что-то не так...

Ну в конце концов, поставьте точки остановки, включите дебаг и посмотрите какая ошибка прибегает на сервер. Может быть у вас запрос даже не доходит до контроллера из-за ошибки в Spring Security

Comments

[Вергилий Меркель]

с camelCase всё ок, так как переменная у меня названа adminproductname, а значится findByAdminproductname, а еще у меня этот метод в crud репозитории рабатает при update:

Iterable<Sales> sales = saleRepo.findAll();
        Iterable<AdminProductions> adminproductions = adminproductionsRepo.findAll();


        Integer whCount = 0;
        List<Warehouse> sls = warehouseRepo.findByWarehousename(salename);
        for (Warehouse sl : sls){
            whCount = Integer.parseInt(sl.getWarehousecount());
            whCount = whCount - Integer.parseInt(salecount);
            sl.setWarehousecount(whCount.toString());
            warehouseRepo.save(sl);
        }
        model.put("adminproductions", adminproductions);
        model.put("sales", sales);

Покажите мне лучше как происходит запрос DELETE в контроллере.
Так правильно? ->
Repo

package com.example.beton.repos;
import com.example.beton.domain.AdminProductions;
import org.springframework.data.repository.CrudRepository;
import java.util.List;
public interface AdminProductRepo extends CrudRepository<AdminProductions, Integer> {
    List<AdminProductions> findByAdminproductname(String adminproductname);
    AdminProductions deleteByAdminproductname(String adminproductname);
}

RemoveController

package com.example.beton.controller;

import com.example.beton.domain.AdminProductions;
import com.example.beton.repos.AdminProductRepo;
import com.example.beton.repos.WarehouseRepo;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.security.access.prepost.PreAuthorize;
import org.springframework.stereotype.Controller;
import org.springframework.web.bind.annotation.GetMapping;
import org.springframework.web.bind.annotation.PostMapping;
import org.springframework.web.bind.annotation.RequestParam;

import java.util.List;
import java.util.Map;



@Controller
//@PreAuthorize("hasAuthority('ADMIN')")
public class RemoveController {
    @Autowired
    private AdminProductRepo adminProductRepo;
    @Autowired
    private WarehouseRepo warehouseRepo;

    @GetMapping("/removeproduct")
    public String remPage(Map<String, Object> model){
        Iterable<AdminProductions> adminproductions = adminProductRepo.findAll();
        model.put("adminproductions", adminproductions);
        return "removeproduct";
    }

    @PostMapping("/removeproduct")
    public String removeAdminProduct(@RequestParam String removname, Map<String, Object> model){

        adminProductRepo.deleteByAdminproductname(removname);

        Iterable<AdminProductions> adminproductions = adminProductRepo.findAll();
        model.put("adminproductions", adminproductions);
        return "removeproduct";
    }
}

[Орхан Гасанлы]

AdminProductions deleteByAdminproductname(String adminproductname);

Думаю необязательно возвращать AdminProductions, особенно если в контроллере вы не используете.

adminProductRepo.deleteByAdminproductname(removname);

[Вергилий Меркель]

azerphoenix, кажется я решил проблему, от части мне помогла метка @Transactional на уровне метода, а запрос удаления я сделал по другому, чтобы не думать правильно ли у меня метод в Repo написан

@Transactional
    @PostMapping("/removeproduct")
    public String removeAdminProduct(@RequestParam String removname, Map<String, Object> model){
        System.out.println("_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_");
        System.out.println(removname);

        List<AdminProductions> adp = adminProductRepo.findByAdminproductname(removname);

        for(AdminProductions ad : adp){
            adminProductRepo.deleteById(ad.getId());
            System.out.println(ad.getId() +" : "+ad.getAdminproductname());

        }

        Iterable<AdminProductions> adminproductions = adminProductRepo.findAll();
        model.put("adminproductions", adminproductions);
        return "removeproduct";
    }

Answer 2

[Вергилий Меркель]

Вот с этого участка не работает))

Answer 3

[Dmitry Roo]

В логах, которые вы предоставили написано:

javax.persistence.TransactionRequiredException: No EntityManager with actual transaction available for current thread - cannot reliably process 'remove' call

Это значит, что пора разбираться с транзакциями.
Почитайте, например:
https://habr.com/ru/company/otus/blog/431508/
akorsa.ru/2016/08/kak-na-samom-dele-rabotaet-trans...

У Eugen Paraschiv можно подсмотреть примеры:
https://www.baeldung.com/transaction-configuration...