FrontAccounting forum — Import Transactions 2.3.22-4 2.3.22-4

Re: Import Transactions 2.3.22-4 2.3.22-4

2015-03-19T15:28:02Z

Link to commit would be useful. Your commit list does not show it as yet.

Re: Import Transactions 2.3.22-4 2.3.22-4

2015-03-16T14:13:20Z

Code has been adjusted in the extension.

Re: Import Transactions 2.3.22-4 2.3.22-4

2014-12-18T16:20:50Z

$trial = (isset($_POST['trial']) && ($_POST['trial']!=null)) ;

is way better.

Re: Import Transactions 2.3.22-4 2.3.22-4

2014-12-17T08:45:58Z

I believe it sets the value to 1.

It is a very confusing way of testing if a box is checked or not.

It was failing to cater for the box not being checked.

if (isset($_POST['trial']) && ($_POST['trial']!=null)) {
$trial = true;
} else {
$trial = false;
}

Is probably neater.

HTH

Re: Import Transactions 2.3.22-4 2.3.22-4

2014-12-17T04:31:13Z

What is the meaning of $trial = !null; ? Is the value true or false? Is it's value dependent on PHP version used?

This logic can certainly do with some housekeeping.

Re: Import Transactions 2.3.22-4 2.3.22-4

2014-12-16T15:54:04Z

5.2.17

Re: Import Transactions 2.3.22-4 2.3.22-4

2014-12-16T09:06:23Z

What is the version of your PHP?

Import Transactions 2.3.22-4 2.3.22-4

2014-12-15T17:09:53Z

When testing this feature I get the message

67 Payments would have been successful if imported. Uncheck Trial check before importing.

Around line 238 of /modules/import_transactions/import_transactions.php change

// Commit import to database
if ((isset($_POST['trial'])) && ($_POST['trial']==null)){
$trial = null;
}
if (isset($_POST['trial']) && ($_POST['trial']!=null)) {
$trial = !null;
}

if (!isset($_POST['trial']) ) {
$trial = null;
}

seems to do the trick.

HTH