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

Completed payment does not correctly detects waived vs paid scenario #10834

Open
jardakotesovec opened this issue Jan 22, 2025 · 1 comment
Open
Assignees
Labels
Bug:2:Moderate A bug that is causing problems for a substantial minority of users.

Comments

@jardakotesovec
Copy link
Contributor

jardakotesovec commented Jan 22, 2025

Describe the bug
From @Tribunal33 :
For 3.5 OJS when I have payments enabled and I move from unpaid or paid, to waived it doesn't seem to save it and goes back to paid when I reload the submission. This doesn't happen on 3.4 as far as I could tell.

Image

Investigation
I tracked it down to the change of the completed_payments amount column type - https://github.com/pkp/ojs/blame/main/classes/migration/install/OJSMigration.php#L418

As result this condition is not working as expected - https://github.com/pkp/ojs/blob/main/classes/components/forms/publication/SubmissionPaymentsForm.php#L53

Just improving this condition might be enough.. but better to properly review impact of this change.

What application are you using?
OJS, OMP or OPS version 3.5

@jardakotesovec jardakotesovec added the Bug:2:Moderate A bug that is causing problems for a substantial minority of users. label Jan 22, 2025
@jardakotesovec
Copy link
Contributor Author

@touhidurabir Could you have a look?

@touhidurabir touhidurabir self-assigned this Jan 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug:2:Moderate A bug that is causing problems for a substantial minority of users.
Projects
None yet
Development

No branches or pull requests

2 participants