feature/customer_orders #77
无审核者
标签
未选择标签
bug
dependencies
duplicate
enhancement
help wanted
invalid
missing feature
question
refactoring
retest
review
tschüss
ui
wontfix
未选择里程碑
未指派成员
2 名参与者
到期时间
未设置到期时间。
依赖工单
没有设置依赖项。
参考:localhorst/e-commerce#77
正在加载...
在新工单中引用
屏蔽一个用户
没有提供说明。
删除分支 feature/customer_orders
删除分支是永久的。虽然已删除的分支在实际被删除前有可能会短时间存在,但这在大多数情况下无法撤销。是否继续?
Ich übernehme Review
@ -0,0 +14,4 @@
import java.util.*;
@Service
public class RestServiceForDelivery {
Name vlcht etwas zu ausführlich, aber OK :D
@ -0,0 +22,4 @@
public String getDeliveryID(CustomerOrder customerOrder) throws ResourceAccessException
{
String url = "http://[::1]:8082/newDelivery";
Besser hier Klassen-Konstate mit http://[::1]:8082
@ -0,0 +68,4 @@
{
return response.getBody();
} else {
return new DeliveryData("DHL-Server ist gerade nicht erreichbar","--:--:----");
Server ist in diesem Fall aber erreichbar.
Error Handling über Strings sollte hier nicht passieren, sondern erst in Templates
@ -0,0 +73,4 @@
}
catch (ResourceAccessException e)
{
return new DeliveryData("DHL-Server gerade nicht erreichbar","--:--:----");
Same
@ -3,9 +3,12 @@ package org.hso.ecommerce.app;
import org.springframework.boot.SpringApplication;
import org.springframework.boot.autoconfigure.SpringBootApplication;
import org.springframework.boot.autoconfigure.domain.EntityScan;
import org.springframework.context.ConfigurableApplicationContext;
Imports wurden hinzygefügt, aber die Klasse nicht verändert?
@ -51,1 +58,3 @@
model.addAttribute("orders", orders);
List<CustomerOrder> orders = customerOrderRepository.getAllOrders();
Map<CustomerOrder, DeliveryData> customerOrderDeliveryDataMap = orders.stream().collect
Formatierung komisch
@ -0,0 +1,75 @@
package org.hso.ecommerce.uiModel;
packetnamen sollten klein sein
@ -0,0 +28,4 @@
isDelivered = status.equals("Lieferung erfolgreich");
}
public static DeliveryData getDeliveryDataFromCustomerOrder(CustomerOrder customerOrder, CustomerOrderRepository customerOrderRepository, RestServiceForDelivery restServiceForDelivery)
Logic sollte in Action oder Controller stattfinden. UiModels sollten einfache Datenklassen sein
@ -0,0 +31,4 @@
public static DeliveryData getDeliveryDataFromCustomerOrder(CustomerOrder customerOrder, CustomerOrderRepository customerOrderRepository, RestServiceForDelivery restServiceForDelivery)
{
if(customerOrder.trackingId == null)
return new DeliveryData("Bestellung wurde elektronisch angekündigt","");
Strings im Idealfall eher im Template. Hier besser mit ENUMs oder booleans etc. arbeiten
@ -16,3 +16,3 @@
<nav></nav>
<div>
<h1>Bestellung 1010</h1>
<h1 th:text="|Bestellung ${order.id}|"></h1>
Wozu die "|" ?
Das hat Hannes so gemacht, es ging darum das man so Text reinschreiben kann wenn ich mich richtig erinnere.
Ahh okay
Also entweder ist ein schließendes "|" zu viel oder überall anders fehlts
@ -7,3 +8,4 @@
import java.util.Optional;
import static jdk.nashorn.internal.runtime.regexp.joni.Syntax.Java;
Bitte Imports checken
@ -26,3 +26,3 @@
<div class="content-width">
<div>
<h2 id="20202701"> Bestellung vom 27.01.2020 </h2>
<h2 th:text="| Bestellung vom ${order.created.toString().substring(0,10)}"></h2>
Schließendes "|" fehlt.
Nur Formatiert, passt.
@ -24,2 +24,2 @@
<div th:each="order: ${orders}">
<h2 id="20202701" th:text="|Bestellung vom ${order.formatCreated()}" />
<div th:each="order: ${orderDeliveryDataMap}">
<h2 id="20202701" th:text="|Bestellung vom ${order.getKey().formatCreated()}" />
Schließendes "|" fehlt
Nur Formatiert, passt.