Merge pull request #10193 from eileenmcnaughton/pan_tunc

CRM-20264 handling for storing card_type_id

Merge pull request #10194 from eileenmcnaughton/master

CRM-20264 add test for submitting pan_truncation & card_type_id via t…

CRM-20264 add test for submitting pan_truncation & card_type_id via the api

CRM-20264 handling for storing card_type_id

Merge pull request #10189 from eileenmcnaughton/pan_tunc

CRM-20264 make pan_truncation save on contribution form, additional payment, event

Merge pull request #10191 from eileenmcnaughton/report

CRM-20451 Do not assume all BAOs have exportableFields method in Core_…

CRM-20451 Do no assume all BAOs have exportableFields method in Core_report

CRM-20264 make pan_truncation save on event form.

note that this PR should be testable & mergeable as making 'no change' without the changes to Core_Payment_Manual as it is primarily code consolidation

Merge pull request #10182 from totten/master-listall-checksum

GenCode - Reduce merge-conflicts on AllCoreTables

Merge pull request #10188 from colemanw/CRM-20105

CRM-20105 - Add case restore and merge api actions

Merge pull request #9544 from colemanw/CRM-19773

CRM-19773 - Call hook_civicrm_selectWhereClause when checking event permissions

CRM-20105 - Add merge function to case api

Merge pull request #10087 from lcdservices/CRM-20368

CRM-20368

CRM-19773 - Call hook_civicrm_selectWhereClause when checking event permissions

Fully remove deprecated Constants

CRM-20392 enable remaining unit tests added by Pradeep

I did alter the fix side on this a little more - notably to use the shared
functions beginPostProcess and assignPaymentRelatedVariables so this will require
independent review

CRM-20264, Added Card type and Pan Truncation form fields on Payment form

I deliberately excluded card_type because I think we need to review the appropriateness of
the field first. My issue is that the wrangling wound up really wierd & it was apparent the issue
is that card_type is 'Visa' whereas card_type_id is 1 - we should either rename the field
or store the word.

Merge pull request #10187 from eileenmcnaughton/prad

CRM-20392 add unit test and fix for incorrect total used in line item…

Merge pull request #10186 from eileenmcnaughton/mandatory_created_id

CRM-20316 the mailing should still create without created_id (not man…

CRM-20392 add unit test and fix for incorrect total used in line item creation code.

Eileen's note - this is code done by Pradeep that I am reviewing - I am committing
and merging the portion I have reviewed so far. This fixes a bug replicable in
the test Pradeep wrote, and some minor obvious cleanups such as removing unused parameters.

I've commented out some tests to complete this portion & there is still code of Pradeeps
not yet reviewed.

CRM-20105 - Add restore function to case api

CRM-20316 the mailing should still create without created_id (not mandatory).

CRM-20316 the mailing should still create without created_id (not mandatory).

Merge pull request #10185 from eileenmcnaughton/master

[NFC] Fix invalid php in test-generated example

Fix invalid php in test-generated example

Merge pull request #10171 from seamuslee001/CRM-20427

CRM-20427 Fix issue with the regex in I18n query rewriting and add tests

CRM-20427 Skip tests that are currently failing as edge cases and not blockers to current issue

Merge pull request #10184 from colemanw/CRM-17711

CRM-17711 - Remove "-unstable" from backdrop filename

CRM-17711 - Remove "-unstable" from backdrop filename

GenCode - Reduce merge-conflicts on AllCoreTables

The previous checksum was based on the full schema spec.  This is
technically safe, but many irrelevant schema changes (such as editing field
comments) will ding the checksum.

The main goal here is to avoid `$template->run()` (line ~44) and its
code-beautifier (which is expensive/slow).  This revised algorithm for
`needsUpdate()` should:

 * Still be "fast enough" (given that it's a singular file built from a simple template)
 * Still be "accurate enough" (given that `getRaw()` is a fair predictor of `run()`)
 * Be less prone to merge-conflics (given that it doesn't use the checksum)

Merge pull request #10175 from TobiasLounsbury/CRM-20438-fix-panels

Added object type selector to panel rule to prevent broad disabling o…

Merge pull request #10179 from yashodha/CRM-20417

CRM-20417 - translate strings

Merge pull request #10178 from JMAConsulting/CRM-20439

CRM-20439: Rename card_type to card_type_id of civicrm_financial_trxn table (IIDA-126)

Merge pull request #10155 from totten/master-pdf-save2

CRM-20420 - Add option to save generated PDF/doc files

CRM-20417 - translate strings

Additional fixes in report and Search form

Rename card_type to card_type_id of civicrm_financial_trxn table (IIDA-126)

Merge pull request #10177 from eileenmcnaughton/master

Fix visibility typo

Fix visibility typo

Merge pull request #10174 from civicrm/colemanw-contributing

Document PR scope & size

Minor alter to regex

CRM-20427 add more tests in

Added object type selector to panel rule to prevent broad disabling of all objects with class panel on civicrm pages.

Update CONTRIBUTING.md

CRM-20427 Fix regex issue with I18n query rewrite

Add tests of I18n rewiteQuery to test regex

Merge pull request #10109 from eileenmcnaughton/ayd

CRM-20312 index reconciliation

CRM-20312 - Add a few more action buttons to system status screen

CRM-20312 - Perform actions from within system status screen

CRM-20312 add a little more detail as to how to update indexes

CRM-20312 add api wrapper for updating indexes

Merge pull request #10170 from seamuslee001/CRM-20436

CRM-20436 Make method updateKenyanProvinces Static

CRM-20436 Make method updateKenyanProvinces Static

Merge pull request #10169 from seamuslee001/new_indexes

CRM-20313,CRM-19357,CRM-20204,CRM-19383

CRM-20313,CRM-19357: Add index to civicrm_activity.status_id and unique index to civicrm_entity_financial_account table

CRM-20204,CRM-19383 Add in Indexes to civicrm_activity.activity_date_time and civicrm_event_queue.hash

Merge pull request #10166 from aydun/CRM-19464-upgrade

CRM-19464 sql upgrade

Merge pull request #10163 from JohnFF/bugfix/fix_static_warning

Fixed static call of non static method.

Merge pull request #10150 from eileenmcnaughton/friend

CRM-20328 remove call to duplicated code sequence.

Merge pull request #9857 from aydun/CRM-19464

CRM-19464  Make 'Supplemental Address 3' usable

Localize description

Merge pull request #10160 from JohnFF/bugfix/silent_sql_file_error

Make sourceSQLFile throw an error when it can't find a file.

Merge pull request #10162 from yashodha/CRM-20429

CRM-20429 - Deleting an activity with target contacts > 50 results in…

CRM-19464 sql upgrade

CRM-19464 Enable use of 'Supplemental Address 3'

In some situations, the address format is too limited.  The 'Supplemental Address 3'
already exists in the schema but was not exposed to the GUI.

This touches many files but the changes are fairly simple.  Wherever 'Supplemental Address 2'
existed, there is now a 'Supplemental Address 3' as well.

Fixed static call of non static method. Assume the method was meant to be static as there's nothing in there requiring it to be dynamic.

CRM-20429 - Deleting an activity with target contacts > 50 results in error

Merge pull request #10148 from yashodha/CRM-20417

CRM-20417 - translate strings

Make sourceSQLFile throw an error when it can't find a file - previously it just failed silently.

Merge pull request #10112 from JMAConsulting/CRM-20384

CRM-20384, Added Contribution ID column and Filter to Book Keeping report

CRM-20420 - PDFLetterCommon::createActivities() - Combine calls

Combine the calls to CRM_Activity_BAO_Activity::create() and
CRM_Activity_BAO_ActivityContact::create() into one call.

Remove unnecessary `$activityContacts`.

CRM-20384, added function to get Organization Name associated with Financial Account.

----------------------------------------
* CRM-20384: Bookkeeping report add contrib ID, filter by financial account owner (IIDA-85)
  https://issues.civicrm.org/jira/browse/CRM-20384

CRM-20384, Added Contribution ID column and Filter
Added option to display and Filter by Financial Account Owner

----------------------------------------
* CRM-20384: bookkeeping transaction report
  https://issues.civicrm.org/jira/browse/CRM-20384

CRM-20420 - Extract utility class, CRM_Utils_ConsoleTee

CRM-20420 - PDFLetterCommon::postProcess - Improve precondition check

It's not valid to apply the policy `combined-attached` unless you have
exactly 1 activity.  The previous code would silently degrade if the
precondition wasn't met. The revision makes the failure clear.

CRM-20420 - Update sundry comments/docblocks

CRM-20420 - PDFLetterCommon::getMimeType() - Fix typo

CRM-20420 - Core.settings.php - Fix copy/pasted version#

The setting was written by copying and editing a similar one, but one of the
fields (`add => 4.3`) was stale.

CRM-20420 - PDFLetterCommon - Record attachments (in mode "combined-attached")

If the setting `recordGeneratedLetters` is `combined-attached`, then we
should capture the content of the generated file and save it to an
attachment.

Note: There are several different ways generating files (eg HTML=>PDF,
HTML=>docx/ODT, or docx/ODT template file).  Ideally, these would support
common set of options for directing output (to a file, string, or console).
In fact, some do, and all the underlying libraries do...  but the
middle-logic only handles the console consistently.  We use output buffering
(ob_start()) to capture the content and save it to a file.

CRM-20420 - CRM_Utils_PDF_Document::printDocuments - Normalize $fileName

Normalize semantics of `$fileName` to align with
`CRM_Utils_PDF_Document::html2doc` and `CRM_Utils_PDF_Utils::html2pdf`.

Merge pull request #10144 from eileenmcnaughton/iida8

CRM-20413 fix to setting wrong payment_instrument_id, by passing resp…

Merge pull request #10127 from WeMoveEU/CRM-20389

CRM-20389 ignore mailing_type when cloning

CRM-20420 - Add docblocks

CRM-20413 fix to setting wrong payment_instrument_id, by passing responsibility to the processor.

This also reflects work towards allowing payment_instrument specific fields on the payment form - ie. check, credit_card_type, pan_truncation.

This idea is to finish the code allowing pay-later to be the manual payment processor indexed with a 0 on all payment forms, and have the
form render with appropriate fields. Note that I got this working on multiple forms, this commit reflects the smallest piece I could
break off that is complete within itself, and resolves an issue

I tested that it was still possible to enter checks on the offline contribution as well as membership & additional payment forms, and the
check number still shows up.

I also tested front end contribution credit card & pay later & front end event forms

CRM-20359 Extended the use of contribution tokens to include custom fields using API (#10082)

* CRM-20359 Extended the use of contribution tokens to include custom fields using API

----------------------------------------
* CRM-20359: Support contribution.custom_nn tokens for Thank You letters - print or email
  https://issues.civicrm.org/jira/browse/CRM-20359

* extended testBuildContributionArray unit test

* Fix missing comma

CRM-20420 - Display `recordGeneratedLetters` setting in GUI

CRM-20420 - Generate combined or separated activities, per recordGeneratedLetters

When a user uses "Print/Merge Document" to generate a PDF or Word doc,
they may designate multiple contacts to include in the PDF/Doc. By
setting `recordGeneratedLetters`, they can specify whether to:

 * Create one activity for each contact. (This is the traditional behavior,
   but when you retain the fully-generated PDF documents, it wouldn't
   be a good idea to copy the PDF to all those activities.)
 * Create a single activity associated with all the contacts.

CRM-20420 - Define setting "recordGeneratedLetters"

CRM-20420 - PDFLetterCommon::createActivities() - Simplify. Address comment.

The comment asks why the code appears to use of `$contactIds` and
`$form->contactIds` interchangeably.  If you grep for calls to
`createActivities()`, all 3-ish callpaths do set `$contactIds` based on
`$form->contactIds` -- so they are the same.

```
// Look for calls to createActivities()
$ grep -r ::createActivities CRM/ Civi/ api/
CRM/Contact/Form/Task/PDFLetterCommon.php:      self::createActivities($form, $html_message, $form->_contactIds);
CRM/Member/Form/Task/PDFLetterCommon.php:    self::createActivities($form, $html_message, $contactIDs);
CRM/Contribute/Form/Task/PDFLetterCommon.php:    self::createActivities($form, $html_message, $form->_contactIds);

// Look for calls to createActivities via CRM/Member/Form/Task/PDFLetterCommon

$ rgrep -r postProcessMembers\( CRM/ Civi/ api/
CRM/Member/Form/Task/PDFLetter.php:    CRM_Member_Form_Task_PDFLetterCommon::postProcessMembers(
CRM/Member/Form/Task/PDFLetterCommon.php:  public static function postProcessMembers(&$form, $membershipIDs, $skipOnHold, $skipDeceased, $contactIDs) {
```

Merge pull request #10092 from jitendrapurohit/CRM-20339

CRM-20339: Fix custom data edit on Relationship

CRM-20328 remove call to duplicated code sequence.

On looking  is actually getting an ID so I renamed it

Merge pull request #10143 from eileenmcnaughton/iida7

Towards CRM-20392 (IIDA-108) extract submit function

CRM-20417 - translate strings

minor fix

Remove e-notice causing declaration.

Since this is a deprecated name which doesn't work I have just removed it

Towards CRM-20392 form cleanup, shared setting of ->mode

Towards CRM-20392 (IIDA-108) extract submit function

Towards CRM-20392 enotice fixes

Merge pull request #10130 from JMAConsulting/codeCleanup

Used Api to get financial account for an entity defined in civicrm_en…

Merge pull request #10081 from jitendrapurohit/CRM-20361

CRM-20361: Populate tokens when Scheduled Reminders are set to Record Activity for Automated Email

Used Api to get financial account for an entity defined in civicrm_entity_financial_account table