#10 story/pfleger-modul-sperren-und-loschen-von-pflegern #37

Merged
SZUT-Dorian merged 11 commits from story/pfleger-modul-sperren-und-loschen-von-pflegern into main 2024-05-16 11:57:12 +00:00
Contributor
No description provided.
SZUT-Armin added 4 commits 2024-05-16 08:21:54 +00:00
#10: Add Base for Delete Date Functionality
All checks were successful
Quality Check / Linting Check (push) Successful in 12s
Quality Check / Javadoc Check (push) Successful in 21s
917aa8275f
#10: Deletedate calculated & DeletreBurtton enabled/disabled
Some checks failed
Quality Check / Linting Check (push) Failing after 12s
Quality Check / Javadoc Check (push) Successful in 21s
Quality Check / Linting Check (pull_request) Failing after 12s
Quality Check / Javadoc Check (pull_request) Successful in 20s
353146cbd7
SZUT-Armin changed title from story/pfleger-modul-sperren-und-loschen-von-pflegern to #10 story/pfleger-modul-sperren-und-loschen-von-pflegern 2024-05-16 08:22:13 +00:00
SZUT-Dominik requested review from SZUT-Dominik 2024-05-16 08:24:01 +00:00
SZUT-Dominik started working 2024-05-16 08:29:15 +00:00
SZUT-Dominik added 1 commit 2024-05-16 08:38:00 +00:00
#10: Cleanup
All checks were successful
Quality Check / Linting Check (push) Successful in 17s
Quality Check / Linting Check (pull_request) Successful in 21s
Quality Check / Javadoc Check (push) Successful in 34s
Quality Check / Javadoc Check (pull_request) Successful in 32s
d8de1e17fa
Signed-off-by: Dominik Säume <Dominik.Saeume@hmmh.de>
SZUT-Dominik requested changes 2024-05-16 08:41:03 +00:00
Dismissed
@ -91,0 +113,4 @@
}
}
private void loadActiveNursePage() {
Owner
  • Missing Javadoc
- [x] Missing Javadoc
SZUT-Dominik marked this conversation as resolved
@ -100,0 +128,4 @@
}
}
private void loadLockedNursePage() {
Owner
  • Missing Javadoc
- [x] Missing Javadoc
SZUT-Dominik marked this conversation as resolved
@ -0,0 +43,4 @@
private NurseDao dao;
private TreatmentDao treatmentDao;
public void initialize() {
Owner
  • Missing Javadoc
- [x] Missing Javadoc
Owner
  • Ist technisch falsch, technisch korrekt wäre:

    Initialization method that is called after the binding of all the fields.

    Weil Initalize & Instanciate nicht dasselbe sind.

    Initalize ist eine Methode, die im Hintergrund von JavaFx, nach dem Instaciating & Binding aufgerufen wird.

- [x] Ist technisch falsch, technisch korrekt wäre: > Initialization method that is called after the binding of all the fields. Weil **Initalize** & **Instanciate** nicht dasselbe sind. **Initalize** ist eine Methode, die im Hintergrund von **JavaFx**, nach dem **Instaciating** & **Binding** aufgerufen wird.
SZUT-Dominik marked this conversation as resolved
@ -0,0 +60,4 @@
}
private void readAllAndShowInTableView() {
Owner
  • Missing Javadoc
- [x] Missing Javadoc
SZUT-Dominik marked this conversation as resolved
@ -46,0 +68,4 @@
this.locked = new SimpleBooleanProperty(isLocked);
}
public LocalDate calculateDeleteDate() {
Owner
  • Missing Javadoc
- [x] Missing Javadoc
SZUT-Dominik marked this conversation as resolved
SZUT-Dominik stopped working 2024-05-16 08:42:34 +00:00
13 minutes 19 seconds
SZUT-Armin was assigned by SZUT-Dominik 2024-05-16 08:42:47 +00:00
SZUT-Dorian started working 2024-05-16 09:17:45 +00:00
SZUT-Dorian added 1 commit 2024-05-16 09:43:34 +00:00
#10 Javadoc addiert
All checks were successful
Quality Check / Linting Check (push) Successful in 16s
Quality Check / Linting Check (pull_request) Successful in 21s
Quality Check / Javadoc Check (push) Successful in 37s
Quality Check / Javadoc Check (pull_request) Successful in 34s
b401bdd91b
SZUT-Dorian stopped working 2024-05-16 09:44:51 +00:00
27 minutes 6 seconds
SZUT-Dorian added 1 commit 2024-05-16 09:58:56 +00:00
#10 Javadoc addiert
All checks were successful
Quality Check / Linting Check (push) Successful in 16s
Quality Check / Linting Check (pull_request) Successful in 21s
Quality Check / Javadoc Check (push) Successful in 34s
Quality Check / Javadoc Check (pull_request) Successful in 32s
d982242718
SZUT-Dominik requested changes 2024-05-16 10:17:32 +00:00
Dismissed
@ -1,7 +1,7 @@
<?xml version="1.0" encoding="UTF-8"?>
<project version="4">
<component name="DataSourceManagerImpl" format="xml" multifile-model="true">
<data-source source="LOCAL" name="Database" uuid="5a5b8be1-080b-4129-b89d-42f1ea832b90">
Owner
  • Diese Datei Rollbacken
- [ ] Diese Datei Rollbacken
Contributor

Die Contents vom 07.05. sind identisch zu dieser momentanen Änderung

Die Contents vom 07.05. sind identisch zu dieser momentanen Änderung
SZUT-Dorian marked this conversation as resolved
@ -6,3 +7,4 @@
import javafx.scene.control.SelectionModel;
import javafx.scene.control.Tab;
import javafx.scene.control.TabPane;
import javafx.scene.layout.AnchorPane;
Owner
  • Missing @author Tag
    fügt euch bitte, sobald ihr was anpasst in einer Klasse selber als Autoren Hinzu
- [x] Missing `@author` Tag fügt euch bitte, sobald ihr was anpasst in einer Klasse selber als Autoren Hinzu
SZUT-Dominik marked this conversation as resolved
@ -39,1 +49,4 @@
/**
* This method allows instantiating a {@link MainWindowController} object.
*/
Owner
  • Ist technisch falsch, technisch korrekt wäre:

    Initialization method that is called after the binding of all the fields.

    Weil Initalize & Instanciate nicht dasselbe sind.

    Initalize ist eine Methode, die im Hintergrund von JavaFx, nach dem Instaciating & Binding aufgerufen wird.

- [ ] Ist technisch falsch, technisch korrekt wäre: > Initialization method that is called after the binding of all the fields. Weil **Initalize** & **Instanciate** nicht dasselbe sind. **Initalize** ist eine Methode, die im Hintergrund von **JavaFx**, nach dem **Instaciating** & **Binding** aufgerufen wird.
SZUT-Dominik marked this conversation as resolved
@ -91,0 +118,4 @@
}
/**
* Loads the Active Nurse page into its tab.
Owner
  • Typo: active wird kleingeschrieben.
- [x] Typo: **active** wird kleingeschrieben.
SZUT-Dominik marked this conversation as resolved
@ -0,0 +20,4 @@
import java.util.List;
import java.util.Map;
public class LockedNurseController {
Owner
  • Missing Javadoc
    Hier würded ihr euch auch mit Author Tags hinzufügen:
    @author Armin Ribic
    wir haben überall die klar Namen geschrieben
- [ ] Missing Javadoc Hier würded ihr euch auch mit Author Tags hinzufügen: `@author Armin Ribic` wir haben überall die klar Namen geschrieben
SZUT-Dominik marked this conversation as resolved
@ -0,0 +88,4 @@
}
}
/**
Owner
  • Methoden mit @FXML benötigen keine Javadoc, weil sie durch ihr Linking bereits deklarativ genug sind
- [x] Methoden mit @FXML benötigen keine Javadoc, weil sie durch ihr Linking bereits deklarativ genug sind
SZUT-Dominik marked this conversation as resolved
@ -0,0 +111,4 @@
* On clicking a locked nurse, updates the delete button based on if they
* can be deleted.
*/
@FXML
Owner
  • Methoden mit @FXML benötigen keine Javadoc, weil sie durch ihr Linking bereits deklarativ genug sind
- [x] Methoden mit @FXML benötigen keine Javadoc, weil sie durch ihr Linking bereits deklarativ genug sind
SZUT-Dominik marked this conversation as resolved
@ -0,0 +122,4 @@
* Unlocks a locked nurse.
*/
@FXML
public void unlockNurse() {
Owner
  • Methoden mit @FXML benötigen keine Javadoc, weil sie durch ihr Linking bereits deklarativ genug sind
- [x] Methoden mit @FXML benötigen keine Javadoc, weil sie durch ihr Linking bereits deklarativ genug sind
SZUT-Dominik marked this conversation as resolved
@ -14,6 +22,7 @@ import java.util.StringJoiner;
public class Nurse extends Person {
Owner
  • Missing @author Tag
    fügt euch bitte, sobald ihr was anpasst in einer Klasse selber als Autoren Hinzu
- [x] Missing `@author` Tag fügt euch bitte, sobald ihr was anpasst in einer Klasse selber als Autoren Hinzu
SZUT-Dominik marked this conversation as resolved
@ -32,0 +44,4 @@
/**
* This constructor allows instantiating a {@link Nurse} object with
* specifying if the nurse is locked or not.
*/
Owner
  • zu ungenau, mein Vorschlag:
    /**
     * This constructor allows instantiating a {@link Nurse} object,
     * before it is stored in the database, by omitting the {@link Nurse#id ID} value.
     *It includes the locked Property.
     * @implSpec This was added for usage in the {@link de.hitec.nhplus.fixtures.NurseFixture NurseFixture}.
     */
    
- [ ] zu ungenau, mein Vorschlag: ```java /** * This constructor allows instantiating a {@link Nurse} object, * before it is stored in the database, by omitting the {@link Nurse#id ID} value. *It includes the locked Property. * @implSpec This was added for usage in the {@link de.hitec.nhplus.fixtures.NurseFixture NurseFixture}. */ ```
SZUT-Dominik marked this conversation as resolved
@ -46,0 +75,4 @@
/**
* Calculates the date when the nurse can be deleted.
*/
public LocalDate calculateDeleteDate() {
Owner
  • Nutzt Link Tags, auch wenn es selbst Referenzen sind, um die Generierte Javadoc besser lesbar zu machen.
    - nurse
    + {@link Nurse}`
    
- [x] Nutzt Link Tags, auch wenn es selbst Referenzen sind, um die Generierte Javadoc besser lesbar zu machen. ```diff - nurse + {@link Nurse}` ```
SZUT-Dominik marked this conversation as resolved
@ -68,0 +131,4 @@
}
/**
* Returns a string representation of the nurse with each field on a new line.
Owner
  • Die toString braucht keine Javadoc. Hatte Heidemann gefragt.
- [x] Die toString braucht keine Javadoc. Hatte Heidemann gefragt.
SZUT-Dominik marked this conversation as resolved
@ -68,3 +135,4 @@
*/
@Override
public String toString() {
return new StringJoiner(System.lineSeparator())
Owner
  • Fügt die locked property hier mit hinzu
- [x] Fügt die `locked` property hier mit hinzu
SZUT-Dominik marked this conversation as resolved
@ -20,1 +21,4 @@
/**
* This takes the same connection value from its parent class.
*/
public NurseDao(Connection connection) {
Owner
  • Wenn es nur einen Grundkonstruktor gibt, welcher nichts Besonderes macht, braucht der laut heidemann auch keine Javadoc.
- [ ] Wenn es nur einen Grundkonstruktor gibt, welcher nichts Besonderes macht, braucht der laut heidemann auch keine Javadoc.
SZUT-Dominik marked this conversation as resolved
@ -23,1 +27,4 @@
/**
* Insert values into nurse SQL table.
*/
Owner
  • Javadoc ist bereits im DaoImp<T> und wird vererbt, bitte hier weglassen
- [x] Javadoc ist bereits im `DaoImp<T>` und wird vererbt, bitte hier weglassen
SZUT-Dominik marked this conversation as resolved
@ -38,1 +46,4 @@
/**
* Show data of nurse with given id.
*/
@Override
Owner
  • Javadoc ist bereits im DaoImp<T> und wird vererbt, bitte hier weglassen
- [ ] Javadoc ist bereits im `DaoImp<T>` und wird vererbt, bitte hier weglassen
SZUT-Dominik marked this conversation as resolved
@ -45,1 +56,4 @@
/**
* Creates a Nurse instance from the given ResultSet.
*/
Owner
  • Javadoc ist bereits im DaoImp<T> und wird vererbt, bitte hier weglassen
- [ ] Javadoc ist bereits im `DaoImp<T>` und wird vererbt, bitte hier weglassen
SZUT-Dominik marked this conversation as resolved
@ -53,3 +68,4 @@
);
}
/**
Owner
  • Javadoc ist bereits im DaoImp<T> und wird vererbt, bitte hier weglassen
- [x] Javadoc ist bereits im `DaoImp<T>` und wird vererbt, bitte hier weglassen
SZUT-Dominik marked this conversation as resolved
@ -60,2 +78,4 @@
}
/**
* Extracts a list of Nurse instances from the given ResultSet.
Owner
  • Javadoc ist bereits im DaoImp<T> und wird vererbt, bitte hier weglassen
- [ ] Javadoc ist bereits im `DaoImp<T>` und wird vererbt, bitte hier weglassen
SZUT-Dominik marked this conversation as resolved
@ -72,0 +99,4 @@
/**
* Retrieves a list of locked Nurse instances from the database.
*/
Owner
  • Nutz Refernzen & etwas spezifischer:
     /**
      * Read all database entries of locked {@link Nurse}s into a {@link List} of model instances.
      */
    
- [x] Nutz Refernzen & etwas spezifischer: ```java /** * Read all database entries of locked {@link Nurse}s into a {@link List} of model instances. */ ```
SZUT-Dominik marked this conversation as resolved
@ -72,0 +108,4 @@
/**
* Prepares a SQL statement to update a Nurse instance in the database.
*
*/
Owner
  • Javadoc ist bereits im DaoImp<T> und wird vererbt, bitte hier weglassen
- [ ] Javadoc ist bereits im `DaoImp<T>` und wird vererbt, bitte hier weglassen
SZUT-Dominik marked this conversation as resolved
@ -86,2 +129,4 @@
}
/**
* Prepares a SQL statement to delete a Nurse instance from the database by its ID.
Owner
  • Javadoc ist bereits im DaoImp<T> und wird vererbt, bitte hier weglassen
- [ ] Javadoc ist bereits im `DaoImp<T>` und wird vererbt, bitte hier weglassen
SZUT-Dominik marked this conversation as resolved
SZUT-Dorian added 1 commit 2024-05-16 11:15:03 +00:00
#10 Javadoc Änderungen ergänzt
Some checks failed
Quality Check / Linting Check (push) Successful in 16s
Quality Check / Linting Check (pull_request) Successful in 20s
Quality Check / Javadoc Check (push) Failing after 22s
Quality Check / Javadoc Check (pull_request) Failing after 20s
505d9e3f5c
SZUT-Dorian added 1 commit 2024-05-16 11:17:14 +00:00
#10 toString korrigiert
All checks were successful
Quality Check / Linting Check (push) Successful in 15s
Quality Check / Linting Check (pull_request) Successful in 21s
Quality Check / Javadoc Check (push) Successful in 36s
Quality Check / Javadoc Check (pull_request) Successful in 34s
814f0a367b
SZUT-Dorian added 1 commit 2024-05-16 11:24:15 +00:00
#10 dataSources.xml ergänzt
All checks were successful
Quality Check / Linting Check (push) Successful in 15s
Quality Check / Linting Check (pull_request) Successful in 21s
Quality Check / Javadoc Check (push) Successful in 35s
Quality Check / Javadoc Check (pull_request) Successful in 32s
d9e0aa2049
SZUT-Dominik requested changes 2024-05-16 11:36:33 +00:00
Dismissed
@ -91,0 +120,4 @@
}
/**
* Loads the active Nurse page into its tab.
Owner
  • Nurse wird klein geschrieben, oder eine Refernz
- [x] Nurse wird klein geschrieben, oder eine Refernz
SZUT-Dominik marked this conversation as resolved
@ -0,0 +23,4 @@
/**
* @author Armin Ribic
* @author Dorian Nemec
*/
Owner
  • Author alleine reicht Heidemann hier nicht, kurze Beschreibung wie bei den anderen Controllern hier hin, muss nix Langes sein.
- [x] Author alleine reicht Heidemann hier nicht, kurze Beschreibung wie bei den anderen Controllern hier hin, muss nix Langes sein.
SZUT-Dominik marked this conversation as resolved
@ -0,0 +69,4 @@
}
/**
* Reads all locked nurse data and shows it in the table.
Owner
  • Refernz für Nurse hier bitte rein {@link Nurse}
- [x] Refernz für Nurse hier bitte rein {@link Nurse}
SZUT-Dominik marked this conversation as resolved
@ -70,2 +72,3 @@
@Override
/**
* Reads all active nurses.
Owner
  • Read all database entries of active {@link Nurse}s into a {@link List} of model instances.
- [x] `Read all database entries of active {@link Nurse}s into a {@link List} of model instances.`
SZUT-Dominik marked this conversation as resolved
SZUT-Dorian added 1 commit 2024-05-16 11:54:37 +00:00
#10 JavaDoc ergänzt #2
All checks were successful
Quality Check / Linting Check (push) Successful in 16s
Quality Check / Linting Check (pull_request) Successful in 22s
Quality Check / Javadoc Check (push) Successful in 35s
Quality Check / Javadoc Check (pull_request) Successful in 33s
e583d825ec
SZUT-Dominik approved these changes 2024-05-16 11:56:37 +00:00
SZUT-Dorian merged commit 6908e8129b into main 2024-05-16 11:57:12 +00:00
SZUT-Dorian deleted branch story/pfleger-modul-sperren-und-loschen-von-pflegern 2024-05-16 11:57:12 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
3 participants
Notifications
Total time spent: 40 minutes 25 seconds
SZUT-Dorian
27 minutes 6 seconds
SZUT-Dominik
13 minutes 19 seconds
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: SZUT/nhplus#37
No description provided.