feature/cash_bookings #55

Merged
Seil0 merged 6 commits from feature/cash_bookings into master 2020-06-10 19:04:45 +02:00
Collaborator
No description provided.
Seil0 added the
review
label 2020-06-09 14:07:12 +02:00
Seil0 requested changes 2020-06-09 21:43:59 +02:00
@ -0,0 +169,4 @@
}
@GetMapping("/intern/accounting/")
public String accounting(HttpServletRequest request) {
Collaborator

Statt request wird eigentlich model verwendet.

Statt `request` wird eigentlich `model` verwendet.
@ -0,0 +205,4 @@
}
@GetMapping("/intern/accounting/vat")
public String accountingVat(HttpServletRequest request) {
Collaborator

Statt request wird eigentlich model verwendet.

Statt `request` wird eigentlich `model` verwendet.
@ -0,0 +214,4 @@
}
@GetMapping("/intern/accounting/main")
public String accountingIntern(HttpServletRequest request) {
Collaborator

Statt request wird eigentlich model verwendet.

Statt `request` wird eigentlich `model` verwendet.
@ -0,0 +212,4 @@
} catch (NumberFormatException e) {
throw new InvalidFormDataException("Die angegebene Kunden-Nr. ist ungültig.");
}
return bookingAccountEntryRepository.getByUser(userId).or(() -> {
Collaborator

.or() ist erst ab Java 9 verfügbar

.or() ist erst ab Java 9 verfügbar
@ -0,0 +224,4 @@
} catch (NumberFormatException e) {
throw new InvalidFormDataException("Die angegebene Lieferanten-Nr. ist ungültig.");
}
return bookingAccountEntryRepository.getBySupplier(supplierId).or(() -> {
Collaborator

.or() ist erst ab Java 9 verfügbar

.or() ist erst ab Java 9 verfügbar
@ -4,3 +16,3 @@
@Controller
//@RequestMapping("...")
@RequestMapping("/intern/customers")
Collaborator

Wird das nicht in feature/customers gemacht?

Wird das nicht in feature/customers gemacht?
Author
Collaborator

Ich hab die Anzeige der Buchungen in der Kundenübersicht implementiert.
Dieser Branch ist auch schon mit feature/customers zusammengeführt.

Ich hab die Anzeige der Buchungen in der Kundenübersicht implementiert. Dieser Branch ist auch schon mit feature/customers zusammengeführt.
@ -36,3 +36,3 @@
@Autowired
private final BookingRepository bookingRepository = null;
private AccountingController accountingController = null;
Collaborator

Soll das nicht final sein?

Soll das nicht final sein?
@ -24,3 +24,3 @@
<div class="s">
<label for="amount">Betrag</label>
<input type="number" step="0.01" name="amount" value="0.00"/>&nbsp;EUR
<input type="number" step="0.01" th:field="*{amount}" />&nbsp;EUR
Collaborator

Die id für's label fehlt.

Die id für's label fehlt.
Author
Collaborator

Die Änderungen hab ich durchgeführt.

Die Änderungen hab ich durchgeführt.
Seil0 added a new dependency 2020-06-10 18:52:15 +02:00
Seil0 approved these changes 2020-06-10 19:04:40 +02:00
Seil0 left a comment
Collaborator

lgtm

lgtm
Seil0 closed this pull request 2020-06-10 19:04:45 +02:00
Seil0 deleted branch feature/cash_bookings 2020-06-10 19:04:52 +02:00
This repo is archived. You cannot comment on pull requests.
No description provided.