Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Copy donation receipts inconsistent? #169

Closed
MarcMichalsky opened this issue Aug 29, 2023 · 9 comments · Fixed by #170
Closed

Copy donation receipts inconsistent? #169

MarcMichalsky opened this issue Aug 29, 2023 · 9 comments · Fixed by #170
Labels
bug status:reviewed and tested Code has received thorough review and testing and is ready to be committed/merged.
Milestone

Comments

@MarcMichalsky
Copy link
Contributor

Hi,

I think I have encountered an inconsistency in copying donation receipts.

When copying, the field issued_on is overwritten with the current date. This results in the generated copy also showing the current date and not the original issue date. (I am already using the $issued_on field in the template instead of $today)

Maybe this is intentional, but in this case it is a problem that not the current default profile but the original profile is used.

In our case, this leads to the fact that when we issue a copy of an older donation receipt, the current date is used and below it is the signature of our former board member, who does not work for us anymore.

So we have to make a decision at this point: Is a copy a true copy of the original donation receipt with the original date, or is it a new receipt with the current date and current profile (resp. template)?

@bjendres bjendres added enhancement help wanted status:needs work There is code, but it needs additional work before it should be reviewed. labels Aug 30, 2023
@bjendres
Copy link
Member

Thanks for reporting, @MarcMichalsky

Is a copy a true copy of the original donation receipt with the original date, or is it a new receipt with the current date and current profile (resp. template)?

I'll pass this on to the people who work with this most.

@bjendres
Copy link
Member

bjendres commented Aug 30, 2023

So we have to make a decision at this point: Is a copy a true copy of the original donation receipt with the original date, or is it a new receipt with the current date and current profile (resp. template)?

I talked to Juliane and we both think the copy should be a real, but watermarked copy, and the original issued_on value should be preserved.

The scenario is, that the original donation receipt was lost, and the system is asked to produce a copy. This copy then has to

  1. be identical to the "lost" original, including issue date
  2. be clearly marked as a copy (by watermark)

The system must not to be able to produce two originals for the same contribution (short of tampering with the db/system).

@MarcMichalsky So I think you're right, we should fix this behaviour.

@jensschuppe any opposing opinion?

@bjendres bjendres added question bug and removed enhancement help wanted status:needs work There is code, but it needs additional work before it should be reviewed. labels Aug 30, 2023
MarcMichalsky added a commit to MarcMichalsky/de.systopia.donrec that referenced this issue Aug 30, 2023
The issued_by field is not overwritten when copying a profile
@MarcMichalsky
Copy link
Contributor Author

Okay, it would be easy to fix. This worked for me.

@jensschuppe
Copy link
Collaborator

@bjendres I would agree, though this is a legal question I can't answer for sure. But it appears to me that (in Germany) organisations are required to keep a copy of the receipt and store it for at least 10 years - this can be in electronic form (which is basically what we do here), so the copy would have to show data as it would when created on the date the original was issued.

I'm just wondering why this hasn't been an issue so far ...

@MarcMichalsky
Copy link
Contributor Author

We should also adjust the default template accordingly, otherwise the changes will not be applied.

Of course, many organizations use custom templates. They would need to adjust these themselves.

@bjendres
Copy link
Member

[...] so the copy would have to show data as it would when created on the date the original was issued

Yes, I agree, see above. (I edited the comment after first submission, in case you're answering to an email)

@MarcMichalsky
Copy link
Contributor Author

Should I submit a PR with my suggested fixes or do you want to apply them quickly yourself?

@bjendres
Copy link
Member

bjendres commented Sep 1, 2023

Should I submit a PR with my suggested fixes or do you want to apply them quickly yourself?

Yes, please. We like PRs :)

@jensschuppe
Copy link
Collaborator

Also, your commits: your laurels, @MarcMichalsky! 🥇

@jensschuppe jensschuppe added this to the 2.2 milestone Sep 4, 2023
@jensschuppe jensschuppe added status:reviewed and tested Code has received thorough review and testing and is ready to be committed/merged. and removed question labels Sep 4, 2023
jensschuppe added a commit that referenced this issue Nov 6, 2024
The issued_by field is not overwritten when copying a donation receipt

Use $issued_on instead of $today to print the original document date.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug status:reviewed and tested Code has received thorough review and testing and is ready to be committed/merged.
Projects
None yet
3 participants