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

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

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

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

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

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

Merge pull request #14211 from colemanw/routeBinder

crmRouteBinder - Remove params from url if they equal their defaults

Joomla_11 Patch CRM/Contact/BAO/Contact/Utils.php

Make apiRequst nextId accessible for api4 constructor

Merge pull request #14303 from eileenmcnaughton/extract_cache

[REF] extract buildGroupTree function

Merge pull request #14301 from eileenmcnaughton/reltest

[REF] Test class cleanup - Extract function to create nuanced custom fields to a trait

Added unit test

removed space

Merge pull request #14102 from jitendrapurohit/core-888

dev/core#888 - Backoffice participant transfer form does not populate…

NFC Fix DAO Style issues

Remove excess whitespace with Coleman's regex replace

Remove unneeded whitespace

Merge pull request #14306 from eileenmcnaughton/doc_block

[NFC] fix comment blocks capitalising null

Merge pull request #14305 from eileenmcnaughton/bool

[NFC] fix more places where var is declared as boolean rather than bool

[NFC] reformat UFFieldTest

[REF] extract buildGroupTree function

I did this mainly to surface the fact that the variables involved are factored into the cachekey

Cleanup profile test to not use discontinuted dbunit

[NFC] fix comment blocks capitalising null

From running the drupal code sniffer with type checking enabled it turns out the capitalisation of NULL
does not extend to docblocks.

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

[NFC] fix more places where var is declared as boolean rather than bool

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

Merge pull request #14304 from eileenmcnaughton/dao

[NFC] formatting & comment fixes - Regenerate DAO files

Merge pull request #14282 from eileenmcnaughton/profiletest

[NFC] ProfileTest class - formatting & general cleanup

Regenerate DAO files

Merge pull request #14295 from eileenmcnaughton/dao_2

[NFC] Add trailing comma to pseudoconstant array in DAOs

Merge pull request #14300 from colemanw/Fix

[REF] Fix pointless use of CRM_Utils_Array::value in Campaign BAO

Merge pull request #14294 from eileenmcnaughton/stds

[NFC] Fix variable types in DAO doc blocks

Merge pull request #14296 from eileenmcnaughton/bool

[NFC] fix docblock types from boolean to bool

Merge pull request #14297 from eileenmcnaughton/dedupe_cleanup

[code-cleanup] remove orderBy param from getDuplicatePairs, it is not used

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

dev/financial/issues/56, Fixed Financial item entries

Extract function to create nuanced custom fields to a trait

I want to write a test involving set up of multiple custom fields - this seems like the most
useful function for that, so moving to a trait for re-use

Fix pointless use of CRM_Utils_Array::value in Campaign BAO

Merge pull request #14283 from eileenmcnaughton/db_test3

dev/core#980 [dbunit] QueryTest Remove dbunit dependency on discontinued dbunit

Merge pull request #14291 from eileenmcnaughton/refund_ev

[NFC] array formatting, some comments

Merge pull request #14288 from MegaphoneJon/access-page-simplify

[REF] Simplify ACL page template

Merge pull request #14267 from eileenmcnaughton/setting_fin

dev/financial#54 Fix mishandling of deferred revenue settings

Merge pull request #14289 from eileenmcnaughton/event_dep

Fix deprecation notice on manage events page

[code-cleanup] remove orderBy param from getDuplicatePairs, it is not used

[NFC] fix docblock types from boolean to bool

This is the correct type name we should be using

Add trailing comma to pseudoconstant array in DAOs

Code change only includes sample - full regenerate will follow after merge

Fix variable types in DAO doc blocks

Phpcs (stricter) reports

  79 | ERROR   | [x] Expected "intunsigned" but found "int unsigned" for @var tag in member variable comment
 156 | ERROR   | [x] Expected "bool" but found "boolean" for @var tag in member variable comment

We are declaring int unsigned instead of int & boolean instead of bool

Note this commit only contains sample of resulting change - will add more complete set after

[NFC] ProfileTest class - formatting & general cleanup

[NFC] array formatting, some comments

Merge pull request #14285 from civicrm/5.14

5.14 to master

Fix deprecation notice on manage events page

[REF] Simplify ACL page template

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

Merge pull request #14277 from seamuslee001/dev_core_976

Remove htmlpurifier from composer and move it back to within IDS to f…

Merge pull request #12152 from JMAConsulting/dev_drupal_19

dev/drupal#19 Drupal8: Implement set UF locale/language (affects mailing tokens)