Merge pull request #12647 from compucorp/fix-case-type-issues

Case type management fixes

Merge pull request #13480 from pradpnayak/AdvanceSearch

dev/core/issues/506, Fatal error on advance search when using cases from display results as

Merge pull request #13495 from seamuslee001/lab_core_676

dev/core#676 Disable GeoCoding Tests for PR jobs and re-enable Mailin…

dev/core#676 Disable GeoCoding Tests for PR jobs and re-enable Mailing Test jobs for PRs previously disabled now fixed

Merge pull request #13492 from pradpnayak/typoErrors

Removed '>' from comment

Merge pull request #13461 from christianwach/lab-666

Fix trailing ampersands when empty array passed as query param

Removed '>' from comment

Merge pull request #13456 from mfb/geocode-country_id

geocode job: Provide country_id to geocoders.

Merge pull request #13482 from jgillmanjr/front_contribution_page

Test-drive contribution pages to the front

Merge pull request #13427 from chamilwijesooriya/issue-652

GitLab issue 652 Copying activity file custom data doesn't copy mime …

Merge pull request #13486 from deb1990/core-670-fix-activity-tags

(WIP) dev/core#670 - Fix Saving of Case Activity tags

dev/core#670 - Fix saving of tags in case activity

Merge pull request #13437 from mattwire/ref_simplifybillingblock_template

Simplify billingblock in Contribution/Form/Main template

Merge pull request #13473 from deb1990/C51-384-add-case-token-in-email

C51-384: Added case tokens on Email Activity Modal

Merge pull request #13457 from mattwire/contact_note

REF Remove undefined variable when creating note

Merge pull request #13481 from laryn/patch-1

Add main nav icon for CiviCRM (Backdrop)

Removed extra space

Merge pull request #13479 from pradpnayak/cruftCode

removed unwanted parameter from getLineItems()

Add main nav icon for CiviCRM

(since Backdrop core now has main nav icons)

Specify front end (where applicable) when getNotifyUrl is called

Force test contribution pages to front end

Merge pull request #13452 from eileenmcnaughton/depre

Deprecate unused function

dev/core/issues/506, Fatal error on advance search when using cases from display results as

removed unwanted parameter from getLineItems()

Merge pull request #13317 from eileenmcnaughton/order_pseudo

Fix order api to support a pseudoConstant for financial_type_id

Fix order api to support a pseudoConstant for financial_type_id

Adds functionality & test to support 'Event Fee' instead of '4' for Order.create financial_type_id.

This improves consistency with Contribution.create api and others

Merge pull request #13474 from JMAConsulting/dev-financial-39

dev/financial#39: Drop checkMD5 support from Authorize.net

Merge pull request #13475 from civicrm/5.10

5.10 to master

Merge pull request #13451 from eileenmcnaughton/everyone_510

(dev/core#660) Address regression whereby Anonymous users can no longer register for an event if they have ACLs to see a contact

dev/financial#39: Drop checkMD5 support from Authorize.net

Merge pull request #13324 from agileware/CIVICRM-947

Email sent from CiviCRM for a new Case and Activity does not evaluate the $activityTypeName or $manageCaseURL tokens

Merge pull request #13471 from seamuslee001/nfc_note_tests

(NFC) Add unit test of creating notes from the contact.create API

C51-384: Added case tokens on Email Modal

(NFC) Add unit test of creating notes from the contact.create API

Merge pull request #13439 from jgillmanjr/paypal-standard-ipn-tz-fix

Convert Paypal Standard IPN payment_date to system's time zone

Merge pull request #13442 from eileenmcnaughton/paymen_line

Remove inappropriate exception handling.

Merge pull request #13466 from colemanw/tsFix

Translate untranslated string

Merge pull request #13458 from mattwire/unused_var_caseformcustomdata

REF Remove duplicate call to session singleton

Merge pull request #13453 from eileenmcnaughton/report_tpl

Escape the header title & section title in reports to better support extensions

Merge pull request #13465 from mlutfy/setLocale

setLocale: precaution for when setLocale is called on an unilingual site

Translate untranslated string

Activity emails: Fix activityTypeName template var & fix priority_id label

Merge pull request #13455 from eileenmcnaughton/payment_email_test

Add unit test for emailing receipts from additional payment page

setLocale: precaution for when setLocale is called on an unilingual site

Fix trailing ampersands when empty array passed as query param

Remove undefined variable when creating note

Remove duplicate call to session singleton

Remove foreign key on civicrm_acl_contact_id.user_id.

This won't actually remove it from installs - we need to address that separately via ensuring people can and do run
the System.updateIndices api call but it removes it from new installs and from
tests, hence the test should pass

Fix mishandling / loss of check_permission flag

Add test to demonstrate fatal error when accessing permitted users that are cached using the acl_cache.

This has arisen during investigation of a possible regression - it turns out that if you give the 'everyone' group
access to a contact using ACLs (or hooks I believe) they get a fatal error on any attempt at event or other registration.

The issue is that when attempting to check for duplicates the call is made using check_permission. This in itself is a possible
regression as the CRM_Dedupe_Finder::dupesByParams function now drops the check_permission key when it is equal to 0 from

https://github.com/civicrm/civicrm-core/commit/4f33e78b901fb7cdb38a3026f88b59a2f9fd2c68

So we have an issue that
1) we are now applying check_permission when doing a dupe_check from front end forms - this probably is resulting in 5.9
sites getting too many duplicates are they would always be null for anon users
2) if we DO do a permissions check when an acl or hook has been used to give anon users permission to access
contacts then they will get a fatal error. This is because it sets contact_id to 0 and attempts to insert it into the
acl_contact_cache.

I think we need to either remove the array_filter line that we think we may not need per code comments
or add specific handling for the check_permission flag

AND drop the foreign key constraint on the civicm_acl_contact_cache table. This means they will
no longer be removed when a contact is deleted but this is a clean up issue rather than one with
functionaly implications & we *should* have some form of cleanup in play on that table. In addition,
removing the constraint will reduce write contention

geocode job: Provide country_id to geocoders.

Add unit test for payment email from AdditionalPayment form, rationalise some parameters to do this.

Note that the goal is to move this function off this class to somewhere api accessible so breaking the connections
with the form makes sense & in fact some like the fromEmails handling are really just badly copied & pasted from elsewhere
(ie. the fromEmailIds doesn't exist & is created on the fly to house the email being passed in

Merge pull request #13444 from eileenmcnaughton/payment_smarty

[REF] Replace ->assign with CRM_Core_Smarty::singleton()->assign in preparation for making function shareable

Escape the header title & section title in reports.

In extended reports I'm making it possible for column titles to be user edited. since they will represent user input it
makes sense to escape on output. I think core makes sense as the place to do this as it is best practice
to escape all output in the smarty layer and I have tested that it still works fine

Deprecate unused function

Merge pull request #13450 from civicrm/5.10

5.10 to master

Merge pull request #13448 from eileenmcnaughton/everyone_510

Fix unreleased regression on building GroupContact clause

Merge pull request #13449 from colemanw/caseDateSearch

Fix broken case activity date filter

Fix unreleased regression whereby acls are skipped for groups

Quick comment describing need for IPN TZ adjustment

Utilize getTimeZoneString() method

Co-Authored-By: jgillmanjr <jason@rrfaae.com>

Convert Paypal Standard IPN payment_date to system's time zone

Fix broken case activity date filter

Merge pull request #13438 from colemanw/timeIcon

Use icon for timepicker placeholder

Add unit test to cover granting everyone' group permission to access a group

Remove inappropriate exception handling.

Per discussion in https://github.com/civicrm/civicrm-core/commit/570243c292fa7bceea74ce233feceb3f69211e54#r31805899

the core logic has been changed to now accept a payment regardless of contribution status. This reflects the
fact that a payment could be received for a fully paid contribution and we still need to record it.

This exception is inappropriately narrow (& likely in the wrong place since it should probably be closer to
where the decision is made about adding a payment rather than when it is being processed

Merge pull request #13447 from eileenmcnaughton/dumb_stuff

[tidy-up] remove a couple of useless bits of code

Merge pull request #13443 from eileenmcnaughton/move_payment

[REF] Create pseudo bao CRM_Financial_BAO_Payment & move create function to  it

Merge pull request #13446 from civicrm/5.10

5.10

Remove verify_mandatory that does nothing due to nothing to verify

Remove unused parameter

Merge pull request #13445 from seamuslee001/dev-656

(dev/core#656) unreleased regression - attendee report generates unde…

(dev/core#656) unreleased regression - attendee report generates undefined property notice

Replace ->assign with CRM_Core_Smarty::singleton()->assign in preparation for making function shareable

Per PR 1330 there is an intention to make this form receipting function a shared function. Part of making it sharable is making it static.

This is a preparatory refactor towards that end, getting rid of the 'this' in assigning variables to smarty

Create pseudo bao CRM_Financial_BAO_Payment & move create function to it.

This is per discussions coming off https://github.com/civicrm/civicrm-core/pull/13330

Note that this ONLY moves the code - that PR does other things as well that still need to be worked through

Use icon for timepicker placeholder

Merge pull request #13412 from mattwire/extract_customData

REF Convert forms to standard customData template

Don't call getEntitySubTypeId if a subType is not defined

Simplify billingblock in Contribution/Form/Main template

Merge pull request #13134 from agileware/CIVICRM-1006

dev/core#500 Fix user-specific Case filtering on dashboard and searches to exclude cases from inactive relationships

Merge pull request #13433 from colemanw/actSub

Fix undefined index warning

Merge pull request #13377 from eileenmcnaughton/activity_extract2

Fix activity.getcount function to filter out unpermitted activities.

Merge pull request #13409 from yashodha/dev-646

(dev/core#646) Event date sorting doesn't work for ical listing

Fix undefined index warning

Merge pull request #13430 from civicrm/5.10

5.10

Merge pull request #13429 from seamuslee001/5.10

Port Test fixes to 5.10

(NFC) Ensure that when loading in the test data it is done with UTF8 encoding

(dev/core#647) Fix up Test Class names and file names to ensure all test classes are run by Jenkins

(dev/core#647) Fix Up Contact Selector Test by changing class name to get test to run and make changes following code changes that affect clauses and calling get rows

CRM_Contact_SelectorTest - Fix escaping

(dev/core#217) CRM_Contact_SelectorTest - Remove stale/unnecessary assertion

The table `civicrm_prevnext_cache` includes three columns (`entity_id1`,
`entity_id2`, `entity_table`).  These columns have some use-cases (e.g.  for
deduping) and some historical value, but the only one needed for
caching search-results is `entity_id1`.

dev/core#217 introduced an interface `CRM_Core_PrevNextCache_Interface` for
caching search-results.  This interface only passes through a list of CID's
(effectively `entity_id1`); because we don't need `entity_id2` and
`entity_table`, it makes no provisions for passing them through, and they're
not saved.  This simplifies the interface and makes it more realistic to
implement alternative prevnext storage mechanisms.

Merge pull request #13211 from eileenmcnaughton/search_grant

dev/core#561 Convert grant search fields to use datepicker

Merge pull request #13395 from eileenmcnaughton/datepicker-update

Add routine for updating smartgroups, currently handling datepicker conversion

Merge pull request #13426 from eileenmcnaughton/report_instance_domain

Add default for domain_id for report_instance.

Merge pull request #13428 from mattwire/nfc_membership_function

REF Remove redundant function

Remove redundant function

GitLab issue 652 Copying activity file custom data doesn't copy mime type

CIVICRM-1128: Added Unit Tests to validate the Case counts when case role is active/inactive.

Merge pull request #13416 from seamuslee001/647_selector_test

(dev/core#647) Fix Up Contact Selector Test by changing class name to…

Add default for domain_id for report_instance.

This is consistent with other api like Job, MembershipType, PaymentProcessor

Merge pull request #13424 from seamuslee001/dev-drupal-43

dev/drupal#43 Update composer requirement on PSR Log and civicrm-cxn-…

(dev/core#647) Fix Up Contact Selector Test by changing class name to get test to run and make changes following code changes that affect clauses and calling get rows

CRM_Contact_SelectorTest - Fix escaping

(dev/core#217) CRM_Contact_SelectorTest - Remove stale/unnecessary assertion

The table `civicrm_prevnext_cache` includes three columns (`entity_id1`,
`entity_id2`, `entity_table`).  These columns have some use-cases (e.g.  for
deduping) and some historical value, but the only one needed for
caching search-results is `entity_id1`.

dev/core#217 introduced an interface `CRM_Core_PrevNextCache_Interface` for
caching search-results.  This interface only passes through a list of CID's
(effectively `entity_id1`); because we don't need `entity_id2` and
`entity_table`, it makes no provisions for passing them through, and they're
not saved.  This simplifies the interface and makes it more realistic to
implement alternative prevnext storage mechanisms.