Thread Rating:
  • 0 Vote(s) - 0 Average
  • 1
  • 2
  • 3
  • 4
  • 5
ImportBankTrans Error
01-27-2018, 01:45 AM,
#1
ImportBankTrans Error
includes/DefineImportBankTrans.php

Line 63 has one too many opening parenthesis:
self::__construct(($ValueDate, $Amount);

Should be this:
self::__construct($ValueDate, $Amount);
https://www.linkedin.com/in/eclipsepaulbecker
Reply
01-27-2018, 05:31 AM,
#2
RE: ImportBankTrans Error
Hmmm, that is already fixed in SVN. In fact, I applied that change in April last year! Wink

Not sure how you might have it the old way?

Oh well, good to know that it's already covered.
Reply
01-27-2018, 05:46 AM, (This post was last modified: 01-27-2018, 05:47 AM by TurboPT.)
#3
RE: ImportBankTrans Error
Tim, on a side note, at the top of the bank trans class file, the SVN version/timestamp line is present, but has not updated since 2009??

I've noted too, that in several other files that line is not updating. Is there something special with the format to make it work?

I search around sourceforge a bit, but I could not find a reference about that line, or my search terms were not finding the information.
Reply
01-27-2018, 06:03 AM,
#4
RE: ImportBankTrans Error
Now then, that is stretching my memory a bit. Big Grin
It has to do with keyword substitution (http://svnbook.red-bean.com/en/1.7/svn.a...words.html), Nicholas Lee set this up when we moved to Subversion from CVS many years ago. As I remember the string

$Id$

needs to be inserted at the beginning of the file and then subversion will substitute Id with the details of that revision. Looking at it that needs to be within a comment so

/* $Id$ */

but that's about as far as I remember. These days I only use Git as it does so much more and so much better so my subversion knowledge is sketchy at best.

Tim
Reply
01-27-2018, 06:08 AM, (This post was last modified: 01-27-2018, 06:09 AM by TurboPT.)
#5
RE: ImportBankTrans Error
Unfortunately, that link is dead now. Sad

Though that does make me think to look at SVN info instead of scouring sourceforge!

I'll carefully compare the files that are working against those that are not as well to see if there's any difference.
Reply
01-27-2018, 06:12 AM,
#6
RE: ImportBankTrans Error
Does this link work now?

http://svnbook.red-bean.com/en/1.7/svn.a...words.html
Reply
01-27-2018, 06:13 AM, (This post was last modified: 01-27-2018, 06:18 AM by TurboPT.)
#7
RE: ImportBankTrans Error
The link is not dead, there are extra chars at the end of the link in post 4.

Yes, it works now, thanks!
Reply


Forum Jump:


Users browsing this thread: 1 Guest(s)