Merge pull request #15818 from colemanw/fields

Field metadata cleanup: Contact & Activity & Custom

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

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.

translate strings

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

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....

dev/core#183 Convert creating of temporary tables in CRM/Mailing files to using CRM_Utils_SQL_TempTable

dev/core#183 Convert Logging report summary report to using CRM_Utils_SQL_TempTable

Merge pull request #15822 from eileenmcnaughton/label

Fix LabelFormat class to throw exceptions rather than fatals

Merge pull request #15803 from eileenmcnaughton/participant

Add unit test for searching by participant_status_id & switch to generic function

Fix LabelFormat class to throw exceptions rather than fatals

I also added some throws to the comment blocks & fixed on fn not to be called by ref

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

This test wasn't really testing much since the functions were kinda contorted into doing what the
test was testing. I've updated it to do set up using Order.create & not to use the 'partial_amount_to_pay'
keys which turn out not to create valid transactions (& which we want to fully remove from the other places).

I had to co-erce tax_amount into a float in a couple of places & shortened a param for readability.

I commented out a couple of checks that only worked because of unrealisitic function calls
I added https://lab.civicrm.org/dev/financial/issues/102 to track that.

Add unit test for searching by participant_status_id & switch to generic function

Don't guess input type; rely on dao fields

Field metadata cleanup: Contact & Activity

Merge pull request #15791 from seamuslee001/event_search_url_variables

Fix Participant Search in force mode to support query parameters in URL

Merge pull request #15812 from eileenmcnaughton/fee

Convert fee selection test to use Order api

Merge pull request #15809 from eileenmcnaughton/except

[NFC] declare thrown exceptions

Convert fee selection test to use Order api

This switches the changeFeeSelection test to use Order api to set up the line items etc correctly

I had to remove a couple of checks that were no longer true but they were never 'true' in a
real working flow

Merge pull request #15699 from mattwire/participant_cleanup_completeOrderPBRef

Don't pass \$input to completeOrder by reference - return values are never used

Fix Participant Search in force mode to support query parameters in URL in Advanced Search.

[NFC] declare thrown exceptions

Merge pull request #15807 from eileenmcnaughton/part_status

Handling comma separated list of participant statuses in url

Merge pull request #15806 from eileenmcnaughton/member_urls

[REF] Add setSearchMetadata to each of the classes that extend CRM_Core_Form_Search

Merge pull request #15805 from eileenmcnaughton/cont_urls

[REF] extract setDeprecatedDefaults

Merge pull request #15802 from eileenmcnaughton/complete_order_to_calling

Test fixes - addresses more invalid setup data

Merge pull request #15799 from eileenmcnaughton/valid

FinancialTrxnTest - Fix unit test to  be 'legit'

Update url handling of select fields

Merge pull request #15808 from civicrm/5.20

5.20

Merge pull request #15804 from eileenmcnaughton/fns

dev/core#1381 fix fatal on case search (unreleased regression

Merge pull request #15795 from seamuslee001/dev_core_183_mailing_test

dev/core#183 Convert creating of temp table in mailing test to use st…

Add setSearchMetadata to each of the classes that extend CRM_Core_Form_Search

This doesn't do something in itself but makes it simpler to handled ensuring url_params work & stdises them

[REF] extract setDeprecatedDefaults

Remove 'partially paid' as a contribution status option for 'record payment'

Fixes a bug where it is possible to select contribution statuses that do not result in valid financial
transactions. Specifically the 'Partially Paid' option creates no payment transaction and any subsequent
financial_trxns get the wrong line item allocations as a result.

dev/core#1381 fix fatal on case search (unreleased regression

dev/core#183 Convert creating of temp table in mailing test to use standardised CRM_Utils_SQL_TempTable interface

[REF] very minor tidy up

This just brings the 3 lines that built this 'array within an array' together & makes them a little more
readable

Fix data to be valid for one of the a.net tests

The tests were munging the line items in set up - this fixes it - but only for one so far

Merge pull request #15801 from eileenmcnaughton/format

[NFC] reformat CiviUnitTestCase.php

Fix unit test to  be 'legit'

This unit test is one of a bunch that came up as not creating valid payments. The issue is
in the test set up methodology (the 'partial_payment_to_pay' key doesn't work).

On tidying up the test proved to be mostly cruft from an earlier iteration

Merge pull request #15798 from seamuslee001/dev_core_183_imports_2

dev/core#183 Convert database testing to use CRM_Utils_SQL_TempTable …

[NFC] reformat CiviUnitTestCase.php

dev/core#183 Convert database testing to use CRM_Utils_SQL_TempTable fuctionality and remove Incomplete Jobs handling as it is not called

Implement drupal_get_user_timezone()

Merge pull request #15790 from civicrm/5.20

5.20

Merge pull request #15787 from eileenmcnaughton/recur_ug

Remove unhelpful alert from contribution search

Merge pull request #15665 from MikeyMJCO/patch-1

Create a GitLab general issue template

Merge pull request #15789 from seamuslee001/5_19_1_version_notes

5.19.1 Version Release Notes

Merge pull request #15783 from eileenmcnaughton/sort_id

[REF] replace copy & paste with shared getSortID function.

Update 5.19.1.md

Add in release notes for 5.19.1 drop

Merge pull request #15786 from eileenmcnaughton/cont_type

Remove obsolete warning

Remove unhelpful alert from contribution search

[REF] replace copy & paste with shared getSortID function.

I've replaced this repetitive code chunk across the forms that inherit from CRM_Core_Form_Search

Merge pull request #15784 from civicrm/5.20

5.20

Remove obsolete warning

The time has come to let this one go.....

Merge pull request #15781 from eileenmcnaughton/cont_search

dev/core#1374 Fix search formValue handling on contribution search