Hibernate occurs too many queries - spring

I use hibernate(with ehcache)/spring/mysql/jsf.I'am getting data from two tables in mysql.And these tables are as follows:
User
id int (primary key)
name etc.
Lesson
id int (primary key)
lesson varchar
teacher_id int (foreign key from user(id) )
and Lesson model class:
#Entity
#Table(name = "oys_lesson")
#Cache(usage = CacheConcurrencyStrategy.READ_WRITE, region = "myregion")
public class Lesson {
#Id
#GeneratedValue(strategy = GenerationType.AUTO)
#Column(name = "id", unique = true, nullable = false)
private Integer id;
#Column(name="lesson")
private String lessonName;
#ManyToOne(cascade=CascadeType.ALL,fetch=FetchType.LAZY)
#JoinColumn(name="teacher_id",nullable=false)
private User user;
//Getters and Setters
User model Class
#Entity
#Table(name = "oys_user")
#Cache(usage = CacheConcurrencyStrategy.READ_WRITE, region = "myregion")
public class User implements Serializable {
private static final long serialVersionUID = 1L;
#Id
#GeneratedValue(strategy = GenerationType.AUTO)
#Column(name = "id", unique = true, nullable = false)
private Integer id;
private String username;
etc....
#OneToOne(cascade = CascadeType.REMOVE)
#JoinTable(name = "oys_user_role", joinColumns = { #JoinColumn(name = "user_id", referencedColumnName = "id") }, inverseJoinColumns = { #JoinColumn(name = "role_id", referencedColumnName = "id") })
private Role role;
#OneToMany(cascade = CascadeType.ALL, fetch = FetchType.LAZY, mappedBy = "user")
private Set<Lesson> teacherLessons;
//Getters and Setters
I'm getting lesson list with this daoimpl.
public List<Lesson> getLessonList() {
// TODO Auto-generated method stub
String username = SecurityContextHolder.getContext()
.getAuthentication().getName();
Query query = openSession()
.createQuery(
"from Lesson l where l.user in (select id from User u where u.username=:username)");
query.setParameter("username", username);
// query.setCacheable(true);
List<Lesson> lessonList = query.list();
if (lessonList.size() > 0)
return lessonList;
return null;
}
This query is getting lessons of the current user.My jsf page below:
<h:form id="form">
<p:dataTable var="lesson" value="#{lessonManagedBean.lessonList}"
paginator="true" rows="10" rowKey="#{lesson.id}"
paginatorTemplate="{RowsPerPageDropdown} {FirstPageLink} {PreviousPageLink} {CurrentPageReport} {NextPageLink} {LastPageLink}"
rowsPerPageTemplate="5,10,15" selectionMode="single"
selection="#{lessonManagedBean.selectedLesson}" id="carTable" lazy="true">
<p:ajax event="rowSelect" listener="#{lessonManagedBean.onRowSelect}"
update=":form:lessonDetail" oncomplete="PF('lessonDialog').show()" />
<p:column headerText="id" sortBy="#{lesson.id}"
filterBy="#{lesson.id}">
<h:outputText value="#{lesson.id}" />
</p:column>
<p:column headerText="Lesson Name" sortBy="#{lesson.lessonName}"
filterBy="#{lesson.lessonName}">
<h:outputText value="#{lesson.lessonName}" />
</p:column>
</p:dataTable>
<p:dialog header="Ders Detayları" widgetVar="lessonDialog" showEffect="fade"
hideEffect="fade" resizable="false">
<p:outputPanel id="lessonDetail" style="text-align:center;">
<p:panelGrid columns="2"
rendered="#{not empty lessonManagedBean.selectedLesson}"
columnClasses="label,value">
<h:outputText value="Id:" />
<h:outputText value="#{lessonManagedBean.selectedLesson.id}" />
<h:outputText value="Lesson Name" />
<h:outputText value="#{lessonManagedBean.selectedLesson.lessonName}" />
</p:panelGrid>
</p:outputPanel>
</p:dialog>
</h:form>
When I opened this page,I get the following output in console.
Hibernate: select lesson0_.id as id0_, lesson0_.lesson as lesson0_, lesson0_.teacher_id as teacher3_0_ from oys_lesson lesson0_
where lesson0_.teacher_id in (select user1_.id from oys_user user1_
left outer join oys_user_role user1_1_ on user1_.id=user1_1_.user_id
where user1_.username=?)
When page opened,This query is running 3 times.3 of the same query.But when I selected row.dialog window opens and running the same query more 19 times.
managedBean.class
#ManagedBean
#SessionScoped
public class LessonManagedBean implements Serializable {
private Lesson selectedLesson=null;
List<Lesson> lessonList;
//Getters and Setters
Why is running the same query repeatedly?When the dialog window opens, do not need to run any query?Why it works 19 times?thanks in advance..

I solved my problem.Hibernate occurs too many queries.Because JSF calls getters multiple times.So my getList methods works multiple times.
This usually is not seen as a major problem.Because getter method is a very cheap operation.
But, if you're performing expensive business logic(db operation) in the getter method (like me),this would be re-executed everytime.So, the same query(in the getter method) works again and again.
The simplest solution:
public List<Object> getPropList(){
if(propList==null){
propList=loadListFromDb();
}
return propList;
}
For more details:
Why JSF calls getters multiple times?

Number of queries is dependent on FetchMode, by default it is FetchMode.SELECT, so it will fire select statements, change it to FetchMode.SUBSELECT or FetchMode.JOIN and check the queries.
#OneToMany(cascade = CascadeType.ALL, fetch = FetchType.LAZY, mappedBy = "user")
#Fetch(FetchMode.SUBSELECT)
private Set<Lesson> teacherLessons;

Related

Can you render jsp/html elements (i.e. a form:form) dynamically in one page?

I am new to web-dev, and coding in general, so I apologize if this question doesn't make sense.
Background: I am working on a web application utilizing the Spring Framework. My application allows users to register a user account, post pictures, make comments on pictures, like pictures etc. (think of a mini Pintrest clone). Users and comments (along with some other non-relevant entities) are all saved as in table entities in a MySQL database.
Problem: I would like users be able to edit their comments after they post them. For the best user experience I do not want users to be taken to a separate page with a form to edit their comment. I would like user to be able to edit their comment in a single page/the same page, without rendering a new one. Flow goes something like this: User goes to their comment on a picture, presses an "edit" button/link and the comment is put in a text box (form:form) which is then editable.
So far I have tried conditional rendering with a boolean switch of "editPressed" but I can't seem to work out the logic. Is this the correct approach or should something like JS be used here? (not sure how js handles form:forms) Below are some code snippets of the models and the .jsp where the comment would be edited.
.jsp:
<c:forEach var="eachComment" items="${allCommentsByPhotoId}">
<div class="card-text d-flex gap-2 mb-4">
<img class="user-comment-card" src="https://cdn-icons-png.flaticon.com/512/1053/1053244.png" alt="" />
<div>
<div class="mb-2">
<c:choose>
<c:when test="${ editPressed == false }">
<c:out value="${eachComment.user.getFirstName()} says: ${ eachComment.getComment() }"></c:out>
</c:when>
<c:otherwise>
<span class="username">
<c:out value="${eachComment.user.getFirstName()}"></c:out>:
</span>
<c:out value="${ eachComment.getComment()}"></c:out>
</c:otherwise>
</c:choose>
</div>
<div class="d-flex align-items-center gap-3">
<!-- -->
<c:if test="${ eachComment.user.getId() == currentUser.id }">
<form:form action="/delete/${eachComment.getId()}" method="delete">
<p class="submit">
<input class="btn btn-danger del-cmt-btn" type="submit" value="Delete" />
</p>
</form:form>
<form:form action="/edit/comment/${eachComment.getId()}">
<p class="submit">
<input class="btn btn-secondary edit-cmt-btn" type="submit" value="Edit" />
</p>
</form:form>
</c:if>
</div>
</div>
</div>
</c:forEach>
Comment Model:
#Entity
#Table(name="user_comments_on_photo")
public class Comment {
#Id
#GeneratedValue(strategy = GenerationType.IDENTITY)
private Long id;
private String comment;
#ManyToOne(fetch = FetchType.LAZY)
#JoinColumn(name="user_id")
private User user;
#ManyToOne(fetch = FetchType.LAZY)
#JoinColumn(name="photo_id")
private Photo photo;
#ManyToMany(fetch=FetchType.LAZY)
#JoinTable(
name="user_likes_comment",
joinColumns = #JoinColumn(name="comment_id"),
inverseJoinColumns = #JoinColumn(name="user_id")
)
private List<User> usersWhoLikeComment;
#Column(updatable=false)
#DateTimeFormat(pattern="yyyy-MM-dd")
private Date createdAt;
#DateTimeFormat(pattern="yyyy-MM-dd")
private Date updatedAt;
USER MODEL:
#Id
#GeneratedValue(strategy=GenerationType.IDENTITY)
private Long id;
#Size(min=3)
private String firstName;
#Size(min=3)
private String lastName;
#Email
private String email;
#Size(min=8)
private String password;
#Transient
private String passwordConfirmation;
#Column(updatable=false)
private Date createdAt;
private Date updatedAt;
//RELATIONSHIPS
#OneToMany(mappedBy="user", fetch = FetchType.LAZY)
private List<Photo> photos;
#ManyToMany(fetch = FetchType.LAZY)
#JoinTable(
name = "user_likes_photo",
joinColumns = #JoinColumn(name="user_id"),
inverseJoinColumns = #JoinColumn(name="photo_id")
)
private List<Photo> likedPhotos;
//NEW ONE-TO-MANY
#OneToMany(mappedBy = "user", fetch = FetchType.LAZY)
private List<Comment> userComments;
#ManyToMany(fetch=FetchType.LAZY)
#JoinTable(
name="user_likes_comment",
joinColumns = #JoinColumn(name="user_id"),
inverseJoinColumns = #JoinColumn(name="comment_id")
)
private List<Comment> commentsUserLikes;
#ManyToMany(fetch = FetchType.EAGER)
#JoinTable(
name = "users_roles",
joinColumns = #JoinColumn(name = "user_id"),
inverseJoinColumns = #JoinColumn(name = "role_id"))
private List<Role> roles;
PHOTO MODEL:
#Entity
#Table(name="photos")
public class Photo {
#Id
#GeneratedValue(strategy = GenerationType.IDENTITY)
private Long id;
private String photoURL;
private String photoFileName;
#NotEmpty
private String photoTitle;
#NotEmpty
private String photoDescription;
//will add tags later
//RELATIONSHIPS
#ManyToOne(fetch = FetchType.LAZY)
#JoinColumn(name="user_id")
private User user;
#ManyToMany(fetch=FetchType.LAZY)
#JoinTable(
name="user_likes_photo",
joinColumns = #JoinColumn(name="photo_id"),
inverseJoinColumns = #JoinColumn(name="user_id")
)
private List<User> usersWhoLikePhoto;
//NEW ONE-TO-MANY
#OneToMany(mappedBy="photo", fetch = FetchType.LAZY)
private List<Comment> comments;
//CREATED AND UPDATED AT
#Column(updatable=false)
#DateTimeFormat(pattern="yyyy-MM-dd")
private Date createdAt;
#DateTimeFormat(pattern="yyyy-MM-dd")
private Date updatedAt;

How to prevent user from injecting field into form backing bean?

An user upload his comment via this form.
Thymeleaf
<form th:action="#{/comment}" th:id="form" method="post">
<input type="hidden" th:name="productId.id" th:value="${product.id}">
<textarea th:field="${comment.message}" class="comment"
placeholder="Write comment here"></textarea>
<input type="submit" id="submit" value="comment">
</form>
Actual HTML
<form action="/comment" id="form" method="post" class="">
<input type="hidden" name="_csrf" value="f6b3f296-3284-4d2d-a2b2-0a9975f5e071">
<input type="hidden" name="productId.id" value="38">
<textarea class="comment" placeholder="Write comment here" id="message" name="message"></textarea>
<input type="submit" id="submit" value="comment">
</form>
However if user overwrites the actual HTML like this, the product's name will be changed to "ABCD"
<form action="/comment" id="form" method="post" class=""><input type="hidden" name="_csrf" value="f6b3f296-3284-4d2d-a2b2-0a9975f5e071">
<input type="hidden" name="productId" value="38">
<input type="hidden" name="productId.name" value="ABCD">
<textarea class="comment" placeholder="Write comment here" id="message" name="message"></textarea>
<input type="submit" id="submit" value="comment">
</form>
I think what happened here is Spring queried the productId and it became managed Entity, and when the user set the name to be "ABCD", it would be saved.
Here is my solution:
Basically just use #Validated with a bunch of groups and put constraint with appropriate groups (UploadCommentValidation in this case) on every single field, which works but seems really messy especially when it gets big.
Example with upload comment above:
Comment Entity: productId and message must be #Not Null, productId must be #Valid,other fields must be #Null
Product Entity: Id must be #NotNull, other fields must be #Null
Comment entity
public class Comment implements Comparable<Comment> {
#Id
#GeneratedValue(strategy = GenerationType.IDENTITY)
#Null(groups = {UploadCommentValidation.class})
#NotNull(groups = {DeleteCommentValidation.class, UpdateCommentValidation.class})
private Integer id;
#ManyToOne
#JoinColumn(name = "product_id", referencedColumnName = "id")
#JsonBackReference
#Valid
#NotNull(groups = {UploadCommentValidation.class})
#Null(groups = {DeleteCommentValidation.class, UpdateCommentValidation.class})
private Product productId;
#ManyToOne
#JoinColumn(name = "user_id", referencedColumnName = "id")
#JsonBackReference
#Null(groups = {UploadCommentValidation.class, DeleteCommentValidation.class, UpdateCommentValidation.class})
private User userId;
#Column(name = "message")
#NotBlank(message = "please write a comment", groups = {UploadCommentValidation.class, UpdateCommentValidation.class})
#Null(groups = {DeleteCommentValidation.class})
private String message;
#Column(name = "created_at", insertable = false, columnDefinition = "timestamp with time zone not null")
#Temporal(TemporalType.TIMESTAMP)
#Null(groups = {UploadCommentValidation.class, DeleteCommentValidation.class, UpdateCommentValidation.class})
private Calendar createdAt;
#Column(name = "updated_at", columnDefinition = "timestamp with time zone not null")
#Temporal(TemporalType.TIMESTAMP)
#Null(groups = {UploadCommentValidation.class, DeleteCommentValidation.class, UpdateCommentValidation.class})
private Calendar updatedAt;
#Override
public int compareTo(Comment o) {
return this.getId().compareTo(o.getId());
}
}
Product entity
public class Product implements Serializable {
#Id
#GeneratedValue(strategy = GenerationType.IDENTITY)
#NotNull(message = "product id null", groups = {AddOrderValidation.class, UploadCommentValidation.class})
#Null(message = "bad request", groups = {ProductRegisterValidation.class})
private Integer id;
#NotBlank(message = "please fill in product name", groups = {ProductRegisterValidation.class})
#Length(max = 255, message = "too long", groups = {ProductRegisterValidation.class})
#Null(groups = {AddOrderValidation.class, UploadCommentValidation.class})
#Column(name = "name")
private String name;
#Column(name = "price")
#Positive(message = "the price must be non-negative", groups = {ProductRegisterValidation.class})
#NotNull(message = "please fill in price", groups = {ProductRegisterValidation.class})
#Null(groups = {AddOrderValidation.class, UploadCommentValidation.class})
private Integer price;
#ManyToOne(fetch = FetchType.LAZY)
#JoinColumn(name = "category_id", referencedColumnName = "id")
#Valid
#NotNull(message = "please select category name", groups = {ProductRegisterValidation.class})
#Null(groups = {AddOrderValidation.class, UploadCommentValidation.class})
private Category categoryId;
#NotBlank(message = "please fill in description", groups = {ProductRegisterValidation.class})
#Length(max = 10000, message = "too long", groups = {ProductRegisterValidation.class})
#Null(groups = {AddOrderValidation.class, UploadCommentValidation.class})
#Column(name = "description")
private String description;
#OneToMany(mappedBy = "product", cascade = CascadeType.ALL, fetch = FetchType.LAZY)
#Null(groups = {ProductRegisterValidation.class, AddOrderValidation.class, UploadCommentValidation.class})
private List<ProductImage> productImages;
#OneToOne(mappedBy = "product", cascade = CascadeType.ALL, fetch = FetchType.LAZY)
#Null(groups = {ProductRegisterValidation.class, AddOrderValidation.class, UploadCommentValidation.class})
private Thumbnail thumbnail;
#OneToMany(mappedBy = "productId", fetch = FetchType.LAZY)
#JsonManagedReference
#Null(groups = {ProductRegisterValidation.class, AddOrderValidation.class, UploadCommentValidation.class})
private List<Comment> comments;
#OneToMany(mappedBy = "product", fetch = FetchType.LAZY)
#Null(groups = {ProductRegisterValidation.class, AddOrderValidation.class, UploadCommentValidation.class})
private List<Order> orders;
}
Any ideas how to do it the right way? This seems super messy!
UPDATE 1: This is my rest controller
#PostMapping("/comment")
public ResponseEntity<Map<String, String>> commentResponseEntity(#Validated({UploadCommentValidation.class}) Comment comment, BindingResult result) {
if (result.hasErrors()) {
result.getAllErrors().forEach(System.out::println);
return ResponseEntity.noContent().build();
}
User user = getUser();
comment.setUserId(user);
commentRepository.saveAndFlush(comment);
Map<String, String> response = new HashMap<>();
response.put("comment", comment.getMessage());
response.put("user", user.getName());
response.put("commentId", comment.getId().toString());
return ResponseEntity.ok().body(response);
}
You can do this by registering an #InitBinder method
You can do this at the individual controller level or by registering a #ControllerAdvice to be applied to all, or a subset of all, controllers.
#InitBinder()
public void initBinder(WebDataBinder binder) {
binder.setDisallowedFields(new String[] { "id", "version" });
}

Spring MVC Error: Failed to convert property value of type java.lang.String to required type

I can't let this exception go:
Failed to convert property value of type java.lang.String to required type com.company.springdemo.entity.Product for property productId; nested exception is java.lang.IllegalStateException: Cannot convert value of type java.lang.String to required type com.company.springdemo.entity.Product for property productId: no matching editors or conversion strategy found
Order Model
#Entity
#Table(name = "orders") // naming the table only order, will throw exception
public class Order {
#Id
#GeneratedValue(strategy = GenerationType.AUTO)
#Column(name = "order_id")
private Integer orderId;
#OneToOne(cascade = {CascadeType.DETACH,CascadeType.MERGE,CascadeType.PERSIST,CascadeType.REFRESH})
#JoinColumn(name = "product_id")
private Product productId;
#ManyToOne(fetch = FetchType.EAGER, cascade = {CascadeType.DETACH,CascadeType.MERGE,CascadeType.PERSIST,CascadeType.REFRESH})
#JoinColumn(name = "client_id")
private Client client;
....
Product Model
#Entity
#Table(name = "product")
public class Product {
#Id
#GeneratedValue(strategy = GenerationType.AUTO)
#Column(name = "product_id")
private Integer id;
#Column(name = "product_name")
private String productName;
#Column(name = "product_serial")
private String productSerial;
...
Client Model
#Entity
#Table(name = "clients")
public class Client {
#Id
#GeneratedValue(strategy = GenerationType.AUTO)
private Integer id;
#NotEmpty
#Column(name = "first_name")
private String firstName;
#NotEmpty
#Column(name = "last_name")
private String lastName;
#NotEmpty
#Email
#Column(name = "email")
private String email;
#NotEmpty
#Column(name = "location")
private String location;
#OneToMany(mappedBy = "client",cascade = CascadeType.ALL, fetch = FetchType.EAGER)
private List<Order> orders;
Controller, where I save the order with related client and product
#PostMapping("add")
public ModelAndView addOrder( #Validated #ModelAttribute("ords") Order order, BindingResult bindingResult ){
if (bindingResult.hasErrors()) {
System.out.println("Having errors: " + bindingResult.getAllErrors());
Iterable<Product> products = productService.listProducts();
Iterable<Client> clients = clientService.listClients();
System.out.println("Error "+ bindingResult.getAllErrors());
ModelAndView mv = new ModelAndView("orders/add-order");
mv.addObject("products",products);
mv.addObject("clients",clients);
return mv;
}
try {
orderService.saveOrder(order);
} catch (Exception e) {
e.printStackTrace();
}
ModelAndView mv = new ModelAndView("redirect:list");
return mv;
}
Finally, my JSP form View page
<form:form action="add" method="post" modelAttribute="ords">
<label for="productId" >Product Id</label>
<form:select path="productId" >
<c:forEach var="product" items="${products}">
<form:option value="${product.id}">${product.productName}</form:option>
</c:forEach>
</form:select>
<form:errors path="productId"/>
<br>
<label for="client" >Client Id</label>
<form:select path="client" >
<c:forEach var="client" items="${clients}">
<form:option value="${client.id}">${client.id} - ${client.lastName}</form:option>
</c:forEach>
</form:select>
<form:errors path="client"/>
<br>
<input type="submit" value="Place Order">
</form:form>
What am I doing wrong?
You most likely need to build a converter class such as this one :
#Component("facilityConverter")
public class FacilityConverter implements Converter<String, Facility>
{
#Autowired
FacilityService facilityService;
#Override
public Facility convert(String id)
{
return facilityService.findById(Integer.parseInt(id));
}
}
Then, you need to register it by implementing the addFormatters method inside of a configuration class implementing WebMvcConfigurer like so :
#Override
public void addFormatters (FormatterRegistry registry)
{
registry.addConverter((FacilityConverter)ctx.getBean("facilityConverter"));
}
Your entities will then correctly be mapped from a dropdown selection. Also, this might not be part of your issue but you can just build your dropdowns like this :
<form:select name="linkedInterface" path="linkedInterface" id="linkedInterface">
<form:options items="${interfaces}" itemLabel="name" itemValue="id"/>
</form:select>
The productId field is actually a Product object, not an ID (String/int). You need your JSP to use path="productId.id" rather than path="productId".
(Although I'd also suggest you also rename the field product rather than productId.)
<form:select path="product.id">
I think you'll hit the same issue on your <form:select path="client"> too.

Spring 4 MVC Form: Create Object with list of sub objects

I looking for help in understanding how to create a new object through a form which has the user select multiple sub-objects (which will come pre-populated) and available to select with a checkbox.
OrderController.java
#RequestMapping(value = { "/order" }, method = RequestMethod.GET)
public String order(ModelMap model) {
List<Exam> exams = examService.findAllExams();
List<Document> documents = documentService.findAllDocuments();
model.addAttribute("exams", exams);
model.addAttribute("documents", documents);
return "order"; // jsp page reference
}
Order.java
#Entity
#Table(name="\"order\"")
public class Order implements Serializable
{
private static final long serialVersionUID = 1L;
#Id
#GeneratedValue(strategy=GenerationType.IDENTITY)
#Column(name = "order_id", unique = true, nullable = false)
private Integer id;
#GeneratedValue(generator = "uuid")
#GenericGenerator(name = "uuid", strategy = "uuid2")
#Column(name = "uuid", unique = true, nullable = false)
private String uuid;
#Temporal(TemporalType.DATE)
#Column(name = "order_date", unique = true, nullable = false)
private Date orderDate;
#Column(name="order_status", nullable=false)
private String orderStatus;
#ManyToOne(fetch = FetchType.LAZY)
#JoinColumn(name = "user_id", nullable = false)
private User user;
#OneToMany(fetch = FetchType.LAZY, mappedBy = "order")
private Set<OrderExam> orderExams = new HashSet<OrderExam>(0);
#OneToMany(fetch = FetchType.LAZY, mappedBy = "order")
private Set<OrderDocument> orderDocuments = new HashSet<OrderDocument(0);
//getters & setters
}
OrderExam.java
#Entity
#Table(name="order_exam")
public class OrderExam implements Serializable
{
private static final long serialVersionUID = 1L;
#Id
#GeneratedValue(strategy=GenerationType.IDENTITY)
#Column(name = "order_exam_id", unique = true, nullable = false)
private Integer id;
#ManyToOne(fetch = FetchType.LAZY)
#JoinColumn(name = "order_id", nullable = false)
private Order order;
#ManyToOne(fetch = FetchType.LAZY)
#JoinColumn(name = "exam_id", nullable = false)
private Exam exam;
#Column(name="exam_amount", nullable=true)
private Integer examAmount;
#Column(name="answer_sheet_amount", nullable=true)
private String answerSheetName;
#Column(name="students_per_csv", nullable=true)
private String studentsPerCSV;
#Column(name="pas", nullable=true)
private Boolean pearsonAnswerSheet;
//getters & setters
}
Exam.java
#Entity
#Table(name="exam")
public class Exam implements Serializable
{
private static final long serialVersionUID = 1L;
#Id
#GeneratedValue(strategy=GenerationType.IDENTITY)
#Column(name = "exam_id", unique = true, nullable = false)
private Integer id;
#NotEmpty
#Column(name="name", unique=true, nullable=false)
private String name;
#NotEmpty
#Column(name="code", unique=true, nullable=false)
private String code;
#OneToMany(fetch = FetchType.LAZY, mappedBy = "exam")
private Set<OrderExam> exams = new HashSet<OrderExam>(0);
//getters & setters
}
As you can see I am passing in a list of exams and documents which will populate a form with available options (can be seen in the image below (exams anyway)). The user needs to be able to select multiple rows, so that a single order has multiple exams and documents associated to it.
My order.jsp is a little much to post the entire thing here but here is the part I have which is displayed in the image above.
Order.jsp
<form:form method="POST" modelAttribute="order" class="form-horizontal form-label-left">
<c:forEach items="${exams}" var="exam">
<tr>
<th scope="row"><input type="checkbox" class="flat"></th>
<td><input id="middle-name" type="text" name="middle-name" readonly="readonly" value="${exam.name} - ${exam.code}" class="form-control col-md-7 col-xs-12"></td>
<td><input id="middle-name" type="text" name="middle-name" value="0" class="form-control col-md-3 col-xs-12"></td>
<td><input id="middle-name" type="text" name="middle-name" value="0" class="form-control col-md-3 col-xs-12"></td>
<td><input id="middle-name" type="text" name="middle-name" value="0" class="form-control col-md-3 col-xs-12"></td>
<c:choose>
<c:when test="${exam.name == 'Algebra 2 (Common Core)'}">
<th scope="row"><input type="checkbox" class="flat"></th>
</c:when>
<c:otherwise>
<th scope="row"></th>
</c:otherwise>
</c:choose>
</tr>
</c:forEach>
<!-- Other Stuff Goes Here -->
</form:form>
So in short, would someone be willing to show me how to set up the form in the way I described above? Thanks in advance.
Your question is a bit broad however you could try this as below. I have only covered exams. Principal for documents will be the same.
You will need a couple of new classes to capture the submitted form inputs:
Order Form to Capture Selections
public class OrderForm{
private List<ExamWrapper> allAvailableExams = new ArrayList<>();
private XOptionPrintWrapper selectedWrapper;
public OrderForm(){
}
//getters and setters
}
Exam Wrapper : Decorates an Exam with a 'selected' property
public class ExamWrapper{
private boolean selected;
private Exam exam;
public ExamWrapper(Exam exam){
this.exam = exams;
}
//getters and setters
}
Change Contoller to
public class OrderController{
//Exams model populated by the method below
//moved as we also need it populated on POST
#RequestMapping(value = { "/order" }, method = RequestMethod.GET)
public String order(ModelMap modelMap) {
//only needed on GET so put in model here
List<XOptionPrintWrapper> availableWrappers = //someList;
modelMap.put("availableWrappers", availableWrappers);
return "order";
}
//handles for submit
//model atribute is automatically populated by the framework
#RequestMapping(value = { "/order" }, method = RequestMethod.POST)
public String order(#ModelAttribute("orderForm") OrderForm orderForm) {
//process selected exams
return "nextView";
}
//on get populates the initial model for display
//on post create an instance which the form params will be bound to
#ModelAttribute("orderForm")
public OrderForm getOrderForm(){
OrderForm orderForm = new OrderForm();
List<Exam> exams = examService.findAllExams();
for(Exam exam : exams){
orderForm.getAllAvailableExams.add(new ExamWrapper(exam));
}
return orderForm;
}
}
In JSP use Sping support for binding to indexed properties:
<form:form method="POST" modelAttribute="orderForm" class="form-horizontal form-label-left">
<c:forEach items="${orderForm.allAvailableExams}" var="exam" varStatus="status">
<tr>
<th scope="row"><input name="allAvailableExams[${status.index}].selected"
type="checkbox" class="flat"></th>
</tr>
</c:forEach>
<form:select path="selectedWrapper">
<form:options items="${availableWrappers}"
itemValue="somePropertyOfXOptionPrintWrapper "
itemLabel="somePropertyOfXOptionPrintWrapper " />
</form:select>
</form>
I obviously haven't been able to try all of this but think it should all be okay.

#NotEmpty annotation validator for oneToMany

So I have the following Model:
#Entity
#Table(name = "Project")
public class Project implements java.io.Serializable {
#Id
#GeneratedValue(strategy = GenerationType.AUTO)
#Column(name = "PROJECT_ID", unique = true, nullable = false)
private int projectId;
private String name;
...
#ManyToOne
#JoinColumn(name = "MANAGER_ID")
#NotEmpty(message = "test")
private User manager;
....
}
For some reason the "#NotEmpty" annotation gives the following error when
the input is blank:
org.springframework.web.util.NestedServletException: Request processing failed; nested exception is javax.validation.UnexpectedTypeException: HV000030: No validator could be found for type: com.springmvc.model.User.
I am passing my values to the backend the following way:
<form:select path="manager" class="selectpicker form-control">
<form:option value="" >Alle managers</form:option>
<c:forEach var="manager" items="${managers}">
<form:option value="${manager.userId}" >${manager.getUserDetail().firstName}
${manager.getUserDetail().lastName}</form:option>
</c:forEach>
</form:select>
What am I doing wrong?
Replace #NotEmpty with #NotNull, that should fix your problem.

Resources