Update QrCode.php wrong variable#1129
Merged
nielsdrost7 merged 1 commit intoInvoicePlane:developmentfrom Dec 6, 2024
Merged
Conversation
When an invoice is paid in part. The QR code always displays the total paid, not what remains to be paid.
Contributor
|
Good one! Let us know when it's ready to be mergedM |
Contributor
Author
Except for the branch, it's ready to be merged. |
Contributor
Good question, I can make a "discussion" out of it, if you like. We only have Whenever I see an issue, I create a branch for it, so Contributors can work on it. Apparently it doesn't work that way since everyone wants to work on their own repository, which is fine. Other branches:
I'll clean up the branches after 1.6.2 goes live |
|
Excellent decision
|
Unfortunately, this is a necessity, even if you don't like it. If something needs to be edited, it is necessary to branch it into your own repository - this is the only way some people have their own repository. This could be unnecessary if the branch could be edited directly. |
Contributor
I'm going to say 1 thing about this. Because you're part of the Contributors you should get able to push your branch to the InvoicePlane repository. If not, then just howler and we'll look at the rights and things like that. No need to fork the repository. But ok, it is what it is. PR is merged @VizardAlpha , thanks for your contribution man! |
Merged
nielsdrost7
added a commit
that referenced
this pull request
Dec 30, 2024
## New Contributors Huge thanks to @sudwebdesign , @xam-ps and @AeroBytesNL for helping. Without you guys this release wouldn't have been possible * @sudwebdesign made their first contribution in #1149 * @VizardAlpha made their first contribution in #1129 * @pumpi made their first contribution in #1079 * @jmclaren7 made their first contribution in #1013 * @RobiNN1 made their first contribution in #1014 * @xeruf made their first contribution in #1061 * @NiklasSchmitt made their first contribution in #1073 * @redxtech made their first contribution in #1098 ## What's Changed * Add buttons on client view to delete note (Ajax) by @sudwebdesign in #1149 * Translate Client "Extra Field Title" to complete #1003 by @sudwebdesign in #1150 * [IP-1146]: after posting a Payments Form when amount > inv. total by @sudwebdesign in #1151 * [IP-1147]: Fix setup sql filenames by @sudwebdesign in #1152 * [IP-1130]: Added required input check on full page loaded to fix #1130 by @AeroBytesNL in #1155 * bug-fix-#1147-error-on-database-migration by @AeroBytesNL in #1159 * Fix setup Red Screen Of Death with bad DB query by @sudwebdesign in #1160 * **[IP-1128]**: Update QrCode.php wrong variable by @VizardAlpha in #1129 * Improve download function in #1127 * Chore: Fix upload_file function in #1141 * Show list of themes on Windows by @RobiNN1 in #1014 * [IP-1038]: Fix for issue 1038: Wrong translation string in setup by @naui95 in #1039 * [IP-1012]: Revert change to invoice_logo() by @jmclaren7 in #1013 * Slight improvements to README.md by @xeruf in #1061 * [IP-1070] by @naui95 in #1071 * [IP-1072] Fixed broken customer-link in projects-widget on dashboard. by @NiklasSchmitt in #1073 * [IP-1078]: ZUGFeRD Name should not be user name by @pumpi in #1079 * [IP-1006]: feature: payments v1.6.2 by @naui95 in #1046 * added information on theming by @naui95 in #1087 * [IP-1097]: add docker publish workflow by @redxtech in #1098 * [IP-841]: Copy Quote UI feedback by @Verony-makesIT in #932 * [IP-998]: copy all fields available in quotes by @naui95 in #1086 * [IP-1083]: Add pagination for tabs in client detail by @pumpi in #1084 * [IP-731]: Add another digit for quantity (new version) by @naui95 in #1112 * [IP-1003]: Add extra field title by nielsdrost7 in #1101 * add function to check post variables by nielsdrost7 in #1090 * [IP-1096] Payments array variable gets lost for invoices by nielsdrost7 in #1104 * Chore: add pint by nielsdrost7 in #1081 * [IP-1033] creation dates of invoices should never be changed by nielsdrost7 in #1040 * [IP-1034]: Fix error where default_language at some point switches to English by nielsdrost7 in #1035 * Make sure we don't get unintended output in our PDF files by nielsdrost7 in #1089 * [IP-1010]: Add php-dom and php-xml to the Dockerfile by nielsdrost7 in #1110 * Special project by nielsdrost7 in #1120 * [IP-1063]: Allowing for Dynamic Properties (**PHP 8.2**) by nielsdrost7 in #1117 Co-authored-by: Thomas Ingles Co-authored-by: AutiCodes Co-authored-by: AeroBytes Co-authored-by: naui95 Co-authored-by: VeRony Co-authored-by: Marc Heiduk Co-authored-by: Róbert Kelčák Co-authored-by: John Mclaren Co-authored-by: Janek Co-authored-by: Niklas Co-authored-by: stephan4p Co-authored-by: pumpi Co-authored-by: der-peer Co-authored-by: Nathan Mattes Co-authored-by: Kevin Joudrier Co-authored-by: Gabe Dunn Co-authored-by: VizardAlpha
ThierryHFR
pushed a commit
to ThierryHFR/InvoicePlane
that referenced
this pull request
Dec 21, 2025
Huge thanks to @sudwebdesign , @xam-ps and @AeroBytesNL for helping. Without you guys this release wouldn't have been possible * @sudwebdesign made their first contribution in InvoicePlane#1149 * @VizardAlpha made their first contribution in InvoicePlane#1129 * @pumpi made their first contribution in InvoicePlane#1079 * @jmclaren7 made their first contribution in InvoicePlane#1013 * @RobiNN1 made their first contribution in InvoicePlane#1014 * @xeruf made their first contribution in InvoicePlane#1061 * @NiklasSchmitt made their first contribution in InvoicePlane#1073 * @redxtech made their first contribution in InvoicePlane#1098 * Add buttons on client view to delete note (Ajax) by @sudwebdesign in InvoicePlane#1149 * Translate Client "Extra Field Title" to complete InvoicePlane#1003 by @sudwebdesign in InvoicePlane#1150 * [IP-1146]: after posting a Payments Form when amount > inv. total by @sudwebdesign in InvoicePlane#1151 * [IP-1147]: Fix setup sql filenames by @sudwebdesign in InvoicePlane#1152 * [IP-1130]: Added required input check on full page loaded to fix InvoicePlane#1130 by @AeroBytesNL in InvoicePlane#1155 * bug-fix-InvoicePlane#1147-error-on-database-migration by @AeroBytesNL in InvoicePlane#1159 * Fix setup Red Screen Of Death with bad DB query by @sudwebdesign in InvoicePlane#1160 * **[IP-1128]**: Update QrCode.php wrong variable by @VizardAlpha in InvoicePlane#1129 * Improve download function in InvoicePlane#1127 * Chore: Fix upload_file function in InvoicePlane#1141 * Show list of themes on Windows by @RobiNN1 in InvoicePlane#1014 * [IP-1038]: Fix for issue 1038: Wrong translation string in setup by @naui95 in InvoicePlane#1039 * [IP-1012]: Revert change to invoice_logo() by @jmclaren7 in InvoicePlane#1013 * Slight improvements to README.md by @xeruf in InvoicePlane#1061 * [IP-1070] by @naui95 in InvoicePlane#1071 * [IP-1072] Fixed broken customer-link in projects-widget on dashboard. by @NiklasSchmitt in InvoicePlane#1073 * [IP-1078]: ZUGFeRD Name should not be user name by @pumpi in InvoicePlane#1079 * [IP-1006]: feature: payments v1.6.2 by @naui95 in InvoicePlane#1046 * added information on theming by @naui95 in InvoicePlane#1087 * [IP-1097]: add docker publish workflow by @redxtech in InvoicePlane#1098 * [IP-841]: Copy Quote UI feedback by @Verony-makesIT in InvoicePlane#932 * [IP-998]: copy all fields available in quotes by @naui95 in InvoicePlane#1086 * [IP-1083]: Add pagination for tabs in client detail by @pumpi in InvoicePlane#1084 * [IP-731]: Add another digit for quantity (new version) by @naui95 in InvoicePlane#1112 * [IP-1003]: Add extra field title by nielsdrost7 in InvoicePlane#1101 * add function to check post variables by nielsdrost7 in InvoicePlane#1090 * [IP-1096] Payments array variable gets lost for invoices by nielsdrost7 in InvoicePlane#1104 * Chore: add pint by nielsdrost7 in InvoicePlane#1081 * [IP-1033] creation dates of invoices should never be changed by nielsdrost7 in InvoicePlane#1040 * [IP-1034]: Fix error where default_language at some point switches to English by nielsdrost7 in InvoicePlane#1035 * Make sure we don't get unintended output in our PDF files by nielsdrost7 in InvoicePlane#1089 * [IP-1010]: Add php-dom and php-xml to the Dockerfile by nielsdrost7 in InvoicePlane#1110 * Special project by nielsdrost7 in InvoicePlane#1120 * [IP-1063]: Allowing for Dynamic Properties (**PHP 8.2**) by nielsdrost7 in InvoicePlane#1117 Co-authored-by: Thomas Ingles Co-authored-by: AutiCodes Co-authored-by: AeroBytes Co-authored-by: naui95 Co-authored-by: VeRony Co-authored-by: Marc Heiduk Co-authored-by: Róbert Kelčák Co-authored-by: John Mclaren Co-authored-by: Janek Co-authored-by: Niklas Co-authored-by: stephan4p Co-authored-by: pumpi Co-authored-by: der-peer Co-authored-by: Nathan Mattes Co-authored-by: Kevin Joudrier Co-authored-by: Gabe Dunn Co-authored-by: VizardAlpha
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
When an invoice is paid in part. The QR code always displays the total paid, not what remains to be paid.
Related Issue
#1128
Motivation and Context
Avoid paying my customer twice for an amount that is already half paid.
As for the branch. I'm having a bit of trouble understanding the variety of branches that InvoicePlane currently has. I didn't check the last two checklists because of this.
Pull Request Checklist
Issue Type