Merge pull request #10220 from civicrm/4.7.19-rc

4.7.19 rc

Merge pull request #10222 from KarinG/CRM-20450

CRM-20450: Fix invoice math for partial payments.

Merge pull request #10221 from JohnFF/bugfix/minor_template_errors

Fixed minor template errors.

Merge pull request #10219 from eileenmcnaughton/token

CRM-20449: enotices from 'send thankyou letter' on contribution search

Upgrade the message template in the database.

Fixed minor template errors.

Merge pull request #10217 from fuzionnz/CRM-20466-php7.1_only_vars_by_ref

Address fatal errors on PHP7.1 - only vars can be passed by reference.

Merge pull request #9471 from adixon/CRM-19697

CRM-19697: No recycling the static _links object when you change it!

CRM-20449: enotices from 'send thankyou letter' on contribution search

Merge pull request #10218 from JMAConsulting/CRM-20465

CRM-20465, changed pan_truncation field type to VARCHAR

Merge pull request #10209 from jmcclelland/CRM-20463

CRM-20463 - simplify parsing of doc URLs and ensure all work.

Address fatal errors on PHP7.1 - only vars can be passed by reference.

It is not clear why we clone these DAO/BAO objects, but we cannot do it inside the function call. It is likely this fix can be improved.

CRM-20466

Merge pull request #10154 from eileenmcnaughton/timestamp

CRM-9683 implement timezone support for CiviMail.

Merge pull request #10216 from JohnFF/refactor/CRM-20428_sourceSQLFile_rework

CRM-20428: refactored sourceSQLFile

Merge pull request #10168 from GinkgoFJG/CRM-20435

CRM-20435: Replace SQL-based activityContact creation with DAO-based approach.

Merge pull request #10200 from eileenmcnaughton/deprecate

CRM-20459 replace all instances where CRM_Core_OptionGroup::getValue …

CRM-9683 regenerate DAOs

Merge pull request #10215 from civicrm/4.7.19-rc

4.7.19 rc

CRM-9683 implement timezone support for CiviMail

This will affect new installs only per discussion on the ticket. Motivation dependent, we could add a method to convert on existing installs.

Note some fields, like event_start_date might have a case for being in a non-timezone aware format

Merge pull request #10201 from eileenmcnaughton/deprecate2

CRM-20459 replace all instances where CRM_Core_OptionGroup::getValue …

Merge pull request #10203 from eileenmcnaughton/deprecate3

CRM-20459 replace last instances where CRM_Core_OptionGroup::getValue…

Merge pull request #10213 from seamuslee001/master

(NFC) CRM-20308 Fix Test due to not truncating civicrm_email table

Merge pull request #10212 from seamuslee001/CRM-20441

CRM-20441 Fix issue where multiple activity Ids were not supported wh…

Refactored CRM-20428. Refactored sourceSQLFile by creating a separate function to run the SQL piece. This means that sourceSQLFile no longer has an 'isQueryString' variable, which was used to pass an SQL query in the filename parameter.

Merge pull request #10210 from JMAConsulting/CRM-20464

CRM-20464, fixed smarty error

CRM-20308 Fix Test due to not truncating civicrm_email table

CRM-20441 Fix issue where multiple activity Ids were not supported when check permissions was used

CRM-20465, changed pan_truncation field type to VARCHAR

----------------------------------------
* CRM-20465: pan truncation doesn't respect values preceding with 0
  https://issues.civicrm.org/jira/browse/CRM-20465

Set version to 4.7.20

Merge pull request #10199 from eileenmcnaughton/comment

[NFC] Remove comment as this has been fixed

CRM-20464, fixed smarty error

----------------------------------------
* CRM-20464: Smarty error when receipt is send for recording payment
  https://issues.civicrm.org/jira/browse/CRM-20464

CRM-20463 - simplify parsing of doc URLs and ensure all work.

Merge pull request #10198 from eileenmcnaughton/iida3

[NFC] Further test improvement

CRM-20459 replace last instances where CRM_Core_OptionGroup::getValue is called

CRM-20459 replace all instances where CRM_Core_OptionGroup::getValue is called to get contribution_status_id

CRM-20459 replace all instances where CRM_Core_OptionGroup::getValue is called on activities.

I limited this to activities as there are only 2 separate values & I thought it would keep it readable.

Remove comment as this has been fixed

Further test improvement

Fix syntax errors.

Fix typo.

Fix invoice math for partial payments.

Merge pull request #10157 from eileenmcnaughton/iida3

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

Merge pull request #10099 from eileenmcnaughton/anet

CRM-19597 allow receive_date to be passed in for A.net, plus some code simplification

Merge pull request #10030 from eileenmcnaughton/test

CRM-20008 fix pending flow.

Merge pull request #10161 from totten/master-hook-inspector

CiviEventInspector - Add util for displaying hook metadata

CRM-19597 move function only used by Authorize.net to a.net for clarity.

We could do with a better solution around this, but it needs to be around the completetransaction api

CRM-19597 allow receive_date to be passed in for A.net

Merge pull request #10125 from jitendrapurohit/eventtest

CRM-19773: unit test to check getLocationEvent invokes selectWhereClause() hook

Merge pull request #10158 from JMAConsulting/CRM-20308

CRM-20308: Activity copy is always sent FROM logged in user's email ID

Merge pull request #10176 from JMAConsulting/codeCleanup

Code cleanup

CRM-20264 support card_type_id and pan_truncation in all backoffice form

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

Used generalized function, removed unused function

removed cruft code

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

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