feature/reorder_cronjob #27

Merged
Seil0 merged 11 commits from feature/reorder_cronjob into master 2020-05-29 16:20:05 +02:00
Collaborator

Den Code für die Bereitstellung der Supplier-Api habe ich hier hinzu gemerged.

Die Logik zur Ausführung von Cronjobs und der Reorder-Cronjobs sind implementiert.

Den Code für die Bereitstellung der Supplier-Api habe ich hier hinzu gemerged. Die Logik zur Ausführung von Cronjobs und der Reorder-Cronjobs sind implementiert.
Seil0 added the
review
label 2020-05-25 23:00:12 +02:00
Seil0 self-assigned this 2020-05-25 23:00:14 +02:00
Seil0 requested changes 2020-05-26 18:37:27 +02:00
Seil0 left a comment
Collaborator

Bitte das nächste mal mehr Kommentare.

Bitte das nächste mal mehr Kommentare.
@ -0,0 +45,4 @@
return input == null ? 0 : input;
}
private int getAmountInStock() {
Collaborator

Seit a2f20938cd gibt es getArticleStock() in WarehouseBookingPositionSlotEntryRepository. Kann man das hier verwendet werden?

Seit a2f20938cd gibt es getArticleStock() in WarehouseBookingPositionSlotEntryRepository. Kann man das hier verwendet werden?
@ -0,0 +70,4 @@
int i = null_to_zero(undeliveredReorders);
int l = getAmountInStock();
int o = n - i - l;
Collaborator

Ist redundant -> return n - i - l;

Ist redundant -> `return n - i - l;`
@ -0,0 +68,4 @@
@Component
class Reorder implements ICronjob {
private static final Logger log = LoggerFactory.getLogger(Reorder.class);
Collaborator

Wird nicht benutzt.

Wird nicht benutzt.
@ -0,0 +126,4 @@
Calendar threeDaysBefore = (Calendar) time.clone();
threeDaysBefore.add(Calendar.DAY_OF_MONTH, -3);
return new Integer[] { //
Collaborator

Für was sind die //?

Für was sind die `//`?
Author
Collaborator

Damit der Auto-Formatter das nicht umbaut.

Damit der Auto-Formatter das nicht umbaut.
@ -0,0 +6,4 @@
@Repository
public interface BackgroundJobRepository extends JpaRepository<BackgroundJob, Long> {
Collaborator

Was macht dieses Repository?

Was macht dieses Repository?
Author
Collaborator

Das wird in CronjobController.java:264 genutzt.

Bitte um Rückmeldung, falls das einfacher geht.

Das wird in [CronjobController.java:264](https://git.mosad.xyz/localhorst/e-commerce/src/commit/ffb683bdd0891ef6c376add65ea00b10e7595145/prototype/src/main/java/org/hso/ecommerce/controller/cronjob/CronjobController.java#L264) genutzt. Bitte um Rückmeldung, falls das einfacher geht.
@ -0,0 +4,4 @@
import org.springframework.data.jpa.repository.JpaRepository;
public interface ArticleOfferRepository extends JpaRepository<ArticleOffer, Long> {
Collaborator

Was macht dieses Repository?

Was macht dieses Repository?
Author
Collaborator

Gleicher Fall: Wird in CronjobController.java:151 genutzt.

Gleicher Fall: Wird in [CronjobController.java:151](https://git.mosad.xyz/localhorst/e-commerce/src/commit/ffb683bdd0891ef6c376add65ea00b10e7595145/prototype/src/main/java/org/hso/ecommerce/controller/cronjob/CronjobController.java#L151) genutzt.
@ -0,0 +6,4 @@
@Repository
public interface SupplierRepository extends JpaRepository<Supplier, Long> {
Collaborator

Was macht dieses Repository?

Was macht dieses Repository?
Author
Collaborator

Gleicher Fall: Wird in CronjobController.java:147 genutzt.

Gleicher Fall: Wird in [CronjobController.java:147](https://git.mosad.xyz/localhorst/e-commerce/src/commit/ffb683bdd0891ef6c376add65ea00b10e7595145/prototype/src/main/java/org/hso/ecommerce/controller/cronjob/CronjobController.java#L147) genutzt.
@ -0,0 +25,4 @@
try {
String jsonData = Files.readString(path, StandardCharsets.UTF_8);
ObjectMapper objectMapper = new ObjectMapper();
Supplier sup = (Supplier)objectMapper.readValue(jsonData, Supplier.class);
Collaborator

Der Cast ist redundant.

Der Cast ist redundant.
Seil0 reviewed 2020-05-27 18:00:41 +02:00
@ -0,0 +1,32 @@
HELP.md
.gradle
build/
!gradle/wrapper/gradle-wrapper.jar
Collaborator

Der wrapper gehört ins Repo.

Der wrapper gehört ins Repo.
Author
Collaborator

@CodeSteak
Das kommt aus der supplier Api, die du geschreiben hast. (Hab die hier dazu gemerged)

Die Datei gibts bei mir nicht und wird beim Ausführen auch nicht angelegt. Der Service läuft trotzdem. Was hat es mit dem Eintrag auf sich?

@CodeSteak Das kommt aus der supplier Api, die du geschreiben hast. (Hab die hier dazu gemerged) Die Datei gibts bei mir nicht und wird beim Ausführen auch nicht angelegt. Der Service läuft trotzdem. Was hat es mit dem Eintrag auf sich?
Collaborator

Das nicht "!" excluded das exclude. Ist das Default .gitignore vom .

Das nicht "!" excluded das exclude. Ist das Default .gitignore vom <Generator Name>.
Collaborator

*Default vom generator

*Default vom generator
Collaborator

Der Ordner gradle/wrapper fehlt aber im supplier Projekt.

Der Ordner `gradle/wrapper ` fehlt aber im supplier Projekt.
Collaborator

Fail, fixed.

Fail, fixed.
Seil0 approved these changes 2020-05-29 16:19:51 +02:00
Seil0 left a comment
Collaborator

lgtm

lgtm
Seil0 closed this pull request 2020-05-29 16:20:05 +02:00
Seil0 deleted branch feature/reorder_cronjob 2020-05-29 16:21:51 +02:00
This repo is archived. You cannot comment on pull requests.
No description provided.