Opened 8 years ago
Closed 8 years ago
#912 closed defect (fixed)
Fix report delivery system
Reported by: | Nicklas Nordborg | Owned by: | Nicklas Nordborg |
---|---|---|---|
Priority: | critical | Milestone: | Reggie v4.7.1 |
Component: | net.sf.basedb.reggie | Keywords: | |
Cc: |
Description (last modified by )
This seems to not work properly. The only use case that seems to get properly registered and shared to the correct site is when all of the following are fulfilled:
- Only a single specimen tube is registered
- The YellowLabel annotation is NOT SET in the "Specimen tube registration" wizard
- The YellowLabel annotation is SET in the "Referral form registration" wizard.
More investigations are needed since there may be other unknown problems (there seems to be a properly shared case that is not showing up in the delivery client).
It would also be nice if the final "Publish for delivery" functionality could check (and fix) everything. This will currently only make sure that the PDF file is correctly shared which doesn't help in case the other information is incorrect.
Change History (8)
comment:1 by , 8 years ago
Description: | modified (diff) |
---|
comment:2 by , 8 years ago
Milestone: | Reggie v4.8 → Reggie v4.7.1 |
---|
comment:3 by , 8 years ago
comment:4 by , 8 years ago
The reason that specimen are not showing up in the "senaste 30 dagarna" is that the calculation for the start date uses the calculates the number of milliseconds in 30 days using 30*24*3600*1000 which yields the result: -1702967296!!
This is due to an Java using an integer data type instead of long which causes a silent overflow to a negative number. Switching to a long data type seems to fix the problem.
comment:5 by , 8 years ago
comment:6 by , 8 years ago
(In [4135]) References #912: Fix report delivery system
Re-designed they way items are published to sites. Most logic has been moved to the SetPermissionsForDeliverPlugin
class. The calling code only need to provide a list with items and the plug-in will check which permissions they should or should not have.
This change fixes the problems with the "Pathology referral form registration" wizard. Publishing the PDF reports do not work at all yet.
comment:7 by , 8 years ago
(In [4136]) References #912: Fix report delivery system
Changes the "Publish for delivery" function to only send the PDF files to the SetPermissionsForDeliveryPlugin
. The plug-in will make sure that the file and parent directories are shared to the correct site and to the patient curator group.
Using the "Mark as completed" function in the "Yellow label tracking" wizard will now also check and fix the site sharing if needed.
comment:8 by , 8 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
I have made some investigations when it comes to registering new specimen tubes and patients.