Implement Warehouse.... #47

Merged
Seil0 merged 6 commits from feature/intern_warehouse into master 2020-06-08 13:40:11 +02:00
Collaborator

Sry, only one commit. No time to make history ;D

Sry, only one commit. No time to make history ;D
CodeSteak changed title from WIP: Implement Warehouse.... to Implement Warehouse.... 2020-06-03 16:32:54 +02:00
CodeSteak changed title from Implement Warehouse.... to WIP: Implement Warehouse.... 2020-06-03 16:41:10 +02:00
Author
Collaborator

inDeliverySince wird noch nicht gesetzt

inDeliverySince wird noch nicht gesetzt
CodeSteak changed title from WIP: Implement Warehouse.... to Implement Warehouse.... 2020-06-03 16:55:54 +02:00
Seil0 added the
review
label 2020-06-03 18:36:46 +02:00
Seil0 requested changes 2020-06-04 12:05:32 +02:00
@ -0,0 +40,4 @@
HttpServletRequest request
) {
request.setAttribute("articles", articleRepository.findAll());
Collaborator

Dafür wird eigentlich model.addAttribute() verwendet.

Dafür wird eigentlich ` model.addAttribute()` verwendet.
Author
Collaborator

fixed in eb82d05f4f

fixed in eb82d05f4fe21c4659cc9d12db1b85ef09834b67
@ -0,0 +47,4 @@
}
@PostMapping("addManual")
public String warehouseAddMaualPost(
Collaborator

Statt request.setAttribute() wird eigentlich model.addAttribute() verwendet.

Statt `request.setAttribute()` wird eigentlich `model.addAttribute()` verwendet.
Author
Collaborator

fixed in eb82d05f4f

fixed in eb82d05f4fe21c4659cc9d12db1b85ef09834b67
@ -0,0 +94,4 @@
}
Optional<WarehouseBookingPositionSlotEntry> sourceSlot = Optional.empty();
if (sourceIsSlot == true) {
Collaborator

Warum nicht if (sourceIsSlot)?

Warum nicht `if (sourceIsSlot)`?
Author
Collaborator

sourceIsSlot könnte auch null sein

sourceIsSlot könnte auch null sein
@ -0,0 +104,4 @@
}
Optional<WarehouseBookingPositionSlotEntry> destinationSlot = Optional.empty();
if (destinationIsSlot == true) {
Collaborator

Warum nicht if (destinationIsSlot)?

Warum nicht `if (destinationIsSlot)`?
Author
Collaborator

destinationIsSlot könnte auch null sein

destinationIsSlot könnte auch null sein
@ -0,0 +24,4 @@
private final SlotRepository slotRepository = null;
@GetMapping("slots/")
public String accountingWarehouseSlots(
Collaborator

Statt request.setAttribute() wird eigentlich model.addAttribute() verwendet.

Statt `request.setAttribute()` wird eigentlich `model.addAttribute()` verwendet.
Author
Collaborator

fixed in eb82d05f4f

fixed in eb82d05f4fe21c4659cc9d12db1b85ef09834b67
@ -0,0 +22,4 @@
private final WarehouseBookingRepository warehouseBookingRepository = null;
@GetMapping("todo")
public String accountingWarehouseTodo(
Collaborator

Statt request.setAttribute() wird eigentlich model.addAttribute() verwendet.

Statt `request.setAttribute()` wird eigentlich `model.addAttribute()` verwendet.
Author
Collaborator

fixed in eb82d05f4f

fixed in eb82d05f4fe21c4659cc9d12db1b85ef09834b67
@ -0,0 +81,4 @@
}
@GetMapping("progress/{id}")
public String getProgressId(HttpServletRequest request,
Collaborator

Statt request.setAttribute() wird eigentlich model.addAttribute() verwendet.

Statt `request.setAttribute()` wird eigentlich `model.addAttribute()` verwendet.
Author
Collaborator

fixed in eb82d05f4f

fixed in eb82d05f4fe21c4659cc9d12db1b85ef09834b67
@ -23,3 +23,3 @@
<form class="detailgrid" th:action="@{/intern/warehouse/todo}">
<form class="detailgrid" method="post" th:action="@{/intern/warehouse/addManual}">
<div class="m">
<label for="amount">Artikelnummer</label>
Collaborator

Die id fehlt bei viele Inputs.

Die id fehlt bei viele Inputs.
Author
Collaborator

Wofür werden die IDs benötigt?

Wofür werden die IDs benötigt?
Seil0 approved these changes 2020-06-08 13:40:06 +02:00
Seil0 left a comment
Collaborator

lgtm

lgtm
Seil0 closed this pull request 2020-06-08 13:40:11 +02:00
Seil0 deleted branch feature/intern_warehouse 2020-06-08 13:40:18 +02:00
This repo is archived. You cannot comment on pull requests.
No description provided.