Gift Batch Import File Specification: Difference between revisions
No edit summary |
|||
Line 23: | Line 23: | ||
'''GL Effective Date''' – <nowiki>[date] </nowiki> Required. The date must be within the date range of the current and open forward periods. | '''GL Effective Date''' – <nowiki>[date] </nowiki> Required. The date must be within the date range of the current and open forward periods. | ||
: '''Validation Rules:''' Validate against period dates. Search for TFinancialYear.IsValidPostingPeriod. See GL.Importing.cs for example. | : '''Validation Rules:''' Must be a valid date in the date format specified for the batch. Validate against period dates. Search for TFinancialYear.IsValidPostingPeriod. See GL.Importing.cs for example. | ||
* [[User:Dwmosman|Dwmosman]]What are the requirements here? Does this date have to be within the date range of the current and following accounting periods? If empty, does it default to today's date? Are there any cross-edits with gift date? | * [[User:Dwmosman|Dwmosman]]What are the requirements here? Does this date have to be within the date range of the current and following accounting periods? If empty, does it default to today's date? Are there any cross-edits with gift date? | ||
* [[User:Pokorra|Pokorra]] gl effective date has to be in the date range of current and open forwarding periods. there should be existing methods to check that. See [http://bazaar.launchpad.net/~openpetracore/openpetraorg/trunkhosted/view/head:/csharp/ICT/Petra/Server/lib/MFinance/GL/GL.Importing.cs csharp\ICT\Petra\Server\lib\MFinance\GL\GL.Importing.cs], search for TFinancialYear.IsValidPostingPeriod. The value cannot be empty. | * [[User:Pokorra|Pokorra]] gl effective date has to be in the date range of current and open forwarding periods. there should be existing methods to check that. See [http://bazaar.launchpad.net/~openpetracore/openpetraorg/trunkhosted/view/head:/csharp/ICT/Petra/Server/lib/MFinance/GL/GL.Importing.cs csharp\ICT\Petra\Server\lib\MFinance\GL\GL.Importing.cs], search for TFinancialYear.IsValidPostingPeriod. The value cannot be empty. | ||
Line 41: | Line 41: | ||
* [[User:Pokorra|Pokorra]] cost centre is not related to bank. can be any active costcentre, table a_cost_centre, depending on the ledger number. | * [[User:Pokorra|Pokorra]] cost centre is not related to bank. can be any active costcentre, table a_cost_centre, depending on the ledger number. | ||
* [[User:Robertpickett|Robertpickett]] The Cost Centre must also have a_cost_centre_type_c = "Local" and must be a Posting Cost Centre (a_posting_cost_centre_flag_l). In Petra, if it can't find the Cost Centre it trys again by adding a leading 0 (because of Excel's tendency to remove leading zero's when you don't want it to. If it then finds a match it asks the user if they want to use that CC instead (eg. "0300" instead of "300") | * [[User:Robertpickett|Robertpickett]] The Cost Centre must also have a_cost_centre_type_c = "Local" and must be a Posting Cost Centre (a_posting_cost_centre_flag_l). In Petra, if it can't find the Cost Centre it trys again by adding a leading 0 (because of Excel's tendency to remove leading zero's when you don't want it to. If it then finds a match it asks the user if they want to use that CC instead (eg. "0300" instead of "300") | ||
* [[User:Dwmosman|Dwmosman]] Rob, are there alternatives to this approach? Interactive batch loads across the client/server boundary can be tricky (for example, what if the requestor fails to notice the question dialog and the table is locked for all users for the day?). I will send an email on this | * [[User:Dwmosman|Dwmosman]] 08:52, 26 February 2013 (UTC) Rob, are there alternatives to this approach? Interactive batch loads across the client/server boundary can be tricky (for example, what if the requestor fails to notice the question dialog and the table is locked for all users for the day?). I will send an email on this | ||
'''Gift Type''' – <nowiki>[16ch.] </nowiki> Optional. Must be "Gift", "Gift In Kind" or "Other". If left blank, it defaults to "Gift". | '''Gift Type''' – <nowiki>[16ch.] </nowiki> Optional. Must be "Gift", "Gift In Kind" or "Other". If left blank, it defaults to "Gift". | ||
Line 48: | Line 48: | ||
* [[User:Pokorra|Pokorra]] please use the hard coded constants from [http://bazaar.launchpad.net/~openpetracore/openpetraorg/trunkhosted/view/head:/csharp/ICT/Petra/Shared/lib/MFinance/Constants.cs Shared/lib/MFinance/Constants.cs], GIFT_TYPE_* | * [[User:Pokorra|Pokorra]] please use the hard coded constants from [http://bazaar.launchpad.net/~openpetracore/openpetraorg/trunkhosted/view/head:/csharp/ICT/Petra/Shared/lib/MFinance/Constants.cs Shared/lib/MFinance/Constants.cs], GIFT_TYPE_* | ||
* [[User:Dwmosman|Dwmosman]] Could these constants be enumerated or a collection or something? Or are they so static that there will always be these three and I can just hard-code them into my validation routine? | * [[User:Dwmosman|Dwmosman]] Could these constants be enumerated or a collection or something? Or are they so static that there will always be these three and I can just hard-code them into my validation routine? | ||
* [[User:Dwmosman|Dwmosman]] In the OpenPetra database design it says that this field defaults to "gift". Is this acceptable? Or must the importer always put a value in this field? | * [[User:Dwmosman|Dwmosman]] 08:52, 26 February 2013 (UTC) In the OpenPetra database design it says that this field defaults to "gift". Is this acceptable? Or must the importer always put a value in this field? | ||
== Gift Record fields== | == Gift Record fields== | ||
Line 54: | Line 54: | ||
*[[User:Pokorra|Pokorra]] I think that one file will either be in the shorter, or in the extended format, not mixed. I see, the import code checks for every transaction. that is fine, but not required. | *[[User:Pokorra|Pokorra]] I think that one file will either be in the shorter, or in the extended format, not mixed. I see, the import code checks for every transaction. that is fine, but not required. | ||
* [[User:Robertpickett|Robertpickett]] In Petra the extended format is export only (provided to give a convenient csv file for manipulation in Excel), the import doesn't support it. I'm not sure whether it makes sense to import the long format since a number of the fields are calculated fields. | * [[User:Robertpickett|Robertpickett]] In Petra the extended format is export only (provided to give a convenient csv file for manipulation in Excel), the import doesn't support it. I'm not sure whether it makes sense to import the long format since a number of the fields are calculated fields. | ||
* [[User:Dwmosman|Dwmosman]] Should I eliminate the extended format? (Fields preceded by a ">" sign are part of the extended format) | |||
'''Row Type''' - “T” | '''Row Type''' - “T” | ||
'''Donor Key''' – <nowiki>[ | '''Donor Key''' – <nowiki>[integer] </nowiki> The partner key of the donor. The donor must be entered as a partner before the gift batch is imported. Note that if the donors partner record has been merged into another partner, then the new partner key must be specified instead of the key of the merged record. | ||
: '''Validation Rules:''' Must be a valid key into the partner table, p_partner. However, if the partner status is "MERGED", then the new partner must be used instead. Ideally the system will look up the new new partner and ask the user if they want to use the new partner. If yes, replace the key with the new key. If no, then fail the import. | |||
* [[User:Dwmosman|Dwmosman]] Must be a valid donor key from partner(?) table, xxx | |||
*[[User:Pokorra|Pokorra]] table p_partner, no special requirement. usually it is a partner of p_partner_class_c = "FAMILY", but I think that is not a requirement??? Rob? | *[[User:Pokorra|Pokorra]] table p_partner, no special requirement. usually it is a partner of p_partner_class_c = "FAMILY", but I think that is not a requirement??? Rob? | ||
* [[User:Robertpickett|Robertpickett]] The donor can be of any Partner Class. The one complication relates to Merged Partners. If the Partner Status is "MERGED" then the import should look up the Partner that it was merged into (maybe there is already a method for that in OpenPetra). If it can't find that Partner then import should fail, otherwise ask the user if they want to use that Partner instead of the one they entered (if they say no then import fails), otherwise use the new Partner. | * [[User:Robertpickett|Robertpickett]] The donor can be of any Partner Class. The one complication relates to Merged Partners. If the Partner Status is "MERGED" then the import should look up the Partner that it was merged into (maybe there is already a method for that in OpenPetra). If it can't find that Partner then import should fail, otherwise ask the user if they want to use that Partner instead of the one they entered (if they say no then import fails), otherwise use the new Partner. | ||
'''Donor Short Name''' – note: This field is required in the gift row but can be empty and is always ignored by the import process. | '''Donor Short Name''' – note: This field is required in the gift row but can be empty and is always ignored by the import process. | ||
: [[User:Dwmosman|Dwmosman]] Is there a reason we can't remove this field from the import since its not used? | |||
'''Method of Giving Code''' – <nowiki>[16ch] </nowiki> | '''Method of Giving Code''' – <nowiki>[16ch.] </nowiki> Select one of the options defined in the Method of Giving table for your installation. | ||
: '''Validation Rules:''' Must be an existing "a_method_of_giving_code_c" code in the Method of Giving table, a_method_of_giving. | |||
'''Method of Payment Code''' – <nowiki>[16ch] </nowiki> | '''Method of Payment Code''' – <nowiki>[16ch.] </nowiki> Select one of the options defined in the Method of Payment table for your installation. | ||
: '''Validation Rules:''' Must be an existing "a_method_of_payment_code_c" code in the Method of Payment table, a_method_of_payment. | |||
'''Reference''' – <nowiki>[20ch]</nowiki> | '''Reference''' – <nowiki>[20ch.] </nowiki> Reference number/code for the gift transaction. | ||
: '''Validation Rules:''' Cannot be longer than 20 characters. | |||
* [[User:Dwmosman|Dwmosman]] Can this field be empty? | |||
'''Receipt Letter Code''' – <nowiki>[16ch] </nowiki>values sourced from ??. Is <nowiki><none> is a valid value? </nowiki>Can this be blank? | '''Receipt Letter Code''' – <nowiki>[16ch.] </nowiki> Optional. "<none>" is the same as blank. Otherwise select one of the values from your installations Form Letter Design table. | ||
: '''Validation Rules:''' If empty or "<none>", populate the field with "<none>". Otherwise, the entry must be an existing code (p_design_name_c) in the table p_form_letter_design. | |||
* [[User:Dwmosman|Dwmosman]] values sourced from ??. Is <nowiki><none> is a valid value? </nowiki>Can this be blank? | |||
*[[User:Pokorra|Pokorra]] I think it will be blank most of the time, because it will only be filled when the gift batch is posted, and a receipt is printed. | *[[User:Pokorra|Pokorra]] I think it will be blank most of the time, because it will only be filled when the gift batch is posted, and a receipt is printed. | ||
* [[User:Robertpickett|Robertpickett]] <none> is valid, as is blank. In Petra, if blank then it gets imported as <none>. Otherwise must exist in p_form_letter_design (field p_design_name_c) | * [[User:Robertpickett|Robertpickett]] <none> is valid, as is blank. In Petra, if blank then it gets imported as <none>. Otherwise must exist in p_form_letter_design (field p_design_name_c) | ||
>'''Receipt Number''' – <nowiki>[ | >'''Receipt Number''' – <nowiki>[integer] </nowiki> Optional. Gift receipt number. Defaults to 0. | ||
: '''Validation Rules:''' Must be blank, 0 or a valid positive integer. | |||
>'''First Time Gift''' – <nowiki>[ | >'''First Time Gift''' – <nowiki>[boolean] </nowiki> Optional. Enter "yes" or "no" to indicate whether this is a first time gift for the donor. Defaults to "no". | ||
: '''Validation Rules:''' Must be "yes", "no" or blank. If blank, default to "no" (zero). | |||
>'''Receipt Printed''' – <nowiki>[bool]</nowiki> | >'''Receipt Printed''' – <nowiki>[bool]</nowiki> Optional. Enter "yes" or "no" to indicate whether the receipt has been printed for this gift. Defaults to "no". | ||
: '''Validation Rules:''' Must be "yes", "no" or blank. If blank, default to "no" (zero). | |||
'''Recipient Key''' – <nowiki>[ | '''Recipient Key''' – <nowiki>[integer] </nowiki> Select the key of the recipient from the partner table. | ||
: '''Validation Rules:''' Must be a valid key (p_partner_key_n) from the partner table, p_partner. Class (p_partner_key_n) must be "FAMILY" or "UNIT". If Family, tnen special type (p_partner_type) must be "WORKER". Partner status must not be "MERGED". | |||
* [[User:Dwmosman|Dwmosman]] Must be (active?) partner from partner table, p_partner: p_partner_key_n | |||
*[[User:Pokorra|Pokorra]] it should actually be a FAMILY partner (p_partner_class_c) that has special type WORKER (p_partner_type), or a UNIT partner. not sure if that should all be validated during the import, because it is quite extensive. we have methods for that during posting of a gift batch. | *[[User:Pokorra|Pokorra]] it should actually be a FAMILY partner (p_partner_class_c) that has special type WORKER (p_partner_type), or a UNIT partner. not sure if that should all be validated during the import, because it is quite extensive. we have methods for that during posting of a gift batch. | ||
* [[User:Robertpickett|Robertpickett]] Should also not allow Partners with status "MERGED". | * [[User:Robertpickett|Robertpickett]] Should also not allow Partners with status "MERGED". | ||
'''Recipient Short Name''' – ignored by import | '''Recipient Short Name''' – ignored by import | ||
* [[User:Dwmosman|Dwmosman]] Can I remove this from the import, since its ignored anyway? | |||
>'''Recipient Ledger Number''' – <nowiki>[ | >'''Recipient Ledger Number''' – <nowiki>[integer] </nowiki>Optional?. | ||
* [[User:Dwmosman|Dwmosman]] Equates to field ledger number? Maybe only for unit level recipients? Or for allocating to a family who works multiple fields? If entered, must match recipient field ledger number. On the gift screens this is auto generated from recipient. | |||
*[[User:Pokorra|Pokorra]] it is important for a worker as well, because the money will go to the field where he is working. but the gift screen generates it, and during gift posting it is calculated again, so it is not so important to be validated, I think. | *[[User:Pokorra|Pokorra]] it is important for a worker as well, because the money will go to the field where he is working. but the gift screen generates it, and during gift posting it is calculated again, so it is not so important to be validated, I think. | ||
* [[User:Robertpickett|Robertpickett]] The Petra import works this out based on the recipient, but I guess the best method for that still has to be worked out for OpenPetra. | * [[User:Robertpickett|Robertpickett]] The Petra import works this out based on the recipient, but I guess the best method for that still has to be worked out for OpenPetra. | ||
* [[User:Dwmosman|Dwmosman]] 08:52, 26 February 2013 (UTC) Can I remove this from the import? It sounds like it is recalculated at posting anyway so its basically ignored? | |||
'''Gift Amount''' – <nowiki>[ | '''Gift Amount''' – <nowiki>[decimal]</nowiki> Enter the gift amount. | ||
: '''Validation Rules:''' Must be a valid decimal number | |||
* [[User:Robertpickett|Robertpickett]] Must be non-zero. Can be positive or negative | * [[User:Robertpickett|Robertpickett]] Must be non-zero. Can be positive or negative | ||
* [[User:Dwmosman|Dwmosman]] 08:52, 26 February 2013 (UTC) Do I need to edit this for the correct number of decimal places? (some currencies have 0 or 3 decimal places) | |||
* [[User:Dwmosman|Dwmosman]] 08:52, 26 February 2013 (UTC) Are there rules about what can be entered here for Gift in Kind or Other? | |||
>'''Gift Amount International''' – <nowiki>[ | >'''Gift Amount International''' – <nowiki>[decimal] </nowiki> | ||
'''Confidential Gift Flag '''–<nowiki> [ | '''Confidential Gift Flag '''–<nowiki> [boolean]</nowiki> Optional. "yes" or "no". Enter yes to indicate a confidential gift. Defaults to "no". | ||
: '''Validation Rules:''' Must be "yes", "no" or blank. If blank, default to false. | |||
'''Motivation Group Code''' – <nowiki>[16ch] </nowiki>Optional? Validate against table a_motivation_group: a_ledger_number_i, a_motivation_group_code_c. (does value of a_restricted_l matter?) | '''Motivation Group Code''' – <nowiki>[16ch.] </nowiki> Select an entry from the installation's Motivation Group table for the active ledger. | ||
: '''Validation Rules:''' Must be a valid code (a_motivation_group_code_c) from the Motivation Group table, a_motivation_group, for the active ledger (a_ledger_number_i). | |||
* [[User:Dwmosman|Dwmosman]] Optional? Validate against table a_motivation_group: a_ledger_number_i, a_motivation_group_code_c. (does value of a_restricted_l matter?) | |||
*[[User:Pokorra|Pokorra]] we usually always require a valid motivation group and detail. a_restricted_l is just for read access permissions, who is allowed to see that gift. should not take effect here. | *[[User:Pokorra|Pokorra]] we usually always require a valid motivation group and detail. a_restricted_l is just for read access permissions, who is allowed to see that gift. should not take effect here. | ||
'''Motivation Detail Code'''– <nowiki>[16ch] </nowiki>Optional? Validate against table a_motivation_detail: a_ledger_number_i, a_motivation_group_code_c, a_motivation_detail_code_c (note: motivation detail already has fields for recipient, receipt, tax deductible, cost centre, etc. Should edit compare these to the corresponding gift fields in some way? Should these be used to populate the gift fields? Also, does the field, a_motivation_status_l, affect whether the motivation code can be used as a valid gift import code? | '''Motivation Detail Code'''– <nowiki>[16ch.] </nowiki> Select an entry from the installations Motivation Detail table for the selected Motivation Group Code for the active ledger | ||
: '''Validation Rules:''' Must be a valid code (a_motivation_detail_code_c) from the Motivation Detail table, a_motivation_detail with the active ledger (a_ledger_number_i), the selected group code (a_motivation_group_code_c) and with status = true (a_motivation_status_1). | |||
: [[User:Dwmosman|Dwmosman]] Optional? Validate against table a_motivation_detail: a_ledger_number_i, a_motivation_group_code_c, a_motivation_detail_code_c (note: motivation detail already has fields for recipient, receipt, tax deductible, cost centre, etc. Should edit compare these to the corresponding gift fields in some way? Should these be used to populate the gift fields? Also, does the field, a_motivation_status_l, affect whether the motivation code can be used as a valid gift import code? | |||
*[[User:Pokorra|Pokorra]] I think you do not need to copy values from motivation detail record. | *[[User:Pokorra|Pokorra]] I think you do not need to copy values from motivation detail record. | ||
* [[User:Robertpickett|Robertpickett]] Only Motivation Details with a_motivation_status_l = true should be allowed. There is some code needed here to populate information like Cost Centre, Account, etc which does need to be done but is not a straightforward matter of copying values (it depends on other factors as well). I imagine the code to do that has already been written for the Gift Batch screen. If p_recipient_key_n on a_motivation_detail is populated and doesn't match the recipient key in the import file (and the recipient key is not 0) then the import should fail with an error. | * [[User:Robertpickett|Robertpickett]] Only Motivation Details with a_motivation_status_l = true should be allowed. There is some code needed here to populate information like Cost Centre, Account, etc which does need to be done but is not a straightforward matter of copying values (it depends on other factors as well). I imagine the code to do that has already been written for the Gift Batch screen. If p_recipient_key_n on a_motivation_detail is populated and doesn't match the recipient key in the import file (and the recipient key is not 0) then the import should fail with an error. | ||
* [[User:Dwmosman|Dwmosman]] 08:52, 26 February 2013 (UTC) Actually, if you create the gift detail in OpenPetra, these fields are generated and you cannot manually enter or change them. So I assume, that they must be regenerated at post? (because the values in the DB could have changed between entry and post). So if they are in the import file and they contradict each other? So it seems that a consistent decision should be made as to which fields can be manually controlled (either online or via batch import) and which are to be automatically derived once other fields, like motivational detail, are selected? | |||
'''Cost Centre Code''' –<nowiki> [24ch] </nowiki> | '''Cost Centre Code''' –<nowiki> [24ch] </nowiki> | ||
* [[User:Robertpickett|Robertpickett]] I don't think this should be here? Is it part of the extended format? | * [[User:Robertpickett|Robertpickett]] I don't think this should be here? Is it part of the extended format? | ||
* [[User:Dwmosman|Dwmosman]] 08:52, 26 February 2013 (UTC) It is in the current OpenPetra code for the normal import format. Its also in the gift detail table in the database. Is this a design question that should be discussed? | |||
'''Gift Comment One''' – optional? Max length? | '''Gift Comment One''' – <nowiki>[160ch.] Optional. Long description of the gift. | ||
: '''Validation Rules''' - Check for SQL injection. | |||
* ~~~ optional? Max length? | |||
*[[User:Pokorra|Pokorra]] yes, is optional. length 160 characters: see http://openpetraorg.sourceforge.net/dbdoc/index.html?table=a_gift_detail&group=gift | *[[User:Pokorra|Pokorra]] yes, is optional. length 160 characters: see http://openpetraorg.sourceforge.net/dbdoc/index.html?table=a_gift_detail&group=gift | ||
'''Gift Comment One Type''' – are comment types hard-coded or tabled? Can this be blank if comment is not blank? | |||
'''Gift Comment One Type''' – Optional. If used, must contain a valid Gift Comment Type. Cannot be blank if Gift Comment One is used. | |||
: '''Validation Rules:''' - blank or a valid GIFT_COMMENT_TYPE. Cannot be blank if Gift Comment One is not blank. | |||
* ~~~ are comment types hard-coded or tabled? Can this be blank if comment is not blank? | |||
*[[User:Pokorra|Pokorra]] hard coded, see [http://bazaar.launchpad.net/~openpetracore/openpetraorg/trunkhosted/view/head:/csharp/ICT/Petra/Shared/lib/MFinance/Constants.cs Shared/lib/MFinance/Constants.cs], GIFT_COMMENT_TYPE*. that Constant.cs file is actually missing more constants: Donor, Recipient, Office | *[[User:Pokorra|Pokorra]] hard coded, see [http://bazaar.launchpad.net/~openpetracore/openpetraorg/trunkhosted/view/head:/csharp/ICT/Petra/Shared/lib/MFinance/Constants.cs Shared/lib/MFinance/Constants.cs], GIFT_COMMENT_TYPE*. that Constant.cs file is actually missing more constants: Donor, Recipient, Office | ||
* [[User:Robertpickett|Robertpickett]] Must contain a value if comment is not blank. | * [[User:Robertpickett|Robertpickett]] Must contain a value if comment is not blank. | ||
* ~~~~ As per my earlier question. Could these constants be kept in perhaps a generic that I could ping it with Linq or some such :) | |||
* ~~~~ For my own edification, can someone | |||
'''Mailing Code''' | '''Mailing Code''' – <nowiki>[16ch.] </nowiki> Optional. "<none>" is the same as blank. Otherwise select one of the values from your installations Mailing Code table. | ||
: '''Validation Rules:''' If empty or "<none>", populate the field with "<none>". Otherwise, the entry must be an existing code (p_mailing_code_c) in the table p_mailing. | |||
* [[User:Robertpickett|Robertpickett]] Same rules as for Receipt Letter Code but using p_mailing (p_mailing_code_c) for the lookup | * [[User:Robertpickett|Robertpickett]] Same rules as for Receipt Letter Code but using p_mailing (p_mailing_code_c) for the lookup | ||
'''Gift Comment Two''' | '''Gift Comment Two''' - See Gift Comment One | ||
'''Gift Comment Two Type''' | '''Gift Comment Two Type''' - See Gift Comment One Type | ||
'''Gift Comment Three''' | '''Gift Comment Three''' - See Gift Comment One | ||
'''Gift Comment Three Type''' | '''Gift Comment Three Type''' - See Gift Comment One type | ||
'''Tax Deductable''' | '''Tax Deductable''' <nowiki>[boolean] </nowiki> Optional. if blank, it will be filled from a_tax_deductible_flag_1 from the Motivation Detail. | ||
: '''Validation Rules''' - Must be blank, "yes" or "no". If blank, then the field will be populated from the a_tax_deductible_flag_1 from the selected Motivation Detail. | |||
* [[User:Robertpickett|Robertpickett]] Boolean. If blank then will be filled from a_tax_deductible_flag_l on a_motivation_detail. | * [[User:Robertpickett|Robertpickett]] Boolean. If blank then will be filled from a_tax_deductible_flag_l on a_motivation_detail. | ||
'''Date Entered''' – Optional. | '''Date Entered''' – <nowiki>[date] </nowiki> Optional. If blank, the gift batch effective date is used. | ||
: '''Validation Rules:''' Must be blank or a valid date in the specified import format. If blank, then the field is populated from the batch effective date. | |||
* [[User:Robertpickett|Robertpickett]] not in Petra. | * [[User:Robertpickett|Robertpickett]] not in Petra. |
Revision as of 08:52, 26 February 2013
NOTE: This is very much a work in progress and needs to be completed and cleaned up before it can be trusted.
The purpose of this entry is to define the Gift Batch file layout and its validation requirements. The validation process should follow the OpenPetra validation framework.
Gift Import Batches consist of a batch header record followed by a collection of gift records. All records are CSV types commonly delimited by semi-colons although commas are acceptable.
Batch Header Record fields
Row Type - “B”
Batch Description – [80ch. maximum] Required. Gift batch description
- Validation Rules: Cannot be empty field. Length must be 1-80 characters.
Bank Account Code – [16ch. maximum] Required. The bank account code for the batch. For a normal gift this should be an active bank account code. For "Gift in Kind" or "Other", it can be any active posting account.
- Validation Rules: Cannot be empty. Must be an existing account in table a_account for the active ledger. Must be a posting account (a_posting_status_1). If gift type(a_gift_type_c) = MFinanceConstants.GIFT_TYPE_GIFT, then must be a bank account (a_account_property = "BANK ACCOUNT"). Generate a warning if the account is not an active account.
- Dwmosman Edit against table ?? (note: for gift in kind, this does not have to be a bank account (i.e., could be any account). I assume that for gift in kind, it does not have to be a bank cost centre either?)
- Pokorra I just checked in Petra 2.x: if it is a "gift in kind", it can be any account from table a_account of the current ledger. perhaps a warning if the account is not active anymore. if it is a normal gift, it is an account that has account property bank account. a_account_property, "BANK ACCOUNT".
- Robertpickett The gift type can also be "Other" and I think that also allows any account, the same as for "Gift in Kind". In all cases only Posting Accounts should be allowed (a_posting_status_l).
Hash Total – [decimal] Optional. If blank or 0, then this hash total will be ignored. If non-zero (plus or minus), then this will be compared against the sum of all gifts in the batch.
- Validation Rules: If non-zero, then the hash total must equal the sum of the gift amounts from the batch.
- Dwmosman Can this be blank on the input record?
- Robertpickett Yes, it can be blank
GL Effective Date – [date] Required. The date must be within the date range of the current and open forward periods.
- Validation Rules: Must be a valid date in the date format specified for the batch. Validate against period dates. Search for TFinancialYear.IsValidPostingPeriod. See GL.Importing.cs for example.
- DwmosmanWhat are the requirements here? Does this date have to be within the date range of the current and following accounting periods? If empty, does it default to today's date? Are there any cross-edits with gift date?
- Pokorra gl effective date has to be in the date range of current and open forwarding periods. there should be existing methods to check that. See csharp\ICT\Petra\Server\lib\MFinance\GL\GL.Importing.cs, search for TFinancialYear.IsValidPostingPeriod. The value cannot be empty.
Currency Code – [16ch.] Must be a valid currency code from the OpenPetra currency table. If the account specified above is a foreign currency account, then this entry must match the currency of the account.
- Validation Rules: The value must exist in table a_currency (a_currency_code_c). If the bank account specified above is a foreign currency account (a_foreign_currency_flag_1), then this field must match the currency of the account (a_foreign_currency_code_c).
- Robertpickett If the Account is a foreign currency account (a_foreign_currency_flag_l) then the currency must match the currency of the account (a_foreign_currency_code_c)
Exchange Rate to Base – [decimal] Cannot be 0. Enter the exchange rate at the time the deposit was made. If the gift currency is the same as the base ledger, then exchange rate must be 1.
- Validation Rules: Cannot be 0. If currency code above equals the currency of the active ledger, then exchange rate must equal 1.
- Dwmosman Cannot be 0. Any other constraints?
- Robertpickett If the currency code matches the base currency of the ledger then the exchange rate must be 1
Bank Cost Centre – [24ch.] The cost centre which is applied to the bank. Must be an active local posting cost centre for the ledger.
- Validation Rules: Must be an existing entry in the cost centre table (a_cost_centre) for the active ledger with a_cost_centre_type_c = "Local" and a_posting_cost_centre_flag_1 = true.
- Dwmosman Must be valid cost centre. Determined how?
- Pokorra cost centre is not related to bank. can be any active costcentre, table a_cost_centre, depending on the ledger number.
- Robertpickett The Cost Centre must also have a_cost_centre_type_c = "Local" and must be a Posting Cost Centre (a_posting_cost_centre_flag_l). In Petra, if it can't find the Cost Centre it trys again by adding a leading 0 (because of Excel's tendency to remove leading zero's when you don't want it to. If it then finds a match it asks the user if they want to use that CC instead (eg. "0300" instead of "300")
- Dwmosman 08:52, 26 February 2013 (UTC) Rob, are there alternatives to this approach? Interactive batch loads across the client/server boundary can be tricky (for example, what if the requestor fails to notice the question dialog and the table is locked for all users for the day?). I will send an email on this
Gift Type – [16ch.] Optional. Must be "Gift", "Gift In Kind" or "Other". If left blank, it defaults to "Gift".
- Validation Rules: If blank, populate the field from MFinanceConstants.GIFT_TYPE_GIFT. If non-blank, must equal one of GIFT_TYPE_GIFT, GIFT_TYPE_GIFT_IN_KIND or GIFT_TYPE_OTHER.
- Dwmosman Must be one of {Gifts, Gifts in kind, Other}. (hard-coded options?)
- Pokorra please use the hard coded constants from Shared/lib/MFinance/Constants.cs, GIFT_TYPE_*
- Dwmosman Could these constants be enumerated or a collection or something? Or are they so static that there will always be these three and I can just hard-code them into my validation routine?
- Dwmosman 08:52, 26 February 2013 (UTC) In the OpenPetra database design it says that this field defaults to "gift". Is this acceptable? Or must the importer always put a value in this field?
Gift Record fields
Note: Gift records have two formats. The normal format is 22 fields. The extended format has greater than 22 fields. Both formats can appear in the same batch. Fields below that are preceded by a ">" are only used in the extended format.
- Pokorra I think that one file will either be in the shorter, or in the extended format, not mixed. I see, the import code checks for every transaction. that is fine, but not required.
- Robertpickett In Petra the extended format is export only (provided to give a convenient csv file for manipulation in Excel), the import doesn't support it. I'm not sure whether it makes sense to import the long format since a number of the fields are calculated fields.
- Dwmosman Should I eliminate the extended format? (Fields preceded by a ">" sign are part of the extended format)
Row Type - “T”
Donor Key – [integer] The partner key of the donor. The donor must be entered as a partner before the gift batch is imported. Note that if the donors partner record has been merged into another partner, then the new partner key must be specified instead of the key of the merged record.
- Validation Rules: Must be a valid key into the partner table, p_partner. However, if the partner status is "MERGED", then the new partner must be used instead. Ideally the system will look up the new new partner and ask the user if they want to use the new partner. If yes, replace the key with the new key. If no, then fail the import.
- Dwmosman Must be a valid donor key from partner(?) table, xxx
- Pokorra table p_partner, no special requirement. usually it is a partner of p_partner_class_c = "FAMILY", but I think that is not a requirement??? Rob?
- Robertpickett The donor can be of any Partner Class. The one complication relates to Merged Partners. If the Partner Status is "MERGED" then the import should look up the Partner that it was merged into (maybe there is already a method for that in OpenPetra). If it can't find that Partner then import should fail, otherwise ask the user if they want to use that Partner instead of the one they entered (if they say no then import fails), otherwise use the new Partner.
Donor Short Name – note: This field is required in the gift row but can be empty and is always ignored by the import process.
- Dwmosman Is there a reason we can't remove this field from the import since its not used?
Method of Giving Code – [16ch.] Select one of the options defined in the Method of Giving table for your installation.
- Validation Rules: Must be an existing "a_method_of_giving_code_c" code in the Method of Giving table, a_method_of_giving.
Method of Payment Code – [16ch.] Select one of the options defined in the Method of Payment table for your installation.
- Validation Rules: Must be an existing "a_method_of_payment_code_c" code in the Method of Payment table, a_method_of_payment.
Reference – [20ch.] Reference number/code for the gift transaction.
- Validation Rules: Cannot be longer than 20 characters.
- Dwmosman Can this field be empty?
Receipt Letter Code – [16ch.] Optional. "<none>" is the same as blank. Otherwise select one of the values from your installations Form Letter Design table.
- Validation Rules: If empty or "<none>", populate the field with "<none>". Otherwise, the entry must be an existing code (p_design_name_c) in the table p_form_letter_design.
- Dwmosman values sourced from ??. Is <none> is a valid value? Can this be blank?
- Pokorra I think it will be blank most of the time, because it will only be filled when the gift batch is posted, and a receipt is printed.
- Robertpickett <none> is valid, as is blank. In Petra, if blank then it gets imported as <none>. Otherwise must exist in p_form_letter_design (field p_design_name_c)
>Receipt Number – [integer] Optional. Gift receipt number. Defaults to 0.
- Validation Rules: Must be blank, 0 or a valid positive integer.
>First Time Gift – [boolean] Optional. Enter "yes" or "no" to indicate whether this is a first time gift for the donor. Defaults to "no".
- Validation Rules: Must be "yes", "no" or blank. If blank, default to "no" (zero).
>Receipt Printed – [bool] Optional. Enter "yes" or "no" to indicate whether the receipt has been printed for this gift. Defaults to "no".
- Validation Rules: Must be "yes", "no" or blank. If blank, default to "no" (zero).
Recipient Key – [integer] Select the key of the recipient from the partner table.
- Validation Rules: Must be a valid key (p_partner_key_n) from the partner table, p_partner. Class (p_partner_key_n) must be "FAMILY" or "UNIT". If Family, tnen special type (p_partner_type) must be "WORKER". Partner status must not be "MERGED".
- Dwmosman Must be (active?) partner from partner table, p_partner: p_partner_key_n
- Pokorra it should actually be a FAMILY partner (p_partner_class_c) that has special type WORKER (p_partner_type), or a UNIT partner. not sure if that should all be validated during the import, because it is quite extensive. we have methods for that during posting of a gift batch.
- Robertpickett Should also not allow Partners with status "MERGED".
Recipient Short Name – ignored by import
- Dwmosman Can I remove this from the import, since its ignored anyway?
>Recipient Ledger Number – [integer] Optional?.
- Dwmosman Equates to field ledger number? Maybe only for unit level recipients? Or for allocating to a family who works multiple fields? If entered, must match recipient field ledger number. On the gift screens this is auto generated from recipient.
- Pokorra it is important for a worker as well, because the money will go to the field where he is working. but the gift screen generates it, and during gift posting it is calculated again, so it is not so important to be validated, I think.
- Robertpickett The Petra import works this out based on the recipient, but I guess the best method for that still has to be worked out for OpenPetra.
- Dwmosman 08:52, 26 February 2013 (UTC) Can I remove this from the import? It sounds like it is recalculated at posting anyway so its basically ignored?
Gift Amount – [decimal] Enter the gift amount.
- Validation Rules: Must be a valid decimal number
- Robertpickett Must be non-zero. Can be positive or negative
- Dwmosman 08:52, 26 February 2013 (UTC) Do I need to edit this for the correct number of decimal places? (some currencies have 0 or 3 decimal places)
- Dwmosman 08:52, 26 February 2013 (UTC) Are there rules about what can be entered here for Gift in Kind or Other?
>Gift Amount International – [decimal]
Confidential Gift Flag – [boolean] Optional. "yes" or "no". Enter yes to indicate a confidential gift. Defaults to "no".
- Validation Rules: Must be "yes", "no" or blank. If blank, default to false.
Motivation Group Code – [16ch.] Select an entry from the installation's Motivation Group table for the active ledger.
- Validation Rules: Must be a valid code (a_motivation_group_code_c) from the Motivation Group table, a_motivation_group, for the active ledger (a_ledger_number_i).
- Dwmosman Optional? Validate against table a_motivation_group: a_ledger_number_i, a_motivation_group_code_c. (does value of a_restricted_l matter?)
- Pokorra we usually always require a valid motivation group and detail. a_restricted_l is just for read access permissions, who is allowed to see that gift. should not take effect here.
Motivation Detail Code– [16ch.] Select an entry from the installations Motivation Detail table for the selected Motivation Group Code for the active ledger
- Validation Rules: Must be a valid code (a_motivation_detail_code_c) from the Motivation Detail table, a_motivation_detail with the active ledger (a_ledger_number_i), the selected group code (a_motivation_group_code_c) and with status = true (a_motivation_status_1).
- Dwmosman Optional? Validate against table a_motivation_detail: a_ledger_number_i, a_motivation_group_code_c, a_motivation_detail_code_c (note: motivation detail already has fields for recipient, receipt, tax deductible, cost centre, etc. Should edit compare these to the corresponding gift fields in some way? Should these be used to populate the gift fields? Also, does the field, a_motivation_status_l, affect whether the motivation code can be used as a valid gift import code?
- Pokorra I think you do not need to copy values from motivation detail record.
- Robertpickett Only Motivation Details with a_motivation_status_l = true should be allowed. There is some code needed here to populate information like Cost Centre, Account, etc which does need to be done but is not a straightforward matter of copying values (it depends on other factors as well). I imagine the code to do that has already been written for the Gift Batch screen. If p_recipient_key_n on a_motivation_detail is populated and doesn't match the recipient key in the import file (and the recipient key is not 0) then the import should fail with an error.
- Dwmosman 08:52, 26 February 2013 (UTC) Actually, if you create the gift detail in OpenPetra, these fields are generated and you cannot manually enter or change them. So I assume, that they must be regenerated at post? (because the values in the DB could have changed between entry and post). So if they are in the import file and they contradict each other? So it seems that a consistent decision should be made as to which fields can be manually controlled (either online or via batch import) and which are to be automatically derived once other fields, like motivational detail, are selected?
Cost Centre Code – [24ch]
- Robertpickett I don't think this should be here? Is it part of the extended format?
- Dwmosman 08:52, 26 February 2013 (UTC) It is in the current OpenPetra code for the normal import format. Its also in the gift detail table in the database. Is this a design question that should be discussed?
Gift Comment One – [160ch.] Optional. Long description of the gift. : '''Validation Rules''' - Check for SQL injection. * ~~~ optional? Max length? *[[User:Pokorra|Pokorra]] yes, is optional. length 160 characters: see http://openpetraorg.sourceforge.net/dbdoc/index.html?table=a_gift_detail&group=gift '''Gift Comment One Type''' – Optional. If used, must contain a valid Gift Comment Type. Cannot be blank if Gift Comment One is used. : '''Validation Rules:''' - blank or a valid GIFT_COMMENT_TYPE. Cannot be blank if Gift Comment One is not blank. * ~~~ are comment types hard-coded or tabled? Can this be blank if comment is not blank? *[[User:Pokorra|Pokorra]] hard coded, see [http://bazaar.launchpad.net/~openpetracore/openpetraorg/trunkhosted/view/head:/csharp/ICT/Petra/Shared/lib/MFinance/Constants.cs Shared/lib/MFinance/Constants.cs], GIFT_COMMENT_TYPE*. that Constant.cs file is actually missing more constants: Donor, Recipient, Office * [[User:Robertpickett|Robertpickett]] Must contain a value if comment is not blank. * ~~~~ As per my earlier question. Could these constants be kept in perhaps a generic that I could ping it with Linq or some such :) * ~~~~ For my own edification, can someone '''Mailing Code''' – <nowiki>[16ch.] Optional. "<none>" is the same as blank. Otherwise select one of the values from your installations Mailing Code table.
- Validation Rules: If empty or "<none>", populate the field with "<none>". Otherwise, the entry must be an existing code (p_mailing_code_c) in the table p_mailing.
- Robertpickett Same rules as for Receipt Letter Code but using p_mailing (p_mailing_code_c) for the lookup
Gift Comment Two - See Gift Comment One
Gift Comment Two Type - See Gift Comment One Type
Gift Comment Three - See Gift Comment One
Gift Comment Three Type - See Gift Comment One type
Tax Deductable [boolean] Optional. if blank, it will be filled from a_tax_deductible_flag_1 from the Motivation Detail.
- Validation Rules - Must be blank, "yes" or "no". If blank, then the field will be populated from the a_tax_deductible_flag_1 from the selected Motivation Detail.
- Robertpickett Boolean. If blank then will be filled from a_tax_deductible_flag_l on a_motivation_detail.
Date Entered – [date] Optional. If blank, the gift batch effective date is used.
- Validation Rules: Must be blank or a valid date in the specified import format. If blank, then the field is populated from the batch effective date.
- Robertpickett not in Petra.