FrontAccounting forum — Import Transactions v2.3.22-5 fix and code cleanup

Re: Import Transactions v2.3.22-5 fix and code cleanup

null@example.com (apmuthu) — Sun, 28 Dec 2014 20:04:28 +0000

I have notified the author Ross Addison but got a cryptic auto reply from another email address!

Only one set of redundant notification lines were there that have been removed - all others are equivalent code for ease of readability besides another notification added for the state when on $trial there were no records.

It is placed in the forums so that users can directly partake of it and in context. Also, any eagle eyes that can spot any errors in it would be more than welcome to improve upon it.

Further cleanup has been done.

Re: Import Transactions v2.3.22-5 fix and code cleanup

null@example.com (itronics) — Sun, 28 Dec 2014 18:47:34 +0000

Please use the author/maintainer email found in the package info to address such issues in publishes packages in the future, there is no warranty the extension author read this forum.

BTW While it may seem a little redundant to your eyes, both forms are proper and one could judge the current one is even more canonical. This ise disputable, but anyway outside interest of most forum users. Therefore I think this is another good reason to address such notices directly to the author.

Janusz

Import Transactions v2.3.22-5 fix and code cleanup

null@example.com (apmuthu) — Sun, 28 Dec 2014 12:55:34 +0000

In today's inclusion of Branch Number in the Import transactions v2.3.22-5 extension , please change the several instances of:

($trial == true)

$trial

There is also duplication of display notification at the end when $trial is true in import_transactions.php.

The full patch is attached.

The commit is in my GitHub Repo.

Additional formatting of the single lined README.md file and the hashes update are also done.

In fact, the initialisation at line 103 may not be needed ($trial=false;).