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'

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

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

Merge pull request #14327 from seamuslee001/fix_warning_job_manager_test

[NFC] Fix issue with Job Manager Test giving out a warning

[NFC] Fix issue with Job Manager Test giving out a warning

Merge pull request #14326 from civicrm/5.14

5.14

Merge pull request #14322 from AlainBenbassat/5.14

fixes issue #981: sort participants did not remember search criteria

[REF] Extract function to get cached duplicate matches

Docblock fixes

These are the drupal coder standard but with less rules commented out. Note the null vs NULL
is coming from there.

I think we can now merge https://github.com/civicrm/coder/pull/7 without undue pain on submitters.

fixes issue #981: sort participants did not remember search criteria

Merge pull request #14310 from eileenmcnaughton/dedupe_custom_tests

Add unit test for merge handling on custom data.

Merge pull request #14314 from eileenmcnaughton/coder

[NFC] Fix comment block for variable declarations

Merge pull request #14319 from eileenmcnaughton/xml_dbunit

[dbunit] remove dependencies on discontinued dbunit test package

[dbunit] remove dependencies on discontinued dbunit test package

Merge pull request #14315 from eileenmcnaughton/dbunit_xml

[dbunit] remove dbunit dependency from MembershipRenewalTest

NFC  - Changed release notes info about Jira to GitLab (#14265)

* Changed info about Jira to GitLab

I'd love some feedback on exactly which release these issues dropped Jira/started using GitLab, as I don't know that information.

* Re-add Jira

* Re-include reference 4.7.14

* Re-include Jira issues, but below GitLab as Jira is currently deprecated

* Remove info about date of Jira depreciation

* Remove info about date of Jira depreciation

* Remove info about date of Jira depreciation

* Remove extra empty line

* Switched order of GitLab and Jira

[REF] minor code re-org

This simply moves the blocks that are wrapped in if ( == 'changedStatus') into
one block & puts them directly after the block which handles early returns for (some)
status changes

Merge pull request #14249 from yashodha/959_dev

(dev/core#959) Expose contribution page in Contribution Summary report

dev/core#982 add dedupe.getstatistics api

Per gitlab the intent is to add a bunch of api so get the logic out of the form into somewhere more accessible.

Following agreement on this I would change all calls to CRM_Dedupe_Merger::getMergeStats to call this api
and get rid of the call to CRM_Core_BAO_PrevNextCache::retrieve from getMergeStats since that retrieve
function really does differrent stuff & is being kinda pointlessly overloaded here.

Fix test / possible live error on submitting credit card renewals

Merge pull request #14269 from eileenmcnaughton/refund_up

[REF] Cleanup input & output on paymentActivityCreate & improve test

[dbunit] remove dbunit dependency from MembershipRenewalTest

Merge pull request #14272 from artfulrobot/artfulrobot-gitlab-issue-973

Fix legacy IPN endpoint for Drupal (gitlab issue 973)

Fix comment block for variable declarations

Per https://github.com/civicrm/coder/pull/7

Merge pull request #14279 from eileenmcnaughton/db_test2

dev/core#980 [test-dbunit]  Cleanup profile test to not use discontinuted dbunit

Merge pull request #14302 from mlutfy/drupal19

dev/drupal#19 Drupal8: fix bootstrap when used by the REST API

Merge pull request #14313 from AlainBenbassat/issue-981

dev/core#981 Sorting participants ignored search values.

Merge pull request #14309 from pradpnayak/FinancialItem

dev/financial#56 Fix inappropriate sales tax row addition

Merge pull request #14312 from spalmstr/Joomla_11

dev/joomla#11 Fix php warning when viewing profiles

Reduce cache bypasses

We have been running this patch in production for some months. When you load up a contribution edit page
(for example) the custom data metadata is loaded without hitting the cache. This means that
unnecessary queries are done. The only reason to bypass the cache would be that it is stale
or parameter dependent. However, the staleness is/ should be dealt with when editing custom
fields & if it is not then there will be other impacts. The entity id is NOT passed into the part of
the code that goes into the cached value

Add unit test for merge handling on custom data.

This also moves the start & stop of the transaction to being further out so a hard fail
later doesn't leave a half-merged state

Merge pull request #14307 from seamuslee001/dao_style

NFC Fix DAO Style issues

corrects test on empty formvalues in postProcess

Merge pull request #2 from civicrm/master

getting newest code

Merge pull request #14308 from eileenmcnaughton/uffield

[NFC] reformat UFFieldTest

Merge pull request #14311 from colemanw/public

Make apiRequst nextId accessible for api4 constructor