feature/basic_functions #7

Merged
Seil0 merged 30 commits from feature/basic_functions into master 2020-05-10 17:48:31 +02:00
Collaborator

Includes:
Shop/Articles
Shop/Mainpage
Shop/Checkout

Does not Include:
Tracking API
Shop/Search

Includes: Shop/Articles Shop/Mainpage Shop/Checkout Does not Include: Tracking API Shop/Search
Seil0 was assigned by CodeSteak 2020-05-10 15:41:22 +02:00
Seil0 reviewed 2020-05-10 16:09:22 +02:00
@ -0,0 +9,4 @@
public static List<Article> getRandomArticles(int quantity, List<Article> advertisedArticles) {
List<Article> randomisedArticles = new ArrayList<Article>();
int loopcount = quantity > advertisedArticles.size() ? advertisedArticles.size() : quantity;
Collaborator

Replace with Math.min?

Replace with Math.min?
Collaborator

fixed

fixed
Seil0 reviewed 2020-05-10 16:12:18 +02:00
@ -154,0 +150,4 @@
// @GetMapping("/privacy")
// public String privacy() {
// return "privacy";
// }
Collaborator

Wenn's nicht mehr gebraucht wird dann bitte raus damit.

Wenn's nicht mehr gebraucht wird dann bitte raus damit.
Author
Collaborator

Fixied in 777786f5b6

Fixied in 777786f5b6faa1def5cf562983511fc24de7ef93
Seil0 reviewed 2020-05-10 16:15:37 +02:00
@ -0,0 +12,4 @@
@Override
public boolean preHandle(
HttpServletRequest request, HttpServletResponse response, Object handler) throws Exception {
Collaborator

Laut Intellij werden die Exceptions nie geworfen.

Laut Intellij werden die Exceptions nie geworfen.
Author
Collaborator

Ist wegen dem Interface, die Methode ist dort so deklariert. Daher würde ich das gerne so lassen, zwecks Konsistenz

Ist wegen dem Interface, die Methode ist dort so deklariert. Daher würde ich das gerne so lassen, zwecks Konsistenz
Collaborator

Ok

Ok
Seil0 reviewed 2020-05-10 16:17:19 +02:00
@ -4,0 +20,4 @@
import java.io.IOException;
import java.io.InputStream;
import java.util.ArrayList;
import java.util.List;
Collaborator

Unused Imports bitte entfernen.

Unused Imports bitte entfernen.
Seil0 reviewed 2020-05-10 16:19:11 +02:00
@ -8,0 +39,4 @@
HttpServletResponse response
) {
Collaborator

Unnötige Leerzeilen entfernen.

Unnötige Leerzeilen entfernen.
Collaborator

fixed

fixed
Seil0 reviewed 2020-05-10 16:21:58 +02:00
@ -2,1 +2,4 @@
import org.hso.ecommerce.action.shop.GetRandomArticlesAction;
import org.hso.ecommerce.entities.shop.Article;
import org.hso.ecommerce.entities.user.User;
Collaborator

Unused Import.

Unused Import.
Seil0 reviewed 2020-05-10 16:24:50 +02:00
@ -30,0 +31,4 @@
public BookingReason() {
}
;
Collaborator

Das kann auch weg.

Das kann auch weg.
Author
Collaborator

Nope, ORM braucht nen leeren Konstruktor. Ich mach ein Kommentar drüber. 713c8ebe86

Nope, ORM braucht nen leeren Konstruktor. Ich mach ein Kommentar drüber. 713c8ebe86faffcde94f076956bdc5d02fe3ddaa
Collaborator

Ich meinte auch das Semikolon.

Ich meinte auch das Semikolon.
Seil0 reviewed 2020-05-10 16:26:43 +02:00
@ -16,2 +14,3 @@
private byte[] data;
public String path;
}
Collaborator

Diese Leerzeile ist zu viel.

Diese Leerzeile ist zu viel.
Seil0 reviewed 2020-05-10 16:29:26 +02:00
@ -18,3 +20,2 @@
public int newSumArticles;
public int newSumWarehousePosition;
// Can;t do, does not work when created in action.
Collaborator

; > '

; > '
Author
Collaborator

removed bd2aeb63f6

removed bd2aeb63f6ea5a3c0d2f78cde2ff8f5691061f30
Seil0 reviewed 2020-05-10 16:43:49 +02:00
@ -0,0 +1,6 @@
INSERT INTO article_offers ("manufacturer", "article_number", "vat_percent")
VALUES ("McDonalds", "1", 7);
Collaborator

"should_be_advertised" fehlt hier.

"should_be_advertised" fehlt hier.
Author
Collaborator

@Hannes, your change

@Hannes, your change
Collaborator

fixed it

fixed it
Seil0 approved these changes 2020-05-10 17:34:05 +02:00
Seil0 left a comment
Collaborator

Kann gemerged werden.

Kann gemerged werden.
Seil0 closed this pull request 2020-05-10 17:48:31 +02:00
Seil0 deleted branch feature/basic_functions 2020-05-10 17:49:32 +02:00
This repo is archived. You cannot comment on pull requests.
No description provided.