Merge pull request #15886 from seamuslee001/nfc_composer_lock_update

[NFC] Update composer lockfile to remove warning

Merge pull request #15864 from mfb/string-type-declaration

Add string type declarations to global API functions.

[NFC] Update composer lockfile to remove warning

Various tests assume civicrm_api() $entity param is nullable.

Add string type declarations to global API functions.

Merge pull request #15880 from mlutfy/fixUserDashboardEvent

Remove nbsp from UserDashboard Event section

Merge pull request #15817 from colemanw/Fix

Fix api explorer formatting IS NULL clauses

Merge pull request #15881 from civicrm/5.20

5.20

Merge pull request #15879 from alifrumin/rn5.20

First pass at sorting and annotating 5.20 release notes

Remove nbsp from UserDashboard Event section

Merge pull request #15878 from civicrm/5.20

5.20

release notes 5.20 - first pass at editing, sorting and annotating

Merge pull request #15846 from greenpeace-cee/fix-cancel-date

Fix contribution cancel_date not being loaded

dev/core#1391 - Fix contribution cancel_date not being loaded

This fixes an issue where the contribution cancel_date is not loaded
in the contribution search, causing cancelled contributions not to be
greyed out and the cancel date not to be shown.

Merge pull request #15800 from eileenmcnaughton/anet_valid

Remove unused  parameter recur from CompleteOrder signature

Merge pull request #15871 from eileenmcnaughton/init_fatal

[REF] Move CRM_Event_Form_EventFees::buildQuickForm to Participant_Form class

Merge pull request #15866 from eileenmcnaughton/test_dumb

[NFC] [Test-only] Remove partially paid from statuses tested for RenewContribution

Merge pull request #15870 from eileenmcnaughton/init_set

[REF] Minor simplifcation on initSet

Merge pull request #15854 from eileenmcnaughton/test_fix

[NFC] Test fix - incorrect records created due to test set up using PartiallyPaid when no payment is made

Merge pull request #15869 from eileenmcnaughton/add_pay

[REF] remove unused lines of code

Merge pull request #15826 from seamuslee001/dev_core_183_dedupe

dev/core#183 CRM/Dedupe/BAO/QueryBuilder/IndividualUnsupervised.php report to using CRM_Utils…

Merge pull request #15853 from demeritcowboy/civicasetestcase-duplicate-option-values

[TEST] dev/core#1394 - Avoid duplicate activity types in tests that use CiviCaseTestCase

Merge pull request #15849 from seamuslee001/dev_core_183_dedupe_test

[NFC] Add in unit test of code being altered by #15826

Merge pull request #15874 from seamuslee001/dev_core_183_env_check

dev/core#183 Use standard temporary table name format when creating t…

Add in unit test of code being altered by #15826

Merge pull request #15875 from civicrm/5.20

5.20

Merge pull request #15872 from eileenmcnaughton/sybunt

Fix Sybunt to select mark rows as selected

dev/core#183 Use standard temporary table name format when creating temporary utf8mb4 temporary table

Merge pull request #15873 from eileenmcnaughton/cont_ag

dev/core#1377 Fix bug where search action doesn't work.

Merge pull request #15865 from eileenmcnaughton/query

Fix incorrect reference on extracted function

dev/core#1377 Fix bug where search action doesn't work.

Fixes a regression where the search actions are not working on the contributionAggregate custom search

Merge pull request #15867 from eileenmcnaughton/test_dumber

[NFC] fix test to correctly set up partially paid contribution

Fix Sybunt to select mark rows as selected

Merge pull request #15820 from seamuslee001/dev_core_183_custom_contribsybnt

dev/core#183 Finish converting contrib sybnt custom search to using t…

Merge pull request #15868 from colemanw/sortable

Upgrade ui-sortable angular library to v0.19.0

[REF] Minor simplifcation on initSet

Instead of passing the information to calculate the priceSetID into this function it's more helpful
to calculate it first & pass it in - this determination makes more sense on the calling form

[REF] Move CRM_Event_Form_EventFees::buildQuickForm to Participant_Form class

It turns out this function is only ever used from the ParticipantForm class so moving it back there simplifies
that for people looking at the code

[REF] remove unused lines of code

On checking this out this variable is never used ( is)

Merge pull request #15815 from artfulrobot/issue-1108-fix-unsubscribe

dev/core#1108 Fix unsubscribe bug

Upgrade ui-sortable angular library to v0.19.0

[NFC] fix test to correctly set up partially paid contribution

Remove partially paid from statuses tested for RenewContribution

We want to deprecate this as it doesn't work properly. In this case there is no need to test
it as it's not really valid & hopefully soon it would trigger a deprecation notice

Test that will work when we can deprecate https://github.com/civicrm/civicrm-core/pull/15855

Fix incorrect reference on extracted function

Per https://github.com/civicrm/civicrm-core/pull/15803#pullrequestreview-317543503

Merge pull request #13232 from JO0st/core-574

(dev/core#574) Prevent memory exhaustion when generating large PDFs

Merge pull request #15848 from colemanw/dateIcons

Improve datePicker placeholder/icons

Merge pull request #15818 from colemanw/fields

Field metadata cleanup: Contact & Activity & Custom

Improve datePicker placeholder/icons

Merge pull request #15794 from KarinG/master

Implement drupal_get_user_timezone()

Merge pull request #15837 from totten/master-prtmpl

(NFC) PULL_REQUEST_TEMPLATE.md - Improve suggestions for under-the-hood/non-UI changes

Merge pull request #15840 from yashodha/participant_edit

dev/core#1065 Change verbiage in accordance with context

Avoid duplicate activity types

Merge pull request #15852 from mfb/api-type-declaration

Add $params type declaration to global API functions.

Merge pull request #15858 from yashodha/translation

translate strings

Remove obsolete type check tests, preserving one to ensure TypeError is thrown.

Unsubscribe bug: Fix gitlab issue 1108 (and clean up old db code)

Unsubscribe bug: Add failing test for Gitlab Issue 1108

translate strings

[NFC] Test fix - incorrect records created due to test set up using PartiallyPaid when no payment is made

Merge pull request #15828 from eileenmcnaughton/dedupe2

[REF] minor refactor - move handling of 'skipMerge' to the skipMerge function

Merge pull request #15813 from eileenmcnaughton/fee

Fix CRM/Event/BAO/AdditionalPaymentTest.php to use Order.create

[REF] minor refactor - move handling of 'skipMerge' to the skipMerge function

Merge pull request #15851 from demeritcowboy/document-customdata

[NFC] dev/core#1352 - Add comments why there's two custom blocks on new case form

Merge pull request #15830 from eileenmcnaughton/dedupe4

[REF] refactor on nasty Dedupe function

Add $params type declaration to global API functions.

Merge pull request #15810 from eileenmcnaughton/mem_fix

[REF] calculate 'amount' on ContributionPage in a shared way in one scenario

document why there's two custom blocks

Merge pull request #15850 from civicrm/5.20

5.20 to master

Merge pull request #15844 from mlutfy/fixMsgTpl

marketing/civicrm-website#163 Mitigate potential upgrade errors on missing msg_templates

Merge pull request #15838 from demeritcowboy/getcasereport-split

[REF] XMLProcessor/Report - Extract method to make it more testable

Merge pull request #15825 from seamuslee001/dev_core_183_logging

dev/core#183 Convert Logging report summary report to using CRM_Utils…

Merge pull request #15649 from JMAConsulting/core-1346

core#1346: Tagsets should display vertically in activity/new case forms

Merge pull request #15843 from totten/master-simplehead

(NFC) (dev/core#878) Simplify standard header. Remove year+version.

Merge pull request #15839 from demeritcowboy/should-it-be-static

CiviCase - Make static function `allActivityTypes()` static

Merge pull request #15771 from eileenmcnaughton/partially

dev/financial#100 Remove 'partially paid' as a contribution status option for 'record p…ayment'

fixup templates/CRM/Admin/Form/Setting/Smtp.hlp

(NFC) (dev/core#878) Simplify copyright header (CRM/*)

(NFC) (dev/core#878) Simplify copyright header (templates/*)

(NFC) (dev/core#878) Simplify copyright header (misc)

(NFC) (dev/core#878) Simplify copyright header (api/*)

(NFC) (dev/core#878) Simplify copyright header (tests/*)

(NFC) (dev/core#878) Simplify copyright header (Civi/*)

(NFC) (dev/core#878) Simplify copyright header template

https://lab.civicrm.org/dev/core/issues/878

(NFC) (dev/core#878) Simplify '@copyright' annotation

marketing/civicrm-website#163 Mitigate potential upgrade errors on missing msg_templates

dev/core#183 Finish converting contrib sybnt custom search to using tem table class

make static function static

(dev/core#1065) change verbiage in accordance with context

pull out to make more testable

(NFC) PULL_REQUEST_TEMPLATE.md - Improve suggestions for under-the-hood/non-UI changes

Overview
----------------------------------------

The "pull-request template" is presented to a developer when submitting a
proposed change; it prompts them to fill in additional information.  This
revision aims to make the prompts more useful in the common case where change
does *not* involve the user-interface.

Before
----------------------------------------

The `PULL_REQUEST_TEMPLATE.md` suggests that the "Before" and "After" should
be demonstrated with screenshots.

This is wonderful for UI-oriented changes.

However, if a change is focused under-the-hood (APIs, data-structures,
class-hierarchies, refactorings, etc) - then the prompt is kind of
meaningless/non-applicable.  A meaningless prompt understandably elicits
weak responses like "*Before: Some incorrect metadata*" or "*After: It's
less confusing*".

After
----------------------------------------

The `PULL_REQUEST_TEMPLATE.md` still recommends using concrete example in
"Before"/"After", but it's more open about what that means.

For a UI-oriented change, it means screenshots.  For an
API/technical-service change, it means code-snippets.

Merge pull request #15833 from yashodha/participant_edit

(dev/core#1065) Contact missing in membership renewal form

Merge pull request #15835 from demeritcowboy/unused-reporturl

[NFC] Remove unused variable not used since v4.4

Add help_pre and help_post to api4 fieldSpec

unused variable since v4.5

Merge pull request #15756 from eileenmcnaughton/qfbug

[REF] very minor tidy up

(dev/core#1065) Contact missing in membership renewal form

[REF] calculate 'amount' on ContributionPage in a shared way in one scenario

I have discovered a lot of tests are creating invalid contributions - https://github.com/civicrm/civicrm-core/pull/15706

So far the issues have been in the test + us permitting something that doesn't work on the form - ie https://github.com/civicrm/civicrm-core/pull/15771

I'm trying to work through them all & then we can ideally validate payments in general. In this case
it turns out that because 'amount' is currently a 'required' parameter the tests have 'any value' stuck in there.
In a real submission it would be calculated so I'm trying to share the code that would do that with
the path used by the test (& in this case the api) and to move towards getting the tests valid

Merge pull request #15832 from eileenmcnaughton/mail_s

[NFC] minimal test for Mailing.update_email_resetdate

[NFC] minimal test for Mailing.update_email_resetdate

ALso removes a couple of unused variables, adds a few exception comments

Merge pull request #15823 from seamuslee001/dev_core_183_mailing

dev/core#183 Convert creating of temporary tables in CRM/Mailing file…

[REF] refactor on nasty Dedupe function

I found with other pieces of refactoring static functions that creating a class to support the refactor
made it much easier as that way I could leverage the fact thatt classes have properties and
get away from the crazy param passing that characterizes a nest of static functions.

This adds a class for that purpose and moves a small chunk of code handling into the class.

The goal is to move the handling is done purely for the form back onto the form....