Forums:

PostMethod not being called

Daniel Demesmaecker

Rancher

Posts: 1171

I like...

posted 6 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

In my applicaion I have one base entity called general, this entity use to contain all the fields, but with normalisation in mind I seperated them in 8 different entities. All other 7 have an one to one relationship with eather general or one of the other entities which has in it's turn again a one to one to general.
I'm using seperate forms to ask the user to create the entities, but having some problems.
The genral idea was after filling in the previous form the user would automaticly be redirected to the next form. I'm struggleing to make it work.
I can't even really explain what is going wrong, so let me just post what I got and maybe you guys can give me some tips on how to tackle this problem.
The one thing I can say, because I'm sure someone will make a remark about it. The reasson why I call the getInvoices method, is because if I redirect like I tried, spring dosn't create a bean for the webform so I get an 404.
The redirecting works, but when I trie to post the invoiceform, nothing happens, I putted a system.out in the postMethod to see if it get's called and it isn't.

InvoiceForm:

<%@ page contentType="text/html;charset=UTF-8"  %>
<%@ taglib prefix="c" uri="http://java.sun.com/jsp/jstl/core" %>
<%@ taglib prefix="fn" uri="http://java.sun.com/jsp/jstl/functions" %>
<%@ taglib prefix="form" uri="http://www.springframework.org/tags/form" %>
<!DOCTYPE html>
<html lang="en">
<head>
    <meta charset="utf-8">
    <meta name="viewport" content="width=device-width, initial-scale=1, shrink-to-fit=no">
    <meta name="description" content="">
    <meta name="author" content="">
    <title>Facturen</title>

<%--@elvariable id="invoiceForm" type=""--%>
    <form:form action="invoices" method="post" commandName="invoiceForm">
        <table  id="forms">
             <tr>
                 <td>
                     <label>Ontvangen:</label>
                 </td>
                 <td>
                     <form:select path="received" id="received" name="received">
                         <form:option value="Ok"/>
                         <form:option value="Niet Ok"/>
                         <form:option value="Niet Aanwezig"/>
                         <form:option value="Aanwezig"/>
                     </form:select>
                 </td>
                 <td>
                     <label>Status: </label>
                 </td>
                 <td>
                     <form:select path="state" id="state" name="state">
                         <form:option value="In Behandeling"/>
                         <form:option value="Goedgekeurd"/>
                         <form:option value="Geweigerd"/>
                         <form:option value="Ingetrokken"/>
                         <form:option value="Moraliteitsonderzoek"/>
                     </form:select>
                 </td>
             </tr>
             <tr>
                 <td>
                     <label>Aantal Vorig jaar:</label>
                 </td>
                 <td>
                     <form:input path="last" id="last" type="text" name="last" required="required"/>
                 </td>
                 <td>
                     <label>Aantal Andere jaren:</label>
                 </td>
                 <td>
                     <form:input path="early" id="early" type="text" name="early" required="required"/>
                 </td>
             </tr>
             <tr>
                 <td>
 
                 </td>
                 <td><label>BevestigingsDatum:</label></td>
                 <td><form:input path="confirmationDate" id="confirmationDate" type="date" name="confirmationDate"
                                 required="required"/></td>
                 </tr>
             <tr>
                 <td></td>
                 <td  ><input type="submit" value="Opslaan" class="button"/>
                 </td>
             </tr>
         </table>
     </form:form>
 
 
 <%@include file="footer.jsp" %>
 
 
 <script src="${pageContext.request.contextPath}/js/jquery.min.js"></script>
 <script src="${pageContext.request.contextPath}/js/popper.min.js"></script>
 <script src="${pageContext.request.contextPath}/js/bootstrap.min.js"></script>
 </body>
 </html>

newReecord.jsp (to created the generalEntity and addressEntity)

<%@ taglib prefix="form" uri="http://www.springframework.org/tags/form" %>
 <%--
   Created by IntelliJ IDEA.
   User: Intec
   Date: 3/07/2018
   Time: 15:19
   To change this template use File | Settings | File Templates.
 --%>
 <%@ page contentType="text/html;charset=UTF-8" language="java" %>
 <html>
 <head>
     <title>Record Toevoegen</title>
     <link rel="stylesheet" href="${pageContext.request.contextPath}/css/general.css">
 
     <link href="${pageContext.request.contextPath}/css/bootstrap.min.css" rel="stylesheet">
     <link href="${pageContext.request.contextPath}/css/modern-business.css" rel="stylesheet">
     <link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/font-awesome/4.7.0/css/font-awesome.min.css">
     <link href="${pageContext.request.contextPath}/css/customUtilities.css" rel="stylesheet">
     <link href="${pageContext.request.contextPath}/css/general.css" rel="stylesheet">
     <link href="https://fonts.googleapis.com/css?family=Pacifico" rel="stylesheet">
     <link rel="icon" href="${pageContext.request.contextPath}/images/vigilis.jpg" type="image/x-icon">
 </head>
 <body>
 <%@include file="navigation.jsp" %>
 
 <%--@elvariable id="wrapperForm" type=""--%>
 <form:form method="post" commandName="wrapperForm">
     <table  id="forms">
         <tr>
             <td>
                 <label>Taal:</label>
             </td>
             <td>
                 <form:select path="language" id="language" name="language">
                     <form:option value="Nederlands"/>
                     <form:option value="Français"/>
                     <form:option value="Deutsch"/>
                     <form:option value="English"/>
                 </form:select>
             </td>
             <td>
                 <label>Type Aanvraag: </label>
             </td>
             <td>
                 <form:select path="applicationType" id="type" name="type">
                     <form:option value="Nieuwe aanvraag"/>
                     <form:option value="Voortzetting"/>
                     <form:option value="Vrijwillige ontbinding"/>
                     <form:option value="Faillisement"/>
                 </form:select>
             </td>
         </tr>
         <tr>
             <td>
                 <label for="nameKbo">KBO-Benaming</label>
             </td>
             <td >
                 <form:input path="nameKbo" id="nameKbo" type="text" name="nameKbo" required="required" size="100%"/>
             </td>
         <tr>
             <td>
                 <label for="kboNumber">KBO-Nummer:</label>
             </td>
             <td>
                 <form:input path="kboNumber" id="kboNumber" type="text" name="kboNumber"/>
             </td>
         </tr>
         <tr>
             <td>
                 <label for="nameOrg">HandelsBenaming:</label>
             </td>
             <td >
                 <form:input path="nameOrg" id="nameOrg" type="text" name="nameOrg" size="100%"/>
             </td>
         </tr>
         <tr>
             <td>
                 <label>Adres:</label>
             </td>
             <td >
                 <form:input path="street" id="street" name="street" type="text" placeholder="Straatnaam" size="100%"/>
             </td>
             <td>
                 <form:input path="number" id="number" name="number" type="text" size="5"/>
 
             </td>
         </tr>
         <tr>
             <td></td>
             <td>
                 <form:input path="zip" id="zip" name="zip" type="text" placeholder="Zip" size="5"/>
             </td>
             <td >
                 <form:input path="city" id="city" name="city" type="text" placeholder="Stad" size="80%"/>
 
             </td>
         </tr>
         <tr>
             <td>
                 <label>Contactgegevens:</label>
             </td>
             <td>
                 <form:input path="telephone" id="telephone" name="telephone" type="text" placeholder="Telefoonnummer"/>
             </td>
             <td >
                 <form:input path="email" id="email" name="email" type="e-mail" placeholder="E-mailadres" size="100%"/>
             </td>
         </tr>
         <tr>
             <td>
                 <label for="notes" style="vertical-align: middle">Opmerkingen:</label>
             </td>
             <td >
                 <form:textarea path="notes" id="notes" name="notes" style="width: 100%"/>
             </td>
         </tr>
         <tr></tr>
         <tr></tr>
         <tr>
             <td></td>
             <td  ><input type ="submit" value="Opslaan" class="button"/></td>
         </tr>
     </table>
</form:form>
<%@include file="footer.jsp" %>
</body>
</html>

The GeneralEntity:

@Entity
@Table(name = "General")
public class General implements Serializable{
    @Id
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    @Column(name = "id")
    private Long id;
    @Column(name = "Language")
    private String language;
    @Column(name = "ApplicationDate")
    private LocalDate application;
    @Column(name = "KboName", unique = true)
    private String nameKbo;
    @Column(name = "KboNumber")
    private long kboNumber;
    @JoinColumn(name = "id_address", referencedColumnName = "id")
    @OneToOne(fetch = FetchType.EAGER, cascade = CascadeType.ALL)
    private Address address;
    @Column
    private String applicationType;
    @Column
    private String notes;
    @JoinColumn(name = "id_invoices", referencedColumnName = "id")
    @OneToOne(fetch = FetchType.EAGER, cascade = CascadeType.ALL)
    private Invoices invoice;
    @JoinColumn(name = "id_additional", referencedColumnName = "id")
    @OneToOne(fetch = FetchType.EAGER, cascade = CascadeType.ALL)
    private Additional additional;
    @ElementCollection(fetch = FetchType.LAZY)
    private List <Employee> employees = new ArrayList();
    private File appendix1;

public General() {
    }

public General(String language, LocalDate application, String nameKbo, long kboNumber, Address address,
                   String applicationType, String notes, Invoices invoice, Additional additional, List<Employee> employees,
                   File apendix1) {
        this.language = language;
        this.application = application;
        this.nameKbo = nameKbo;
        this.kboNumber = kboNumber;
        this.address = address;
        this.applicationType = applicationType;
        this.notes = notes;
        this.invoice = invoice;
        this.additional = additional;
        this.employees = employees;
        this.appendix1 = apendix1;
    }

public Long getId() {
        return id;
    }

public void setId(Long id) {
        this.id = id;
    }

public String getLanguage() {
        return language;
    }

public void setLanguage(String language) {
        this.language = language;
    }

public LocalDate getApplication() {
        return application;
    }

public void setApplication(LocalDate application) {
        this.application = application;
    }

public String getNameKbo() {
        return nameKbo;
    }

public void setNameKbo(String nameKbo) {
        this.nameKbo = nameKbo;
    }

public long getKboNumber() {
        return kboNumber;
    }

public void setKboNumber(long kboNumber) {
        this.kboNumber = kboNumber;
    }

public Address getAddress() {
        return address;
    }

public void setAddress(Address address) {
        this.address = address;
    }

public String getApplicationType() {
        return applicationType;
    }

public void setApplicationType(String applicationType) {
        this.applicationType = applicationType;
    }

public String getNotes() {
        return notes;
    }

public void setNotes(String notes) {
        this.notes = notes;
    }

public Invoices getInvoice() {
        return invoice;
    }

public void setInvoice(Invoices invoice) {
        this.invoice = invoice;
    }

public Additional getAdditional() {
        return additional;
    }

public void setAdditional(Additional additional) {
        this.additional = additional;
    }

public List<Employee> getEmployees() {
        return employees;
    }

public void setEmployees(List<Employee> employees) {
        this.employees = employees;
    }

public File getAppendix1() {
        return appendix1;
    }

public void setAppendix1(File appendix1) {
        this.appendix1 = appendix1;
    }
}

The Controller:

package be.vigilis.controllers;

import be.vigilis.entities.Address;
import be.vigilis.entities.General;
import be.vigilis.entities.Invoices;
import be.vigilis.entities.WrapperClass;
import be.vigilis.repositories.GeneralRepo;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.stereotype.Controller;
import org.springframework.web.bind.annotation.GetMapping;
import org.springframework.web.bind.annotation.ModelAttribute;
import org.springframework.web.bind.annotation.PostMapping;

import java.util.Map;

@Controller
public class NewController {

@Autowired
    private GeneralRepo repo;
    private General general = new General();

@GetMapping("new")
    public String getNew(Map<String, Object> model) {
        WrapperClass wrapper = new WrapperClass();
        model.put("wrapperForm", wrapper);
        return "newRecord";
    }

@PostMapping("new")
    public String handlePost(@ModelAttribute("wrapperForm") WrapperClass wrapper, Map<String, Object> model) {
        Address address = new Address(wrapper.getNameOrg(), wrapper.getStreet(), wrapper.getNumber(), wrapper.getZip(),
                wrapper.getCity(), wrapper.getTelephone(), wrapper.getEmail());
        general.setLanguage(wrapper.getLanguage());
        general.setApplication(wrapper.getApplication());
        general.setNameKbo(wrapper.getNameKbo());
        general.setKboNumber(wrapper.getKboNumber());
        general.setAddress(address);
        general.setApplicationType(wrapper.getApplicationType());
        general.setNotes(wrapper.getNotes());
        general.setAddress(address);
        if (general.getApplicationType().equalsIgnoreCase("voortzetting")) {
            return handleGetInvoice(model);
        } else {
            return "employee";
        }
    }

@GetMapping("invoices")
    public String handleGetInvoice(Map<String, Object> model) {
        Invoices invoiced = new Invoices();
        model.put("invoiceForm", invoiced);
        return "invoices";
    }

@PostMapping("invoices")
    public String handlePost(@ModelAttribute("invoiceForm") Invoices invoiced, Map<String, Object> model) {
        System.out.println("boe");
        general.setInvoice(invoiced);
        System.out.println("invoice saved");
        return "employee";
    }
}

I thought of making an menu-entry for every form I use, instead of redirecting, but I still have to acces the generalentity to update it with the new entities created.
Any ideas...?

It's good to be able to use someting, it's better to understand how it works.
www.goanation.net

Al Hobbs

Bartender

Posts: 669

I like...

posted 6 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

If the forms are on different pages, you can just make the response to receiving the first form, the second form. If they are on the same page you can maybe use javascript to auto focus on the second form.

Daniel Demesmaecker

Rancher

Posts: 1171

I like...

posted 6 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

They're on seperate pages. I don't really know what you mean with you remark. Apperently I cant, that is what I have bin trying...

It's good to be able to use someting, it's better to understand how it works.
www.goanation.net

Randy Tong

Rancher

Posts: 666

I like...

posted 6 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

Have you tried by adding a "/" ?

Life is but a BREATH

Daniel Demesmaecker

Rancher

Posts: 1171

I like...

posted 6 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

The spring.mvc.view.prefix in my application.properties is specified as "/WEB-INF/jsp/", so no additional "/" needed

It's good to be able to use someting, it's better to understand how it works.
www.goanation.net

Al Hobbs

Bartender

Posts: 669

I like...

posted 6 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

If they are on separate jsp the you should be able to just receive the post of the first form then for the return you just put the name of the Jsp that has the second form. That should work if your config is all setup. Should check through it for any small mistakes. Do any of your jsps load properly?

Daniel Demesmaecker

Rancher

Posts: 1171

I like...

posted 6 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

That dosn't work.
And yes my wrapperform uused to populate the general and address table loads properly and the postmethod also works. The data is being saved correctly. But if I simply redirect to the next form from my postmethod, the next form isn't added as a n attribute to the model. I wrote the reasson here for is cause I redirect it myself spring dosn't create a bean.
And what configuration, spring boot does all the configuration, I've done just some basicstuff in application.properties

It's good to be able to use someting, it's better to understand how it works.
www.goanation.net

Al Hobbs

Bartender

Posts: 669

I like...

posted 6 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

Ok finally getting what you're saying. Redirect to an endpoint that will put the model attribute in before returning the jsp. Or you don't even have to use model attribute if you want a more simple way

Daniel Demesmaecker

Rancher

Posts: 1171

I like...

posted 6 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

Adding the form in the generalpostmethod and redirecting instead of just calling the new form takes care of the bean problem, but still when posting the second form, the postmethod of the invoicecontroller isn't called.

It's good to be able to use someting, it's better to understand how it works.
www.goanation.net

Al Hobbs

Bartender

Posts: 669

I like...

posted 6 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

So when you you hit submit from the second form page nothing happens or what? Have you turned on the debug log for spring so you can see what's happening? It will say what requests it receives and what the mapping is. That way you can see where the problem is.

Al Hobbs

Bartender

Posts: 669

I like...

posted 6 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

Which jsp file is one that is not posting? One of your jsp forms doesn't have an action which would make sense that nothing is being posted.

Daniel Demesmaecker

Rancher

Posts: 1171

I like...

posted 6 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

It's the one with action that isn't posting, it dosn't need the action attribute if you specify the method, the eonly reasson I put in the action is cause it wasn't posting and hope that would make a difference, but it didn't

It's good to be able to use someting, it's better to understand how it works.
www.goanation.net

Al Hobbs

Bartender

Posts: 669

I like...

posted 6 years ago

Number of slices to send:

Optional 'thank-you' note:

Send

Didn't know you could post something from a form with no action or JavaScript. Maybe that's a function of spring form tag. Either way you should look at the log, because if you don't know the problem, you'll be able to figure it out pretty easily.

keep an eye out for scorpions and black widows. But the tiny ads are safe.

Gift giving made easy with the permaculture playing cards

https://coderanch.com/t/777758/Gift-giving-easy-permaculture-playing