Re: [WebERP-developers] Bug in GLAccountInquiry

[Pak R. <pak...@gm...>]

Hi Exson:

Thanks for your reply. I already commited the correction of line 214 of
PCAuthorizeExpenses.php

Anyway, I'm afraid this bug has more consequences than the detected mess in
your mod, as in line 202 we were getting the next transno for type = 2, but
we were saving it into DB as type = 1.

So, some rows of banktrans table will have type = 1 and the same transno
but they have nothing in common, so info extracted from banktrans can't be
trusted 100% if Petty Cash Module has been used

Not sure if we can get a way to "un-mess" the data, without getting more
orphaned info

Update manually systype so transno for type = 2 is higher than type 1 or 2.
SELECT all rows in banktrans with texts "Petty cash" AND "ASSIGNCASH"
For every row
    get a new transno for type = 2
    update the row with type = 2 and new transno
endfor

I'm sorry for this mess. It affects all Assign cash transactions recorded
since the beggining of the Petty Cash Module, in 2.010.

Regards,
Ricard

2016-01-13 10:23 GMT+08:00 ExsonQu <hex...@gm...>:

> *Hi, Richard,*
>
>              Thank you for the reply.
>
>             Yes, I think it should be type 2 instead of type 1. Otherwise,
> we can not retrieve a correct data. And the feature is only constraint to
> bank accounts now.
>             I've fixed the variables non refresh bug and commit to trunk.
> The your previous data still can not show correctly due to the transaction
> type mismatch -:).
>
>           Thanks and best regards!
>
>          Exson
>
>
>
> --
> View this message in context:
> http://weberp-accounting.1478800.n4.nabble.com/Bug-in-GLAccountInquiry-tp4658408p4658465.html
> Sent from the web-ERP-developers mailing list archive at Nabble.com.
>
>
> ------------------------------------------------------------------------------
> Site24x7 APM Insight: Get Deep Visibility into Application Performance
> APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
> Monitor end-to-end web transactions and take corrective actions now
> Troubleshoot faster and improve end-user experience. Signup Now!
> http://pubads.g.doubleclick.net/gampad/clk?id=267308311&iu=/4140
> _______________________________________________
> Web-erp-developers mailing list
> Web...@li...
> https://lists.sourceforge.net/lists/listinfo/web-erp-developers
>