ImportBankTrans Error
|
01-27-2018, 01:45 AM
Post: #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 |
|||
01-27-2018, 05:31 AM
Post: #2
|
|||
|
|||
RE: ImportBankTrans Error
Hmmm, that is already fixed in SVN. In fact, I applied that change in April last year!
![]() Not sure how you might have it the old way? Oh well, good to know that it's already covered. |
|||
01-27-2018, 05:46 AM
(This post was last modified: 01-27-2018 05:47 AM by TurboPT.)
Post: #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. |
|||
01-27-2018, 06:03 AM
Post: #4
|
|||
|
|||
RE: ImportBankTrans Error
Now then, that is stretching my memory a bit.
![]() It has to do with keyword substitution (http://svnbook.red-bean.com/en/1.7/svn.a...rds.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 |
|||
01-27-2018, 06:08 AM
(This post was last modified: 01-27-2018 06:09 AM by TurboPT.)
Post: #5
|
|||
|
|||
RE: ImportBankTrans Error
Unfortunately, that link is dead now.
![]() 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. |
|||
01-27-2018, 06:12 AM
Post: #6
|
|||
|
|||
RE: ImportBankTrans Error | |||
01-27-2018, 06:13 AM
(This post was last modified: 01-27-2018 06:18 AM by TurboPT.)
Post: #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! |
|||
« Next Oldest | Next Newest »
|
User(s) browsing this thread: 2 Guest(s)