Merge pull request #17985 from demeritcowboy/sms-apiurl

dev/core#1909 - Avoid E_Notice on SMS provider form when no default url

Merge pull request #17988 from eileenmcnaughton/cache_dup

Remove duplicate cache flush

Merge pull request #17968 from eileenmcnaughton/cont_page

[REF] Extract setUserContext on contribution form & cleanup on backend add membership form

Remove duplicate cache flush

As the comment points out CRM_ACL_BAO_Cache::resetCache is also called in CRM_Utils_System::flushCache
so we are running a query that is inclined to lock tables an extra time for no reason

avoid notice when no default url

Merge pull request #17305 from mlutfy/core1755

dev/core#1755 Fix reCaptcha on Mailing Subscribe

Merge pull request #17979 from eileenmcnaughton/is_email

[REF] Make explicit what we are doing with 'values' in this code

[REF] Make explicit what we are doing with 'values' in this code

I've kept this really small so it can be verified by someone unfamiliar with this code.

Basically if you look you can see the only place 'values' is used is to determine if is_email_receipt is set.
That means everything else in  is unused & ergo we don't need to set it / can throw it away.

Obviously the next step is to extract all the code that determines  into a single place and say
goodbye to  altogether

Merge pull request #17817 from colemanw/getChecksConfig

Simplify caching of status checks

Merge pull request #17920 from eileenmcnaughton/dupe

[REF] Reduce interaction between dedupe code and createProfileContact

Merge pull request #17977 from eileenmcnaughton/custom

ensure custom field checkboxes are populated in profiles

Merge pull request #17927 from monishdeb/core-785

dev/core#785 Differentiate smart group from regular group using icon in select2 field

Revert "revert PR 13958"

This reverts commit ba2ad46f76db57c8f9bea064e7ba4c5b5adefc83.

Merge pull request #17974 from eileenmcnaughton/receipt

[REF] Minor code clean up

ensure custom field checkboxes are populated in profiles

Merge pull request #17976 from civicrm/5.28

5.28 to master

Merge pull request #17973 from seamuslee001/ref_profile_date_fields

[REF] Fix regression where adding any date based field onto a profile…

[REF] Fix regression where adding any date based field onto a profile triggers an error date preferences not configured when previewing the profile

[REF] Minor code clean up

This just consolidates a little of the code used to determin if an email receipt should be sent.
It's clear that there is more code above that could be pulled into this & combined
with the isEmailReceipt function but I wanted to just offer up a small reviewable change unless
there is reviewer enthusiasm for more

Merge pull request #17967 from eileenmcnaughton/grant

[REF] Grant cleanup

Merge pull request #17965 from seamuslee001/widget_test

[NFC] Add in a unit test of calling the contribution page widget endp…

[REF] Extract setUserContext on contribution form & cleanup on backend add membership form

[REF] Grant cleanup

This does a couple of things to cleanup the Grant BAO -
1) moves money handling back to the form layer
2) removes unnecessary date handling - the format that comes from the form layer
is now supported by the BAO.

I was looking to work on another but but then figured I needed a v4 api before
writing the tests and then realised we shouldn't do a v4 api before getting
rid of the legacy money handling.

It does occur to me it might make more sense to start moving to an
extension before doing any of the other steps

[NFC] Add in a unit test of calling the contribution page widget endpoint and remove unneded file

Merge pull request #17963 from civicrm/5.28

5.28

Merge pull request #17962 from seamuslee001/dev_core_1909_profile

dev/core#1909 Fix E-notice when adding a field on a profile

Merge pull request #17964 from seamuslee001/dev_core_1909_paymentprocessor

dev/core#1909 Fix e-notice when adding a payment processor

dev/core#1909 Fix e-notice when adding a payment processor

Merge pull request #17960 from demeritcowboy/contribution-default

dev/core#1911 - Default not being set for fixed contribution amounts or any price field that is not type text

dev/core#1909 Fix E-notice when adding a field on a profile

Merge pull request #17959 from demeritcowboy/file-case-notice

dev/core#1909 - E_NOTICE opening file-on-case

Merge pull request #17941 from eileenmcnaughton/fieldspec

[Ref] Simplify field reference

default not being set for price fields

E_NOTICE opening file-on-case

Merge pull request #17957 from demeritcowboy/combine-createcase

[Test framework] - Combine triplicate createCase functions

Merge pull request #17932 from seamuslee001/ref_radio

[REF] Refactor to use the standard CRM_Core_Form::addRadio function f…

combine duplicate functions

Merge pull request #17948 from seamuslee001/mail_mime_upgrade

Upgrade PEAR/mail_mime package to be compliant with PHP7.4 and deploy it using composer

Merge pull request #17938 from seamuslee001/ref_fix_financial_account

[REF] Fix the default to_financial_account_id for generated transactions

Merge pull request #17955 from colemanw/superstar

APIv4 - Add keyword to select all custom fields

Upgrade PEAR/mail_mime package to be compliant with PHP7.4 and deploy it using composer

Merge pull request #17946 from seamuslee001/ref_dompdf_upgrade

[REF] Upgrade dompdf version to be more compatible with PHP7.4

APIv4 - Add keyword to select all custom fields

Merge pull request #17939 from eileenmcnaughton/ex_meta

[REF] [Tests] Cleanup test declaration to take advantage of mapping improvements

Merge pull request #17914 from demeritcowboy/invisible-messages

dev/drupal#127 - CRM_Core_Session::setStatus() gets ignored sometimes

Merge pull request #17953 from civicrm/5.28

5.28 to master

Merge pull request #17947 from agh1/dev-core-1899-5-28

dev/core#1899 specify display mode for action links with icons

Merge pull request #17949 from eileenmcnaughton/export_im

[REF] Remove unnecessary complexity on im export

[REF] Remove unnecessary complexity on im export

This just removes a couple of lines that were already marked as being likely unhelpful. Because the impact of
these lines is tested the test needs tweaking too - but it only affects the name of the fields in the temp
table and in internal php keys

dev/core#1899 specify display mode for action links with icons

[REF] Upgrade dompdf version to be more compatible with PHP7.4

Merge pull request #17945 from colemanw/commentCleanup

NFC - Docblock cleanup

Merge pull request #17944 from colemanw/caseCheck

CRM_Utils_Check_Component_Case - Guard against post-upgrade crash

NFC - Expand CRM-* comments with full @see annotations

NFC - Fix @return annotation in CRM_Utils_Check_Component_*

CRM_Utils_Check_Component_Case - Guard against post-upgrade crash

Merge pull request #17940 from eileenmcnaughton/abort

Remove error checking by-pass in tests

Merge pull request #17936 from eileenmcnaughton/cache

Remove extraneous opportunistic cache flush.

[Ref] Simplify field reference

Remove error checking by-pass in tests

[REF] [Tests] Cleanup test declaration to take advantage of mapping improvements

[REF] Fix the default to_financial_account_id for generated transactions

Remove extraneous opportunistic cache flush.

The opportunistic cache flush is already called by contact create and then again by each of the places
where the contact might be added to a group so this is just extraneous

[REF] Refactor to use the standard CRM_Core_Form::addRadio function for a number of radio elements

Merge pull request #17930 from twomice/1902_contrib_source_profile

dev/core#1902: "Contribution Source" profile field has no effect

Merge pull request #17928 from colemanw/groupContactCleanup

[REF] GroupContact BAO - Minor code cleanup

Merge pull request #17931 from civicrm/5.28

5.28 to master

Merge pull request #17929 from demeritcowboy/required-not-5.28

dev/core#1903 - Avoid E_WARNING and remove code

dev/core#1902: "Contribution Source" profile field has no effect on new contribution.

avoid E_WARNING and remove code

GroupContact BAO - Minor code cleanup

Merge pull request #17922 from eileenmcnaughton/group_contact

[REF] Minor function signuture cleanup

Merge pull request #17921 from eileenmcnaughton/format

[REF] Do not pass variable by reference

[REF] Reduce interaction between dedupe code and createProfileContact

Ideally we want to simply call apiv4 from the deduper. However, it seems to take a bit of analysis to determine
if anything is done in there that might have some impact. This takes one step in that direction.  I copied the full function over
and then remove all the parts that are unreachable due to the passed in params or meaningless due to us not having Billing
fields. I think the Profile hooks are overkill - the merge hooks and contact hooks are being run and no profile is being submitted

Merge pull request #17926 from civicrm/5.28

5.28

[REF] Minor function signuturre cleanup

 is never altered so it is unnecessary to pass by reference. visibility is just
used to set ignorePermissions

Merge pull request #17925 from seamuslee001/5.28

Add release-notes/5.27.3

Add release-notes/5.27.3

Merge pull request #17918 from mlutfy/docUpdates

[NFC] Update a few doc/wiki links in code comments

[REF] Do not pass variable by reference

This variable is not altered in the function so does not need to be passed by reference

Merge pull request #17916 from colemanw/domainCache

Improve caching of current domain

Merge pull request #17778 from totten/master-setup-validate

Setup UI - Validate that at least one "Component" is enabled

Merge pull request #17911 from agh1/memberdetailreportautorenew

Member detail report: nest "in" options in parentheses

Update a few doc/wiki links in code comments

Simplify caching of status checks and fixes missing domain_id condition

Fixes a few apparent problems:
- domain_id was ommitted causing unexpected results
- Civi::cache was being read but never written to
- Object caching did not persist in memory across different check classes
- If we did write to Civi::cache there is no mechanism for clearing that cache when data changes

Solution: use simple array cache in Civi::$statics

APIv3 - If requesting current domain, read from cache

Use sensible caching for current domain

Merge pull request #17917 from christianwach/lab-core-1901

Fix sticky table header on "Find Activities" page

Fix sticky table header on "Find Activities" page

avoid messages not being shown

Member detail report: nest "in" options in parentheses

Merge pull request #17908 from civicrm/5.28

5.28

Merge branch 'master' into 5.28

Merge pull request #17906 from seamuslee001/lab_core_1846

dev/core#1846 Make DAO upgrade safe

Fix failing settingTest and mark setting.fill as deprecated

The settingTest class was being too aggressive about creating and deleting domains,
this teaches it to not delete pre-existing domains.
Also marks an old unused api function deprecated.

Add upgrade-safe DAO::getSupportedFields method

Switches api v3 and v4 to use that method so they are upgrade-safe by default.

Merge pull request #17903 from eileenmcnaughton/format

[REF] remove unnecessary variable variables

Merge pull request #17901 from demeritcowboy/outputhandler2

dev/core#1855 - Allow different output formats for CiviReport results and untangle code

Merge pull request #17650 from colemanw/getMultilingual

[REF] - Add helper function for the repetitive task of fetching multilingual

Merge pull request #17905 from MegaphoneJon/fix-dummy-processor

[unreleased regression] Dummy processor now is ID 7 on buildkit sites