#10 story/pfleger-modul-sperren-und-loschen-von-pflegern #37
3 changed files with 6 additions and 5 deletions
|
@ -88,7 +88,7 @@ public class MainWindowController {
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Loads the treatment page into its tab.
|
* Loads the {@link } page into its tab.
|
||||||
*/
|
*/
|
||||||
private void loadTreatmentsPage() {
|
private void loadTreatmentsPage() {
|
||||||
try {
|
try {
|
||||||
|
@ -120,7 +120,7 @@ public class MainWindowController {
|
||||||
}
|
}
|
||||||
|
|
||||||
SZUT-Dominik marked this conversation as resolved
Outdated
SZUT-Dominik
commented
Outdated
Review
- [x] Typo: **active** wird kleingeschrieben.
|
|||||||
/**
|
/**
|
||||||
* Loads the active Nurse page into its tab.
|
* Loads the active {@link Nurse} page into its tab.
|
||||||
SZUT-Dominik marked this conversation as resolved
Outdated
SZUT-Dominik
commented
- [x] Nurse wird klein geschrieben, oder eine Refernz
|
|||||||
*/
|
*/
|
||||||
private void loadActiveNursePage() {
|
private void loadActiveNursePage() {
|
||||||
try {
|
try {
|
||||||
|
@ -138,7 +138,7 @@ public class MainWindowController {
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Loads the locked nurse page into its tab.
|
* Loads the locked {@link Nurse} page into its tab.
|
||||||
*/
|
*/
|
||||||
private void loadLockedNursePage() {
|
private void loadLockedNursePage() {
|
||||||
try {
|
try {
|
||||||
|
|
|
@ -21,6 +21,7 @@ import java.util.List;
|
||||||
import java.util.Map;
|
import java.util.Map;
|
||||||
|
|
||||||
/**
|
/**
|
||||||
SZUT-Dominik marked this conversation as resolved
Outdated
SZUT-Dominik
commented
- [ ] Missing Javadoc
Hier würded ihr euch auch mit Author Tags hinzufügen:
`@author Armin Ribic`
wir haben überall die klar Namen geschrieben
|
|||||||
|
* Controller for all locked nurses.
|
||||||
* @author Armin Ribic
|
* @author Armin Ribic
|
||||||
* @author Dorian Nemec
|
* @author Dorian Nemec
|
||||||
SZUT-Dominik marked this conversation as resolved
SZUT-Dominik
commented
- [x] Author alleine reicht Heidemann hier nicht, kurze Beschreibung wie bei den anderen Controllern hier hin, muss nix Langes sein.
|
|||||||
*/
|
*/
|
||||||
|
@ -69,7 +70,7 @@ public class LockedNurseController {
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
SZUT-Dominik marked this conversation as resolved
SZUT-Dominik
commented
- [x] Refernz für Nurse hier bitte rein {@link Nurse}
|
|||||||
* Reads all locked nurse data and shows it in the table.
|
* Reads all locked {@link Nurse} data and shows it in the table.
|
||||||
*/
|
*/
|
||||||
private void readAllAndShowInTableView() {
|
private void readAllAndShowInTableView() {
|
||||||
this.nurses.clear();
|
this.nurses.clear();
|
||||||
|
|
|
@ -71,7 +71,7 @@ public class NurseDao extends DaoImp<Nurse> {
|
||||||
}
|
}
|
||||||
SZUT-Dominik marked this conversation as resolved
Outdated
SZUT-Dominik
commented
- [x] Javadoc ist bereits im `DaoImp<T>` und wird vererbt, bitte hier weglassen
|
|||||||
|
|
||||||
/**
|
/**
|
||||||
* Reads all active nurses.
|
* Read all database entries of active {@link Nurse}s into a {@link List} of model instances.
|
||||||
SZUT-Dominik marked this conversation as resolved
Outdated
SZUT-Dominik
commented
- [x] `Read all database entries of active {@link Nurse}s into a {@link List} of model instances.`
|
|||||||
*/
|
*/
|
||||||
public List<Nurse> readAllActive() throws SQLException {
|
public List<Nurse> readAllActive() throws SQLException {
|
||||||
final String SQL = "SELECT * FROM nurse WHERE isLocked=false";
|
final String SQL = "SELECT * FROM nurse WHERE isLocked=false";
|
||||||
|
|
Loading…
Reference in a new issue