search

Project60 / org.project60.banking

CiviCRM banking extension
18 stars 35 forks source link

viewing a processed transaction does wrongly allow to Analyse again #212

Open tttp opened 5 years ago

tttp commented 5 years ago

So I'm shooting from the hip, so I'm not sure if it's a bug or me that misuse the tool: When reviewing a processed transaction 42:

/civicrm/banking/review?id=42

it does offer to re-analyse the transaction, but the result is:

   [values] => Array
        (
            [payment_count] => 1
            [processed_count] => 0
            [skipped_count] => 1
            [time] => 0
            [timed_out] => 0
        )

ie it does skip an already processed transaction. I'm not sure what processed transaction means yet, but it should either disable the button or properly re-analyse the transaction

bjendres commented 5 years ago

That's weird, on my system the 'analyse' button is always disabled.

@tttp Could you check the error logs, e.g. for JS issues?