-
Notifications
You must be signed in to change notification settings - Fork 10
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
Fixed odoo.dob.field
and odoo.weight.field
properties and 'patientVoided' → 'voided'.
#23
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -23,7 +23,7 @@ | |
|
||
<choice> | ||
<when> | ||
<simple>${exchangeProperty.patient-odoo-id} == null && ${exchangeProperty.patient.get('patientVoided')} == true</simple> | ||
<simple>${exchangeProperty.patient-odoo-id} == null && ${exchangeProperty.patient.get('voided')} == true</simple> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think you might want to check both There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @wluyima it seems like we have There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Sorry! That's what I meant to check both Person.personVoided and Patient.voided There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @VaishSiddharth @wluyima why doing test on both properties personVoided and Patient.voided as in this route we deal with patient data only ( and with patient identifier,...) and only patient are synchronized with Odoo. IMHO, The use case "a patient is voided but the person is not voided" should be ignored as well as this route need patients only. If a paitent is voided, we will have an error when trying to get the patient identifier and it causes erros in this route when a patient is voided. Why should the use case "a patient is voided but the person is not voided" be processed in this route ? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. There was definitely a bug that this PR seems to address. And in fact, these changes aligns with what am saying so I guess it's all good. |
||
<log message="No action to take for voided patient since they have no customer record in odoo" /> | ||
</when> | ||
<otherwise> | ||
|
@@ -36,7 +36,7 @@ | |
<simple>${exchangeProperty.updateExistingCustomer} == true || ${exchangeProperty.createCustomerIfNotExist} == true || ${properties:create.customer.if.not.exist:false} == true</simple> | ||
<setProperty name="isPatientVoidedOrDeleted"> | ||
<!-- TODO support deleted patient --> | ||
<spel>#{getProperty('patient').get('patientVoided') == true}</spel> | ||
<spel>#{getProperty('patient').get('voided') == true}</spel> | ||
</setProperty> | ||
|
||
<when> | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this meant to address a bug?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, everything I have done on this PR is to address bugs that I have encounter.