Re: [WebERP-developers] (no subject)

[iced l. <ice...@gm...>]

Hi Phill and Ricard,

I posted a possible fix in the forum post. Didn't commit as I know you were
working on it Phil.

Revision 6350 Line 605

Was changed from:
'" . -$PaymentItem->Amount . " ',

to

'" . -$_SESSION['PaymentDetail' . $identifier]->Amount . "',


If we revert this at least banktrans entries are correct for type 1
payment.


On Mon, Feb 3, 2014 at 1:47 PM, Phil Daintree <ph...@lo...> wrote:

>   Ah good point ... I am not sure anyone has figured this out ... I will
> dive in.
>
> On February 3, 2014 at 1:35 PM Pak Ricard <pak...@gm...> wrote:
>  Hi Phil:
>
>  I see you are preparing a new release. great!
>
>  I'm not sure if the bug reported on 10th december on this email and the
> forum http://www.weberp.org/forum/showthread.php?tid=1994 has been
> fixed.
>
>  This script is too complex for me to debug, sorry.
>
>  Regards,
> Ricard
>
>
>  2013-12-10 Pak Ricard <pak...@gm...>:
>
>  Hi Tim:
>  Yes, I'm doing 3 transctions in one payment.
>  I think it is not important (for us at least) if in banktrans there
> should be 3 lines or just 1, but if there's 3 lines, each line should have
> its own value, not the sum. On the other side if it should be just one line
> in banktrans with the total sum of the payment, not show it N times.
>  The problem as it is now is that's a mix of the 2 solutions.
>
>  Solution A is good, solution 2 is good, but the mix is wrong.
>  Then, as it is now I guess it's probably easier just to keep the 3 lines
> but correct the value of each line
>
>  Regards,
> Ricard
>
>
>  2013/12/9 Tim Schofield <tim...@gm...>
>
> Hi Ricard, are you doing three transactions as one payment?
>
> We have never really got straight if you do 3 lines of one payment
> whether that should show s 3 lines in banktrans or just 1. The code
> has alternated over the years. I think Phil's code was changed to
> three transactions fairly recently and that must be when the error
> crept in. Had a quick look, but not sure which commit it was.
>
> Tim
>
> On 9 December 2013 02:30, Pak Ricard < pak...@gm...> wrote:
> > Hi all:
> >
> > Using latest Payments.php in SVN with a bank account in non-functional
> > currency I found it does not record the correct amounts in banktrans
> table.
> >
> > Example:
> >
> > Functional currency IDR
> > Bank account in USD: Initial balance 10.000 USD
> > Go to Payments.php and do the following payments from this USD bank
> account
> > (currency of payment USD as well)
> > Payment 1 = 100 USD
> > Payment 2 = 200 USD
> > Payment 3 = 300 USD
> >
> >
> > We should expect 3 new rows in banktrans with -100, -200 and -300 USD
> but we
> > get 3 rows with - 600 USD .
> >
> > Also the bank reconciliation, gets wrong info as well, initial balance
> gets
> > wrong, and quantities to reconcile get wrong as well.
> >
> > Not sure when this problem showed up. It's a quite tricky script, so I
> would
> > prefer if one of the webERP masters could fix it.
> >
> > I also posted this bug on the forum, as I could attach 4 screenshots
> easily.
> >
> > Regards,
> > Ricard
> >
>  >
> ------------------------------------------------------------------------------
>
> > Sponsored by Intel(R) XDK
> > Develop, test and display web and hybrid apps with a single code base.
> > Download it for free now!
> >
> http://pubads.g.doubleclick.net/gampad/clk?id=111408631&iu=/4140/ostg.clktrk
> > _______________________________________________
> > Web-erp-developers mailing list
> > Web...@li...
> > https://lists.sourceforge.net/lists/listinfo/web-erp-developers
> >
>
>
>
> --
> Course View Towers,
> Plot 21 Yusuf Lule Road,
> Kampala
> T   +256 (0) 312 314 418 <+256%20%280%29%20312%20314%20418>
> M +256 (0) 752 963 325 <+256%20%280%29%20752%20963%20325>
> www.weberpafrica.com
> Twitter: @TimSchofield2
> Blog: http://weberpafrica.blogspot.co.uk/
>
>
>
>  Phil
>
> Ph: +64 (0)275 567890
> Skype: daintree
> http://www.logicworks.co.nz
>
>
> ------------------------------------------------------------------------------
> Managing the Performance of Cloud-Based Applications
> Take advantage of what the Cloud has to offer - Avoid Common Pitfalls.
> Read the Whitepaper.
>
> http://pubads.g.doubleclick.net/gampad/clk?id=121051231&iu=/4140/ostg.clktrk
> _______________________________________________
> Web-erp-developers mailing list
> Web...@li...
> https://lists.sourceforge.net/lists/listinfo/web-erp-developers
>
>