[
http://mifosforge.jira.com/browse/MIFOS-3187?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Udai Gupta updated MIFOS-3187:
------------------------------
Attachment: m-pesa-plugin-savings-account-validation-fix.patch
M-Pesa-plugin_testing.tar.gz
During verification of bug I found that validatePayment is broken in 1.6.x and
master for this plugin (savings specifically). The patch for that is attached
in this issue. We just need to apply this patch and things should start working
as they were with 1.5.x
Current behavior
if ImportTransactionOrder=AL, NL, SA
and clients "transaction party details" contains only govt id then this plugin
will look for all (AL, NL, SA) and if anyone is not found then it doesn't allow
to do the transaction for that row
I think the expected behavior is
if ImportTransactionOrder=AL, NL, SA
and clients "transaction party details" contains only govt id then this plugin
will look for all (AL, NL, SA in same order) and if any loan account is not
found then do not report it(i.e. skip it) and make payments for others and
deposit in last of remaining amount.
Now, if I change to skip the loan account not found for client for
ImportTransactionOrder then it will be same behavior for "transaction party
detail". i.e. if
Transaction Party detail = Gid AL NL SA, and AL does not exist for that client
then skip reporting account not found error and make payments for others.
Is this the correct expected behavior?
> If first loan doesn't exist for client, MPESA plugin errors out instead of
> going on to second loan from ImportTransactionOrder
> ------------------------------------------------------------------------------------------------------------------------------
>
> Key: MIFOS-3187
> URL: http://mifosforge.jira.com/browse/MIFOS-3187
> Project: mifos
> Issue Type: Bug
> Components: ImportExport
> Affects Versions: Release 1.5.1
> Reporter: Kay Chau
> Assignee: mifosdeveloperqueue
> Fix For: Release E - Iteration 1, Release E
>
> Attachments: m-pesa-plugin-savings-account-validation-fix.patch,
> M-Pesa-plugin_testing.tar.gz
>
>
> Haven't verified this yet but for repro:
> # Have MPESA plugin set, have a client with a national ID and one loan
> account with short name NL1 and a savings account with short name SAV.
> # Set ImportTransactionOrder = AL1, NL1, and SAV.
> # Client's NL1 loan account should have a payment due next.
> # Import a file with payment due for NL1.
> Expected:
> Row correctly imports
> Actual:
> Errors out saying AL1 account is not found for the client
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://mifosforge.jira.com/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
------------------------------------------------------------------------------
This SF.net email is sponsored by Sprint
What will you do first with EVO, the first 4G phone?
Visit sprint.com/first -- http://p.sf.net/sfu/sprint-com-first
_______________________________________________
Mifos-issues mailing list
Mifos-issues@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mifos-issues
