[26544] dialog on mail change of activated mediorder patient #799
Conversation
|
| } | ||
|
|
||
| @Override | ||
| public List<ISticker> getStickers(Identifiable identifiable, boolean includeMediorderSticker) { |
There was a problem hiding this comment.
Nein, sowas macht in einem zentralen Service keinen Sinn. Der zentrale Service sollte keine Kenntnis eines mediorder stickers haben bzw. wenn, dann brauchst du wirklich eine gute Begründung dafür. Du müsstest das generischer halten, in dem man bspw. einen Filter übergeben kann. Schon der originalcode if (sticker != null && !sticker.getId().equals("activate_mediorder")) { muss raus - weiß gar nich wie der reinkommen konnte!
Der Sticker-Service soll keinen einzelnen Sticker kennen und sonder-behandeln. Der soll nur Methoden bereitstellen Sticker generell zu bearbeiten
| @@ -161,12 +172,27 @@ public String getText(Object element) { | |||
|
|
|||
| setUnlocked(LocalLockServiceHolder.get().isLocked(pat)); | |||
|
|
|||
| emailVerifyListener = e -> { | |||
There was a problem hiding this comment.
Du verwendest den IStickerService ja nur an dieser Stelle! Hol ihn doch hier ab und release ihn gleich nach der Verwendung wieder. dispose sollte generell nicht verwendet werden, und es gibt auch keinen Grund das hier zu tun! Zudem musst die ungetService auf das service-objekt, nicht die klassen anwenden!
Du musst IStickerService gar nicht im Field anlegen!
|
|
||
| public class StickerConstants { | ||
|
|
||
| public static final String PEA_MEDIORDER_STICKER_ID = "activate_mediorder"; |
| * Get all stickers for the {@link Identifiable}. The returned list is sorted by | ||
| * {@link ISticker#getImportance()}. | ||
| * Get all stickers for the {@link Identifiable} excluding pea mediorder | ||
| * sticker. The returned list is sorted by {@link ISticker#getImportance()}. |
There was a problem hiding this comment.
? Die JavaDoc stimmt doch so nicht mehr oder?
Ist der neue pr für #695