Skip to content

Add the IBAN and the mandate reference on the contribution list page - #802

Open
jaapjansma wants to merge 2 commits into
Project60:masterfrom
jaapjansma:issue_801
Open

Add the IBAN and the mandate reference on the contribution list page#802
jaapjansma wants to merge 2 commits into
Project60:masterfrom
jaapjansma:issue_801

Conversation

@jaapjansma

@jaapjansma jaapjansma commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Fixes #801.

Background

At one of our clients we had an issue where the generated contributions ended up with all the same recurring contribution .

This was not visible in the user interface and only visible the bank file. As each transaction was deducted from the same bank account. (So there was one donor who paid for all donors)

One of the things which would likely prevented this is when the IBAN and the mandate reference would have been visible in the UI.

Before

When clicking on Contributions next to a SEPA group no info was given on the mandate.

After

The mandate reference and the IBAN are visible.

2026-06-18_09-50

@jaapjansma
jaapjansma force-pushed the issue_801 branch 7 times, most recently from 03b3090 to f289d25 Compare June 18, 2026 08:20
@jaapjansma

Copy link
Copy Markdown
Contributor Author

The PHP Stan checks are failing but I am not sure why and what can be done about it.

Comment thread CRM/Sepa/Page/ListGroup.php Outdated
@jaapjansma
jaapjansma force-pushed the issue_801 branch 2 times, most recently from e12106d to 968c2e1 Compare June 18, 2026 13:03
@jensschuppe jensschuppe added feature status:needs review Code needs review and testing labels Jun 19, 2026
@jensschuppe

Copy link
Copy Markdown
Contributor

Noting that this depends on #803 as otherwise the batching query will return different results.

The tests not failing is actually a sign we should add more tests …

@jaapjansma

Copy link
Copy Markdown
Contributor Author

@jensschuppe the php unit is failing on the function data_processor_postInstall howver in the dataprocessor.php file that function is removed a while ago, see: https://lab.civicrm.org/extensions/dataprocessor/-/commit/eef9cd6b91a811edcbff9866e4a95e46ab93ff93?file_path=dataprocessor.civix.php#line_f717dbb74_126

Since then there are two new versions released of the data processor extension. And it seems that the phpunit here is running an old version.

@jaapjansma

Copy link
Copy Markdown
Contributor Author

@jensschuppe ah now all checks have passed!

@jensschuppe jensschuppe added status:needs work There is code, but it needs additional work before it should be reviewed and removed status:needs review Code needs review and testing labels Aug 14, 2026
@jensschuppe jensschuppe changed the title Fixed #801: added the IBAN and the mandate reference on the contribution list page Add the IBAN and the mandate reference on the contribution list page Aug 14, 2026
@jensschuppe

Copy link
Copy Markdown
Contributor

Needs rebase once #804 is merged (and probably a refactoring given that this currently contains a workaround for the join issue fixed with switching to EFv2).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature status:needs work There is code, but it needs additional work before it should be reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Show the mandate reference and the IBAN in the contributions list

2 participants