New FA extensions and bug fixes #3
Open
Conversation
This file contains 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
added 30 commits
July 2, 2017 08:45
…journal and support batch processing in import_transactions
…ems with no quantity)
…inventory options to osc
braathwaate
force-pushed
the
master
branch
2 times, most recently
from
6911f9c
to
9cc71f9
Compare
apmuthu
reviewed
Oct 2, 2023
| @@ -317,7 +317,7 @@ function auto_create($current) | |||
| $to_reconciled=date2sql($_POST['reconcile_date']); | |||
| } | |||
| // display_notification($from_account ." " . $to_account . " " . $newdate ." " . $amt ." " . $reference); | |||
| add_bank_transfer($from_account, $to_account, $newdate, $amt, $reference, "", 0, 0, $from_reconciled, $to_reconciled); | |||
| add_bank_transfer($from_account, $to_account, $newdate, $amt, $reference, "", $sim['dim1'], $sim['dim2'], 0,0, $from_reconciled, $to_reconciled); | |||
There was a problem hiding this comment.
Remove the 0,0, in the changed line.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Adds auto-reconcile extension to FA. Takes a CSV bank statement and searches for existing uncleared FA transactions with the same amount and check number (check number must be in memo field). Will also automatically add and reconcile single G/L auto-pay transactions using the G/L account from a prior auto-pay transaction. (Not quite finished yet, but useful in current form to identify missing or mismatched FA transactions).
Adds daily sweep transfer extension to FA. Takes daily activity in a FA bank account and sweeps it to another FA bank account. This is useful for credit card sales that are batched on a daily basis and also for cash sales till that is deposited on a daily basis.
Adds P/L drilldown extension showing all dimensions at once. Based on the built-in FA P/L drilldown, an important feature is that it separates and shows G/L with No Dimension, allowing transactions with no dimension to be easily identified.