bugGNU Health - Bugs: bug #63842, Mismatch between required vs....

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #63842: Mismatch between required vs. readonly fields in Imaging Results

Submitter:  Mathias Behrle <yangoon>
Submitted:  Mon 27 Feb 2023 10:26:14 AM UTC
   
 
Category:  Functionality Severity:  1 - Wish
Item Group:  None Status:  Confirmed
Privacy:  Public Assigned to:  meanmicio
Open/Closed:  Open Release:  4.3.0
Module:  health_imaging Component:  Hospital Management

Thu 06 Apr 2023 09:47:47 AM UTC, comment #1: 

Hi, Mathias!

Thanks for the feedback.

Yes, the medical imaging results currently depends and takes most info from the medical imaging order workflow (Medical Imaging Order req --> Requests --> Results )

While in most cases this should be the best way to approach, I agree sometimes we would need to manually enter the results, such in the case of importing the history of the patient, and not depend on the request/order.

We can create a task for upcoming GH HMIS server release.



Luis Falcon <meanmicio>
Group administrator
Mon 27 Feb 2023 10:26:14 AM UTC, original submission:  

It is possible to create an imaging result without a patient (and a patient can not be filled in because the field is read-only), which typically results in

Traceback (most recent call last):                                                                                                                                                                                                          
  File "/home/mathiasb/projects/gnuhealth4/trytond/trytond/protocols/dispatcher.py", line 182, in _dispatch                                                                                                                                 
    result = rpc.result(meth(c_args, *c_kwargs))                                                                                                                                                                                          
                        ^^^^^^^^^^^^^^^^^^^^^^^^^                                                                                                                                                                                           
  File "/home/mathiasb/projects/gnuhealth4/trytond/trytond/model/modelsql.py", line 870, in read                                                                                                                                            
    getter_results = field.get(                                                                                                                                                                                                             
                     ^^^^^^^^^^                                                                                                                                                                                                             
  File "/home/mathiasb/projects/gnuhealth4/trytond/trytond/model/fields/function.py", line 105, in get                                                                                                                                      
    return dict((name, call(name)) for name in names)                                                                                                                                                                                       
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^                                                                                                                                                                                       
  File "/home/mathiasb/projects/gnuhealth4/trytond/trytond/model/fields/function.py", line 105, in <genexpr>                                                                                                                                
    return dict((name, call(name)) for name in names)                                                                                                                                                                                       
                       ^^^^^^^^^^                                                                                                                                                                                                           
  File "/home/mathiasb/projects/gnuhealth4/trytond/trytond/model/fields/function.py", line 100, in call                                                                                                                                     
    return dict((r.id, method(r, name)) for r in records)                                                                                                                                                                                   
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^                                                                                                                                                                                   
  File "/home/mathiasb/projects/gnuhealth4/trytond/trytond/model/fields/function.py", line 100, in <genexpr>                                                                                                                                
    return dict((r.id, method(r, name)) for r in records)                                                                                                                                                                                   
                       ^^^^^^^^^^^^^^^                                                                                                                                                                                                      
  File "/home/mathiasb/projects/gnuhealth4/trytond/trytond/modules/health_imaging/health_imaging.py", line 170, in patient_age_at_evaluation 
    if (self.patient.name.dob and self.date): 

So generally 
- def patient_age_at_evaluation should handle this case gracefully with
        if (self.patient and self.patient.name.dob and self.date):
- Input of a patient and request_date should be possible to be able to enter manual Imaging Results otherwise the form is too workflow dependent (this would be my preference)
- If OTOH the whole form should only be readable for workflow results then the whole form should be read-only (currently e.g. Study and Evaluated By are editable)


Mathias Behrle <yangoon>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by meanmicio (Posted a comment)
  • -email is unavailable- added by yangoon (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    Follow 6 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2023-04-06 meanmicio CategoryProgramming / Tracebacks Functionality
        Severity3 - Normal 1 - Wish
        StatusNone Confirmed
        Assigned toNone meanmicio
        ReleaseNone 4.3.0
        ComponentNone Hospital Management

    Back to the top

    Powered by Savane 3.12.
    Corresponding source code