Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[26782] mediorder - medication order history #757

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

PatrickDenzler
Copy link
Collaborator

No description provided.

@@ -35,6 +36,12 @@ public IOrderEntry findOpenOrderEntryForStockEntry(IStockEntry stockEntry) {
return null;
}

public List<IOrderEntry> findOrderEntryForStock(IStock stock) {
IQuery<IOrderEntry> query = CoreModelServiceHolder.get().getQuery(IOrderEntry.class);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nicht über den Holder machen, OrderService ist selbst eine Komponente, also über

@Reference(target = "(" + IModelService.SERVICEMODELNAME + "=ch.elexis.core.model)")
private IModelService modelService;

den Service im Field reinholen! Damit ist sichergestellt, dass der OrderService erst verfügbar gemacht wird,
nachdem der ModelService instanziert wurde.

@@ -35,6 +36,12 @@ public IOrderEntry findOpenOrderEntryForStockEntry(IStockEntry stockEntry) {
return null;
}

public List<IOrderEntry> findOrderEntryForStock(IStock stock) {
IQuery<IOrderEntry> query = CoreModelServiceHolder.get().getQuery(IOrderEntry.class);
query.and("stockid", COMPARATOR.EQUALS, stock.getId());
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Statt dem String "stockid" solltest du hier ModelPackage.Literals.ISTOCK_ENTRY__STOCK verwenden, dann ist das Modellgebunden, bitte teste das mal ob das so funktioiert.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ich habe ModelPackage.Literals.ISTOCK_ENTRY__STOCK & ModelPackage.Literals.IORDER_ENTRY__STOCK versucht. Beide geben mir die Fehlermeldung aus: Could not resolve attribute...

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ach meine Güte, du suchst ja in der Klasse IOrderEntry, entsprechend musst du auch die Literale aus der Klasse ModelPackage.Literals.IORDER_ENTRY__STOCK verwenden. Bitte probier das noch, eigentlich sollte der Literal sich dann auflösen.

@huthomas oder ist stockid nicht über das Model erreichbar? Ich bin da grad puzzled!

@@ -290,7 +322,7 @@ public String getText(Object element) {
}
});
TableColumn tblclmntvcPatientNumber = tvcPatientNumber.getColumn();
tcLayout.setColumnData(tblclmntvcPatientNumber, new ColumnPixelData(70, true, true));
tcLayout.setColumnData(tblclmntvcPatientNumber, new ColumnPixelData(100, true, true));
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hier könntest du generell mal die ColumnWeightData anschaun, dann würde das
auch immer die ganze Tabelle in der Breite ausfüllen.

@@ -684,6 +804,49 @@ public int compare(Viewer viewer, Object o1, Object o2) {
}
}

public class MedicationHistoryComparator extends ViewerComparator {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mach Klassen nicht so groß, das Ding sollte in eine eigene Klasse.

Copy link
Member

@col-panic col-panic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sieht bis auf den Literals Punkt schon gut aus, hier bin ich mir selbst nicht sicher. Hab im matrix geschrieben, dass wir den Service als separaten PR mit Test übernehmen sollten, bitte noch entsprechend umsetzen!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants