Add derivative for PAIN format "pain.008.001.02 without BIC and CdtrSchmId:Nm" with conditional address for Switzerland

Project60 / org.project60.sepa

SEPA direct debit integration with civicrm

19 stars 46 forks source link

Open Detsieber opened 1 year ago

Detsieber commented 1 year ago

This is a fix for #639.

This is tested in production and works well.

Detsieber commented 1 year ago

Would be nice to include into 1.7

bjendres commented 1 year ago

Hi @Detsieber, thanks for the PR. But, as stated in https://github.com/Project60/org.project60.sepa/issues/639#issuecomment-1268431566, please adjust the PR so it creates a new file format with the desired properties. This way we won't jeopardise the submissions of people using the file format you're changing...

jensschuppe commented 1 year ago

Copying from #639 what needs to be done here:

Add a directory for the new format in templates/Sepa/Formats/ with an appropriate name
Add the files Format.php, transaction-details.tpl, transaction-header.tpl and transaction-footer.tpl inside that directory (possibly copying from the existing format you wanted to adjust - make sure to adjust the class name in Format.php)
Add the format to the option group resource file
Add an upgrade task that synchronises that option group with the corresponding resource file

bjendres commented 1 year ago