Fix bug whereby sorting by state province gives an error in search builder

Merge pull request #13856 from eileenmcnaughton/test

Test fix

Merge pull request #13860 from alifrumin/patch-2

Fixing formatting of contributors section

Fixing formatting of contributors section

Merge pull request #13858 from mlutfy/dev795

dev/core#795 Fix PHP warning when updating a multiselect country field.

dev/core#811 - Autocomplete only enabled options

Add fieldset and help for activity assignee setting

dev/core#795 Fix PHP warning when updating a multiselect country field.

Minor code cleanups around invoicing assignment

Merge pull request #13843 from eileenmcnaughton/trait_online

Add FrontEndPaymentFormTrait to start to share functionality between Event and Contribution forms

Reset REQUEST, GET & POST in main tearDown

I've been noticing tests failing on lines which indicate they were getting variables from these - e.g

CRM_Member_Form_MembershipTest::testSubmitUpdateMembershipFromPartiallyPaid
CiviCRM_API3_Exception: Expected one Contact but found 0

/home/jenkins/bknix-dfl/build/core-13854-7x4fu/sites/all/modules/civicrm/api/api.php:45
/home/jenkins/bknix-dfl/build/core-13854-7x4fu/sites/all/modules/civicrm/CRM/Contribute/Form/AbstractEditPayment.php:757
/home/jenkins/bknix-dfl/build/core-13854-7x4fu/sites/all/modules/civicrm/CRM/Contribute/Form/AbstractEditPayment.php:256
/home/jenkins/bknix-dfl/build/core-13854-7x4fu/sites/all/modules/civicrm/CRM/Member/Form.php:153
/home/jenkins/bknix-dfl/build/core-13854-7x4fu/sites/all/modules/civicrm/CRM/Member/Form/Membership.php:224
/home/jenkins/bknix-dfl/build/core-13854-7x4fu/sites/all/modules/civicrm/tests/phpunit/CRM/Member/Form/MembershipTest.php:832
/home/jenkins/bknix-dfl/build/core-13854-7x4fu/sites/all/modules/civicrm/tests/phpunit/CRM/Member/Form/MembershipTest.php:895
/home/jenkins/bknix-dfl/build/core-13854-7x4fu/sites/all/modules/civicrm/tests/phpunit/CiviTest/CiviUnitTestCase.php:196
/home/jenkins/bknix-dfl/civicrm-buildkit/extern/phpunit5/phpunit5.phar:598

Merge pull request #13846 from eileenmcnaughton/reformat

[NFC] code formatting only

Merge pull request #13849 from eileenmcnaughton/reformat_2

[NFC] Code reformatting

Merge pull request #13768 from eileenmcnaughton/activity_yay

Activity tab performance fix - switch to faster getActivities & getActivitiesCount

Merge pull request #13259 from agh1/disabled-expired-mem

Disabling or deleting Expired status breaks membership status update

CiviMember: test renaming `expired` doesn't trip up status update

Merge pull request #13545 from pradpnayak/ContactSubType

"Contact Sub Type" dropdown disappears when fields validation occurs

dev/core#641: Implementing Case Activity Assignment Restriction functionality

CiviMember: excluding Expired memberships by name, not label

Display contact sub type always

Merge pull request #13851 from eileenmcnaughton/opt

Code cleanup on membership block loop

Merge pull request #13795 from kewljuice/issue_788

refs issue 788 fix Warning: A non-numeric value encountered in ...

Code cleanup on membership block loop

Merge pull request #13742 from jitendrapurohit/core-771

dev/core#771 - Smart group with uf_group_id does not load contacts wi…

Merge pull request #13508 from spalmstr/core-688

Core-688 - Contacts - New Email Fix Smarty/plugins/function.crmSettin…

Update Cancel Billing & update billing to use status bounce rather than fatal

Per https://lab.civicrm.org/dev/core/issues/560 we have resolved to get rid of fatal & move
to throwing Exceptions.

However, I'm leaning towards making statusBounces the rules for when we just deem a form inaccessible.

In this case we have 3 paralel forms - cancel, update & a similar update but slightly different just cos.

One does a status bounce - 2 spit out fatals when access not available.

Just looking at Cancel these are the places that still have fatal

- The recurring contribution looks to have been cancelled already
- Required information missing

I'm not sure the value of a fatal as opposed to the gentler statusBounces in either of those cases.

Throwing an exception looks very much like a fatal except
1) it doesn't hurt tests or cli tools as much
2) IF display backtrace is enabled THEN the back trace displays.

so, it's generally better BUT I kinda feel like the only time when it's actually better than a status
bounce is when a bug has occurred (eg. an unexpectedly failed api call) - anything we anticipate
seems like a bounce to me....

Merge pull request #13237 from MegaphoneJon/core-571

core#571 - check logged in OR checksum user permissions to edit recur…

Merge pull request #13554 from kewljuice/patch-3

dev/core#706 Edit contribution : wrong decimal separator on total_amount for participant(s)

[NFC] Code reformatting

IDE reformat to fix array() vs []

[NFC] code formatting only

IDE reformatting to fix array formats

Merge pull request #13848 from eileenmcnaughton/report_temp2

Use TempTable methods - contribution detail

Merge pull request #13462 from greenpeace-cee/alter-indexes

dev/core#664 - CRM/Logging - Add indexes when updating log schema

dev/core#789 - Bug fix for event reg error when skipping participants (#13800)

Merge pull request #13847 from eileenmcnaughton/report_temp

Use TempTable methods - bookkeeping report

Merge pull request #13603 from pradpnayak/AdvMailin

dev/core/issues/726, Fixed fatal error when searched using group type

Use TempTable methods.

Use TempTable methods - bookkeeping report

Merge pull request #13845 from seamuslee001/purify_html_tests

Add in tests of purifying HTML output

Merge pull request #13842 from seamuslee001/upgrade_log_package

Remove Log.php require_once statements

Add in tests of purifying HTML output

Fix up test to work with earlier html purfier

Ensure that all original Log Files are removed especially on Joomla

Remove require_once statements

Add FrontEndPaymentFormTrait to start to share functionality between Event and Contribution forms.

I've done one small function extraction in the process

Merge pull request #13807 from eileenmcnaughton/merge_perm

Fix contact.merge api to pass check_permissions parameter through to the deeper layer

Merge pull request #13839 from eileenmcnaughton/invoicing

Minor code cleanup

Merge pull request #13838 from eileenmcnaughton/ids

Minor code cleanup - remove unnecessary ids declaration

Merge pull request #13835 from seamuslee001/upgrade_log_package

Deploy PEAR Log package and upgrade to latest version in the process

Merge pull request #13841 from seamuslee001/dismaker_amavisd

Remove amavisd now that it is removed from the packages repository

Merge pull request #13836 from eileenmcnaughton/session

[NFC] remove unnecessary variable

Remove amavisd now that it is removed from the packages repository

Merge pull request #13832 from eileenmcnaughton/logging

Logging - attempt to fix tests

Merge pull request #13823 from eileenmcnaughton/import_date_test

Import date test

Minor code cleanup

Invoicing code has been poorly integrated into CiviCRM with much copy & paste,
no following of settings spec and an awful lot of outright hacking.

This is a small step towards moving invoice logic to a centralised place -
specifically in this case the unpacking of the non-standard settings that have been
used. Perhaps once using these functions we can standardise them!

Ideally I'd like to see all the invoicing assigns in a function on this
class, better yet once which mimics the pre or buildForm hook & even
better one day to grow up & be a real hook

Minor code cleanup - remove unnecessary ids declaration

 is not used  is only used to set ['id'] so this can be simplified

[NFC] remove unnecessary variable

Deploy PEAR Log package and upgrade to latest version in the process

Merge pull request #13793 from mattwire/case_links_refactor

REF Extract case action links into a separate function to facilitate refactoring

Merge pull request #13834 from civicrm/5.12

5.12

Merge pull request #13825 from mlutfy/dev801-5-12

dev/core#801 Fix from email on PDF Letters, such as Thank You Letters.

Merge pull request #13833 from civicrm/5.12

5.12 to master

Merge pull request #13820 from colemanw/dev/core#790

dev/core#790 - Exclue menubar on frontend pages

Merge pull request #13826 from totten/5.12-assettest-geturl

(NFC, infra/ops#878) AssetBuilderTest - Relax getUrl() assertion for WP compat

Attempt to fix test with 1 second delay

(note in 'real life' the events are not generally so close together

Add more data when test fails

Merge pull request #13830 from mattwire/whitespace_membershipblock

NFC Whitespace cleanup MembershipBlock.tpl

Whitespace on MembershipBlock.tpl

Merge pull request #13779 from eileenmcnaughton/test_recur

Display test contributions when viewing contributions related to a test recurring contribution

Merge pull request #13824 from colemanw/loadScriptCode

Make cacheCode optional in CRM.loadScript

--amend

Change-Id: Icea6f894d7a9e4656beae3cdb4cb5e44842b59b2

Activity tab performance fix - switch to faster getActivities & getActivitiesCount

The getActivitiesCount & getActivities functions are faster & call permission hooks
but we weren't able to switch to them until we resolved some performance issues
(done) and resolved acl inconsistencies (resolved in 5.12) -we can do this now.

From a performance POV the difference is tab-crashes vs tab resolves quickly
on contacts with > 10k activities

Merge pull request #13829 from eileenmcnaughton/revert

[nfc] Attempt to improve false negatives on Logging test

Attempt to improve false negatives on Logging test

It might be significant that the one I added sleep(1) to before wasn't a fail in this one...

https://test.civicrm.org/job/CiviCRM-Core-PR/25137/

Display test contributions when viewing contributions related to a test recurring contribution

Merge pull request #13828 from civicrm/5.12

5.12 to master

Merge pull request #13796 from eileenmcnaughton/exceptions

dev/core#659 Catch payment processor exceptions, log, hide, do not return 500 error

Merge pull request #13797 from eileenmcnaughton/matt_test

Further cleanup on getRelatedMemberships - just get them with the api

(NFC, infra/ops#878) AssetBuilderTest - Relax getUrl() assertion for WP compat

The test ensures that AssetBuilder produces plausible URLs when caching is disabled.

Generated URLs look different in the default `drupal-clean` and `wp-demo` configurations, e.g.

* D7 (drupal-clean): `http://dmaster.bknix:8001/civicrm/asset/builder?an=square.js&ap=...`
* WP (wp-demo): `http://wpmaster.bknix:8001/?page=CiviCRM&q=civicrm%2Fasset%2Fbuilder&an=square.js&ap=...

The assertion in `AssetBuilderTest` was coded in a way that fails on working URLs.

Before
------

The test fails on WP.

After
-----

The test passes on WP.

dev/core#801 Fix from email on PDF Letters, such as Thank You Letters.

Fix error message handling while we are at it

Turns out that instead of translating we are passing the error message to a truly silly function

Add unit test to date parsing on import

This is primarily as a demo of how to test this class.

Basically extract the parts that we want to check the formatting of into manageable
functions. I made the name on the second one pretty generic so slowly all formatting can
be moved into it, with testing.

This would include moving the various bits of work done in
_civicrm_api3_deprecated_formatted_param
into this function - that code really does just belong on this class as it
is not called from anywhere else.

It also does the checking, but not the formatting, for these date fields - sigh

I would have liked to have made the method protected but the mucking around that required
in the test classes didn't seem to justify it

Catch payment processor exceptions, log, hide, do not return 500 error

Add unit test on validate

Support calling ContributionPage.validate in POST context.

Because the validation data is buried deep within the form we need to load the form
in order to validate - but currently the validKey check borks in post requests.

Note the validity of credit card details is not checked when submitting this form,
other than possibly the luhn validation.

 A likely use case for the validation is when the is being submitted and some handling is
 to be done before processing but the validity of input needs to be checked first.

 For example Paypal Checkout will replace the confirm button with it's own but we are able to validate
before paypal launches it's modal. In this case the  is post but we need validation to succeed.

Make cacheCode optional in CRM.loadScript

Merge pull request #13822 from seamuslee001/update_zeta_composer_lock

Update lockfile to take into account of civicrm/zetacomponents-mail#4…

Merge pull request #13816 from MegaphoneJon/core-798

core#798 - fix rendering of public-facing prefix/suffix select2

Update lockfile to take into account of civicrm/zetacomponents-mail#4 being merged

dev/core#790 - Exclue menubar on frontend pages

Merge pull request #13778 from eileenmcnaughton/paymetn_next

[REF] small cleanups on payment.create flow.

Further cleanup on getRelatedMemberships - just get them with the api

Per work by Matt it makes sense to just retrieve as an api rather than
do weird object wrangling. This is intended to not change the logic about what is fetched

Merge pull request #13799 from mfb/zetacomponents

Upgrade zetacomponents/base and zetacomponents/mail

Merge pull request #13802 from mattwire/membership_address

Membership form address fields for payment processors

Merge pull request #13703 from mfb/form-temp-tables

Use temp table methods; provide TempTable::getCreateSql() method for debugging

Merge pull request #13818 from civicrm/5.12

5.12 to master

Merge pull request #13810 from totten/master-wp-stdrr

(ops#878) phpunit.xml.dist - Prevent conflict in WordPress E2E with "stderr=true"

Merge pull request #13811 from totten/master-wp-loadboot

(ops#878) Enforce common signature for CRM_Utils_System::loadBootStrap(). Fix WP E2E error.

Merge pull request #13801 from eileenmcnaughton/5.12

Fix unrelased regression where activity date relative filters are ingnored in the rc

Merge pull request #13383 from seamuslee001/report_cleanup_mailing_summary

(dev/core#491) Standardise the adding of campaign fields on the maili…

Merge pull request #13814 from mattwire/testentity

Add a class to handle test entities consistently

Pass 'check permissions' flag through to merge function

The inner function will only delete the merged contact if checkPermissions is false or
the logged in user has permission to delete. Our api model expects check_permissions to be
respected at the BAO level