Merge pull request #18008 from civicrm/5.28

5.28

Merge pull request #17983 from kcristiano/5.28-wp-notices

dev/wordpress/66  Add Upgrade Notice regarding legacy paths

Merge pull request #17987 from eileenmcnaughton/pseudo_mem

[REF] Reduce calls to CRM_Member_PseudoConstant::membershipType

Merge pull request #18002 from demeritcowboy/missing-soft-credit

dev/core#1915 - E_NOTICE when making pcp contribution

Merge pull request #18003 from demeritcowboy/membership-fail

[Test framework] - Update failing test

remove dubious code that never runs

notice on contribution edit

Improve docs for APIv4 Save action

Merge pull request #17975 from eileenmcnaughton/matt_yet_another

Fix repeattransaction api to use custom data from the template contribution

update failing test

Merge pull request #17777 from agh1/dontchangereceivedate

dev/financial#139 contribution receive_date shouldn't change when payments come in

Cache loader - remove legacy handling, handle null result from setting

We are seeing this symptom in our tests in some cases
https://civicrm.stackexchange.com/questions/28897/error-when-merging-accounts

I'm not quite sure why the setting would be empty - but treating empty the same as 'default'
seems better than a hard fail and we know that settings have a track history of not all being
loaded at the right times.

This also removes some legacy handling

E_NOTICE when pcp contribution

Merge pull request #17997 from civicrm/5.28

5.28

Merge pull request #17868 from seamuslee001/remove_civicrm_files_override

[REF] Remove civicrm.files override for WordPress to fix issues with…

Merge pull request #17970 from eileenmcnaughton/case_export

Fix case activity field set to allow long details to be exported

wrong link to admin page

Merge pull request #17986 from seamuslee001/dev_core_1913_alternate

dev/core#1913 Allow for schemas to be added by extensions if they are…

Merge pull request #17707 from seamuslee001/ref_permission_clause_search

[REF] Use Standard function cacheClause to re-use contact acl cache t…

Merge pull request #17978 from kainuk/issue-1914

Fixed filling default values for tagssets in the advanced search form

dev/wordpress/66  Add Upgrade Notice regarding legacy paths

Signed-off-by: Kevin Cristiano <kcristiano@kcristiano.com>

Merge pull request #17985 from demeritcowboy/sms-apiurl

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

Fix fieldSpec when exporting cases using certain (default) fields

Merge pull request #17988 from eileenmcnaughton/cache_dup

Remove duplicate cache flush

Remove functions deprecated a year ago

Reduce calls to CRM_Member_PseudoConstant::membershipType

This function defaults to not using the cached version so that's a pretty good argument for making a more
active effort to deprecate it. The createMembership function flushes it so we don't need to do that
so much in tests now either. I also included swapping status over in the test

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

dev/core#1913 Allow for schemas to be added by extensions if they are in the format of <entityname>Model

Update test doc blocks

Call api from BaseIPN->sendMail

This function has been deprecated since 2015 and it's tempting just to remove it but at this stage
it is being noisily deprecated & by-passed

avoid notice when no default url

Merge pull request #17305 from mlutfy/core1755

dev/core#1755 Fix reCaptcha on Mailing Subscribe

[REF] Remove pass-by-reference & always empty param

Once https://github.com/civicrm/civicrm-core/pull/17982 is merged there are only 2 places that call this function. In both
places
1) they instantiate an empty values array just to pass in
2) we can see they don't use input or ids afterwards, so they don't need to be passed by reference

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

SystemCheck: add ability to efficiently run only specified checks

This changes the hook_civicrm_check() signature in a backward-compat fashion,
adding 2 params that can be used by the hook subscriber to filter by name if specified.

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

dev/financial#139 contribution completeOrder shouldn't change receive_date

dev/financial#139 rework bulk payments form

dev/financial#139 prep: move CRM_Contribute_Form_Task_Status::getDetails to CRM_Contribute_Form_Task_Status::PDF because it'll be the only place it's used anymore

dev/financial#139: remove contrib status from bulk payments form template

dev/financial#139 prep: standardize markup, closing spans on bulk payment form

dev/financial#139 tests: contribution receive_date shouldn't change when payments received

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

Fixed filling default values for tagssets in the advanced search form

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…

Fix repeattransaction api to use custom data from the template contribution

This ensures (& tests) that custom data is copied from the template transaction - generally the most recent transaction

[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

Wrap multi record custom field inside a div

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

[REF] Simplify location metadata handling in Export class

The metadata array that is passed into getTransformedFieldValue is just a subset of the class metadata. It is
- keyed slightly differently
- only location fields

Obviously passing this around when we can access it directly is silly and it was as a consquence as this code having
started off incredibly toxic and a slow incremental cleanup process that it was done this way. The cleanup had
not reached that array.

This has excellent test cover with multiple variations of phones & Ims tested along with all entities. These exposed
a tangental improvement (not replacing the word 'No' with '' ) Email On Hold.

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

[REF] Fix jquery validation for radios

Fix layout of required radio message

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

dev/core#1906 - Allow payment create api to record payment on Failed Contributions

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