Merge pull request #14317 from eileenmcnaughton/refund

Switch to calling Payment.create api when processing a refund from AdditionalPayment form

Merge pull request #14392 from totten/master-test-exts

CRM_Extension_System - Only allow `test.*` extensions during headless testing. Hide from regular users.

Merge pull request #14384 from samuelsov/CRM-15375-next

Wysiwyg support for i18n event fields

CRM_Extension_System - Only handle `test.*` extensions on headless test sites

The headless test suite includes integration tests for correct handling of
the extension installation/removal process. This involves some
dummy extensions. Unfortunately, the dummy extensions are often picked up on
live deployments -- where they're a bit distracting/annoying.

Before
------

The `test.*` dummy extensions appear as available in all environments,
including headless tests and live sites.

After
-----

The `test.*` dummy extensions are only available in the headless test
environment.

CRM_Extension_Container_Basic - Add mechanism for filtering

This function allows one to filter down the list of extensions which are visible.

A few notes:

* The result of the scanning/filtering is stored in cache.  On a typical
  page-load, the system uses this cache -- which happily means that
  filtering shouldn't result in any extra page-load time.

* The extension-system is generally boot-critical.  I could imagine some
  scope-creep around making making the filter mechanism hookable, but that's
  a separate+complicated issue.  Optimistically...  if `array $containers`
  somehow becomes hookable, then one could use the same mechanism to
  manipulate the filters.

Merge pull request #14385 from seamuslee001/master

[NFC] Update package-lock and civicrm_generated to fix langague issues

Merge pull request #14333 from eileenmcnaughton/phpunit

dev/core#980 [phpunit test] update phpunit extends class to support later versions

Merge pull request #14377 from eileenmcnaughton/fin_batch

dev/financial#59 Fix unreleased regression on batch form

Merge pull request #14325 from eileenmcnaughton/merge_cust_field

dev/core#723 Fix fatal error when trying to merge contacts with a cus…

[NFC] Update package-lock and civicrm_generated to fix langague issues

Wysiwyg support for i18n event fields

Merge pull request #14376 from eileenmcnaughton/location_form

dev/core#994 remove Thousands Separator can not have more than 1 character rule

Merge pull request #14383 from civicrm/5.14

5.14

Merge pull request #14382 from seamuslee001/5_13_5_release_notes

5.14 version of  5.13.5 Release notes

Update 5.13.5.md

Update 5.13.5.md

Add in 5.13.5 Release notes

Merge pull request #14380 from civicrm/5.14

5.14

Merge pull request #14378 from seamuslee001/dev_core_969

Update bower.json to target fixed jquery branch for dev/core#969

Update bower.json to target fixed jquery branch for dev/core#969

Merge pull request #14368 from seamuslee001/dev_core_996

dev/core996 Ensure that the oldest created_date is retained no matter…

dev/financial#59 Fix unreleased regression on batch form

I think changing to use the search form as the parent makes sense here. Relying on the parent without including
it feels like a recipe for trouble

dev/core#994 remove Thousands Separator can not have more than 1 character rule

Per issue it seems unnecessarily opinionated to block longer strings & we not know of at least one case where it is inappropriate

Merge pull request #14375 from davejenx/activity-summary-refactor

[REF] Activity Summary report - move temp table generation etc from postProcess to buildQuery, remove postProcess, don't skip in unit tests

dev/core#723 Fix fatal error when trying to merge contacts with a custom file field

Technical comments here https://lab.civicrm.org/dev/core/issues/723#note_16231

Refactor CRM/Report/Form/ActivitySummary.php, removing postProcess(), moving temp table creation etc into buildQuery(). In tests/phpunit/api/v3/ReportTemplateTest.php, remove activitySummary from reportsToSkip, as tests pass after this refactoring; also rename testActivitySummary() to testActivityDetails(), as it actually tests report_id = Activity which is the activity details report.

Activity Summary report - stop overloading from function on activity summary report

dev/core996 Ensure that the oldest created_date is retained no matter the ids of the contacts being merged

Reduce sleep to 5

Merge pull request #14371 from seamuslee001/nfc_update_civicrm_generated_data

[NFC] Update civicrm_generated data file following addition of contri…

[NFC] Update civicrm_generated data file following addition of contribution_recur_status option group

Merge pull request #14343 from eileenmcnaughton/recur_status

dev/core#905 Separate contribution_recur status_id option group from contribution option group

Merge pull request #14369 from civicrm/5.14

5.14

Merge pull request #14355 from seamuslee001/harden_extern_open

Validate queue_id is a positive integer before passing to the BAO

Merge pull request #14365 from eileenmcnaughton/dedupe_move

[REF] move calculation of migrationInfo into the function that uses it

Validate queue_id is a positive integer before passing to the BAO

Switch to using retrieveValue as per Eileen

[REF] move calculation of migrationInfo into the function that uses it

This doesn't make a lot of sense as a passed parameter & it's better to derive it closer to where it's used.

It's also kinda silly....

Merge pull request #14354 from eileenmcnaughton/cont_search

Fix inconsistent handling when searching contribution text fields

Merge pull request #14098 from eileenmcnaughton/event_income

[REF] extract buildClause from CRM_Report_Form_Event_Income

Merge pull request #14359 from seamuslee001/fix_cache_key_column

Fix name of cache key column to be all lower case rather than camel c…

Fix name of cache key column to be all lower case rather than camel case in civicrm_prevnext_cache

Fix test failures

Merge pull request #14360 from colemanw/byRef

Remove confusing pass params by ref

Remove confusing pass params by ref

Fix inconsistent handling when searching contribution text fields

I found that if you search on contribution source it adds wildcards & uses 'like' but with cancel_reason not
so much.

I found that for contribution_source there was field-specific hacky handling, so I converted both fields
to be metadata based and added handling for string fields on the form layer

Merge pull request #14298 from eileenmcnaughton/dedupe_stats

dev/core#982 add dedupe.getstatistics api

Merge pull request #14358 from samuelsov/billingblockts

Translation in billingblock.tpl - dropping 'Information'

dev/core#905 Separate contribution_recur status_id option group from contribution option group

As we did earlier for pledge status this separates out the option groups - on the basis we have different
statuses for the 2 things. In order to ensure we don't break anything existing the values
we give the new option group the exact same values as the old one>

However, I HAVE added one more value - Processing - which is intended for the period between when
a new contribution starts to be added and when it has been added. This would normally be a few seconds
but when there is an error it would be left in this state - warding off future attempts.

For new sites I have left off some statuses that I think would not be relevant - Refunded,
Partially Paid, Chargeback, Pending Refund

Merge pull request #14357 from colemanw/api4Wrap

Don't apply api3 pre/post processing to api4 actions

Merge pull request #14352 from eileenmcnaughton/fin_status

Fix bug where contribution status is mishandled if label is changed

Translation in billingblock.tpl - dropping 'Information'

Skip wrapping v4 api requests in a transaction

Transactions aren't a bad thing and we probably want to revisit this but
for now I'm disabling it becuase it's broken, as seen in api_v3_ActivityTest.
I think the problem is with the onApiException event not being dispatched
because api4 doesn't go through Kernel->runSafe, so transactions aren't
being rolled back when they should.

Skip adding default api wrappers to v4 actions

Merge pull request #14356 from civicrm/5.14

5.14

Merge pull request #14351 from seamuslee001/ics_file

dev/core#974 Whitelist .ics extension

Whitelist .ics file extension

Include Upgrade step and regenrate civicrm_generated file

Fix upgrade

Merge pull request #14280 from seamuslee001/dev_core_975

dev/core#975 Fix url for new activity in breadcrumbs

dev/core#975 Fix url for new activity in breadcrumbs in find activity search

Fix bug where contribution status is mishandled if label is changed

Merge pull request #14350 from seamuslee001/test_dev_627

dev/core#627 Extend a unit test to verify fix made by Sunil

Merge pull request #14345 from mattwire/case_extraline

Remove duplicate line in case BAO

Merge pull request #13374 from cividesk/dev-627

correct syntax "IS NOT NULL" to get pledge payment count

dev/core#627 Extend a unit test to verify fix made by Sunil

Add in additional test for when no transactions have occured

Merge pull request #14342 from eileenmcnaughton/deprecated

[REF] Remove input format function from deprecated utils to the only place that calls it

Merge pull request #14335 from eileenmcnaughton/bao_contribution

[NFC] phpdoc cleanup

Merge pull request #14332 from civicrm/colemanw-patch-1

dev/core#968 Fix javascript error on public pages

Merge pull request #14334 from eileenmcnaughton/fin_status

[REF] extract updateFinancialAccountsOnContributionStatusChange

Merge pull request #14341 from eileenmcnaughton/deprecated2

Remove never-called function _civicrm_api3_deprecated_contact_check_custom_params

Remove duplicate line

Merge pull request #14336 from eileenmcnaughton/pdfletter

[REF] extract isLiveMode

Merge pull request #14344 from seamuslee001/test_tidy_up

[NFC] Further tests tidy up following removing of dbunit

[NFC] Further tests tidy up following removing of dbunit

Merge pull request #14331 from eileenmcnaughton/select

Fix fatal error when full group by mysql mode enabled & selecting contacts

Merge pull request #14340 from eileenmcnaughton/select2

Unit test for FGB error in #14331

Remove input format function from deprecated utils to the only place that calls it

Remove never-called function _civicrm_api3_deprecated_contact_check_custom_params

dev/core#968 Fix javascript error on public pages

Checks for the existence of the toolbar item before referencing it.

Unit test for FGB error in #14331

Merge pull request #14338 from eileenmcnaughton/contact_search

[REF] use generic loadStandardSearchOptionsFromUrl

[REF] use generic loadStandardSearchOptionsFromUrl

I picked this up as a reviewer's commit of https://github.com/civicrm/civicrm-core/pull/13651

That PR is stale but I like to pick out a bit I can get merged when closing a stale pr

Merge pull request #14138 from johncronan/drupal_login_url

dev/core#898: use user.login route for D8 login URL instead of user.page

[REF] extract isLiveMode

I'm closing https://github.com/civicrm/civicrm-core/pull/13369 as it is stale, but
I figure it makes sense to pull out this extraction from it. I think the PR then
alters the button names taken into account but I'm not seeking to fix here, just
to do the extraction

Merge pull request #14158 from demeritcowboy/deny-for-real

dev/core#913 - update autogenerated .htaccess for apache 2.4

Further phpcs mandated fixes

I removed the poorly formatted comment as it seemed out of date anyway

[NFC] phpdoc cleanup

Fixes style elements that stricter phpcs picks up

[REF] extract updateFinancialAccountsOnContributionStatusChange

dev/core#980 update phpunit extends class to support later versions

Per https://engineering.facile.it/blog/eng/phpunit-upgrade-namespace/ this works with phpunit 4.8 +

Phpunit support for php versions is here https://phpunit.de/supported-versions.html

Basically we need phpunit 7 to support php 7.3 - which we should be adding to our
test suite now. It's hard to make a case that we still need to support phpunit 4.8 - let
alone versions before that - 5.7 still covers php 5.6 which we need for some of our current branches.

Unit testing on master would ideally switch up to a supported version of phpunit - 7 or 8

Merge pull request #14328 from eileenmcnaughton/dbunit

 [dbunit test] remove dbunit dependency

Merge pull request #14292 from eileenmcnaughton/cachedtree

 Reduce cache bypasses - do not bypass custom metadata cache in setGroupTree

Merge pull request #14316 from eileenmcnaughton/membershipRenewal

Fix test / possible live error on submitting credit card renewals

Merge pull request #14324 from eileenmcnaughton/dedupe_cleanup2

[REF] Extract function to get cached duplicate matches

Merge pull request #14318 from eileenmcnaughton/reorg

[REF] minor code re-org

Merge pull request #14323 from eileenmcnaughton/dao

[NFC] Docblock fixes

Merge pull request #14329 from eileenmcnaughton/smtp

dev/core#879 SMTP help text is misleading - references the wrong admin screen

Merge pull request #13997 from eileenmcnaughton/recur_entity

dev/core#830 Expose cancel_reason field by cleaning up the cancel template to use entity field template

Disable full group by when selecting contacts

[dbunit test] remove dbunit dependency

Merge pull request #14321 from eileenmcnaughton/test_dbunit

dev/core#980 [dbunit] remove dependency on discontinued dbunit

dev/core#879 SMTP help text is misleading - references the wrong admin screen

Merge pull request #14087 from seamuslee001/npm_version_update

(NFC) Update node package versions to latest ones

Merge pull request #14082 from pradpnayak/caseReport

dev/core/issues/743, Donot join relationship table when not required

[dbunit] remove dependency on discontinued dbunit