feature/customer_orders #77
No due date set.
No dependencies set.
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
No due date set.
No dependencies set.
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Du besuchst diese Seite mit einem veralteten IPv4-Internetzugang. Möglicherweise treten in Zukunft Probleme mit der Erreichbarkeit und Performance auf. Bitte frage deinen Internetanbieter oder Netzwerkadministrator nach IPv6-Unterstützung.
You are visiting this site with an outdated IPv4 internet access. You may experience problems with accessibility and performance in the future. Please ask your ISP or network administrator for IPv6 support.
Weitere Infos | More Information
Klicke zum schließen | Click to close
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.