OCA / vertical-medical

Open Source Healthcare System for Odoo
GNU Affero General Public License v3.0
263 stars 277 forks source link

medical_medication Discussion #42

Closed lasley closed 9 years ago

lasley commented 9 years ago

Not really sure of a better way to start this discussion, so please let me know if there is one. Wiki seemed a bit hard to manage in terms of understanding opinions. This is a blocker in our current project, so I'll be tackling it next.

I noticed @JonathanNEMRY's change notes in the wiki for the EMR module, and I fully agree about the modularity. Maybe we should start by breaking the two functional improvement suggestions into their own module (medications, templates, dosages, medicaments + categories => medical_medication or similar)

I'm going to copy/paste the wiki entry then provide my thoughts as a comment just to keep the two separate.


Medication And Medication Template

Major issue is that the start-end date is defined on medication and not on medication template. So this forces to create as much template as patient

Data model should be review with inclusion of an abstract concept. (to discuss)

Medicament:

Current implemented version introduce a ambigous relation between 'medical.medicament' and 'product.product' which may possibly beeing a source of additional complexity into the invoicing process of Odoo as product is the core of its billing process This notion has to be reviewed with two possible solution

lasley commented 9 years ago

My thoughts

Medication And Medication Template

Definitely agree that the start and end should not be in the medication & templates. Shouldn't we maybe just pull that out altogether and let the medication.order dictate that (below)?

In order to cover over-the-counter items, I propose that we create medication.order - then inherit prescription.order from that to cover all the doctor related stuff. Dates and times are related to an order, not the product or template, so should be treated as such IMHO

I also think that prescription.order and associated should just be included in the pharmacy module we're creating, but that's another discussion :wink:

Medicament:

I'm wondering if we need another dimension here. @JonathanNEMRY brought up a good point with the tables getting large, which is only going to compound with the pharmacy module that we are in the process of creating.

The issue is that there are multiple medicament dispense types, dosages, routes, etc. for the same "item". We can already see this becoming an issue when the medical_emr_data module starts coming into play (see morphine - All of the different methods of ingestion, UOMs, etc. have all been grouped into the notes section).

I propose something like this:

JonathanNEMRY commented 9 years ago

Hi @lasley ! I have pushed some Pull Request about Disease/Medicament/Medication/Prescription Order. Don't hesitate to review/comment them. I had no other choice but to make them interrelated because of the module dependencies. I'have also updated the wiki graph.

Jonathan

lasley commented 9 years ago

Hi @JonathanNEMRY

Awesome, thanks for the update! I'll take a look now.

Agreed that there was not a way to create an interdependency between the modules. We're going to run into a bit of that with the EMR stuff, I'm sure that's why it was all originally just dropped into one module.

lasley commented 9 years ago

I believe we're covered this conversation, closing