Path Variable not mapping Spring MVC

Question

So i have the following problem. I have a controller which i mapped to "/reservationQuery/*". Whenerver i go on the url for "delete_1" method, it works, but whnever i try and go on the "delete_2" method it says that there was no mapping for that URI. But why does that happen? Shouldn all URI beggining with /reservationQuery/ be mapped by my ReservationQueryController?

And if i want to use the second URI for deletion, do i have to make another controller and map it to /reservationQuery/delete/* ?

@Controller
@RequestMapping("/reservationQuery/*")
public class ReservationQueryController{
private ReservationService reservationService;

@Autowired
public ReservationQueryController(ReservationService reservationService) {
    this.reservationService = reservationService;
}


@RequestMapping(value="/reservationQuery/{reservationID}", method=RequestMethod.GET)
public String delete_1(@PathVariable("reservationID") int reservationID, Model model){
    reservationService.deleteReservation(reservationID);
    return "reservationQuery";
}


@RequestMapping(value="/reservationQuery/delete/{reservationID}",     method=RequestMethod.GET)
public String delete_2(@PathVariable("reservationID") int reservationID, Model model){
    reservationService.deleteReservation(reservationID);
    return "reservationQuery";
}




}

I think you should be using a RequestMapping at the class level and Paths at the method level with the path appending extra onto the original URL defined in the RequestMapping. — JamesB
– JamesB, Commented Aug 14, 2014 at 8:11

jorrin · Accepted Answer · 2014-08-14 08:22:22Z

3

You can remove the extra value of "reservationQuery" in the method level as you have already mapped "reservationQuery" in the Controller, which is the class level.

So what's happening is that at delete your URL is going to begin with /reservationQuery/reservationQuery/.

I think that's the reason for an error and it should be HTTP Error 404

answered Aug 14, 2014 at 8:22

jorrin

6221 gold badge6 silver badges13 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Baby Groot · Accepted Answer · 2014-08-14 08:22:33Z

2

You should map your controller with "/reservationQuery" and your method delete_1 should be mapped with "/{reservationID}" and method delete_2 should be mapped with "/delete/{reservationID}". There is no need to repeat controller mapping in methods.

edited Aug 14, 2014 at 8:22

Baby Groot

4,27939 gold badges55 silver badges70 bronze badges

answered Aug 14, 2014 at 8:16

tomas930

513 bronze badges

1 Comment

Serge Ballesta Over a year ago

IMHO, it is a neater solution than the one proposed by jorrin, because you factorize the upper part of the URL at controller level.

Collectives™ on Stack Overflow

Path Variable not mapping Spring MVC

2 Answers 2

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related