Merge pull request #19435 from civicrm/5.34

5.34

Merge pull request #19434 from alifrumin/rn-5.34

[NFC] 5.34 Release Notes: first pass

Merge pull request #19426 from colemanw/searchCount

dev/core#2312 SearchKit - Improve results loading time in admin UI

5.34 - Release Notes

Merge pull request #19429 from colemanw/searchJoinDefaults

dev/core#2313 SearchKit - Default join conditions

Merge pull request #19422 from mattwire/deprecatedcaseactivity

Flag that Embedded Activity Revisions setting is unsupported and deprecated

Merge pull request #19433 from totten/master-monaco-min

composer.json - Ignore unused variants of Monaco ("dev"/"esm")

Merge pull request #19430 from colemanw/select2Loading

Select2 - Improve styles for disabled, loading elements

Merge pull request #19432 from colemanw/afformApiCleanup

Afform - delete unused, nonfunctional apis

composer.json - Ignore unused from Monaco "dev"/"esm"

Saves ~50mb in the on-disk size of `bower_components`

Before
------

`bower_components` as a full developmental tree for monaco-editor

After
-----

`bower_components` only has the `min` variant of monaco-editor

Afform - delete unused, nonfunctional apis

Merge pull request #18782 from demeritcowboy/logging-binary2

dev/core#2114 - Changes in upper/lower case or accents are not logged when using trigger-based logging

Select2 - Improve styles for disabled, loading elements

Merge pull request #19424 from demeritcowboy/random-relationship

[Test framework] Prevent intermittently failing relationship test

Merge pull request #19423 from colemanw/tabTweak

Greenwich - fix list margin added by some CMS themes

SearchKit - Make UI less confusing by not allowing joins to be switched

Switching a join was confusing and buggy. Best to disable that and only allow them to be deleted

Merge pull request #19386 from totten/master-track-token

(dev/mail#81) Flexmailer - Track click-throughs for URLs with tokens

DAO - Update field labels for activity_contact & relationship_cache entities

https://lab.civicrm.org/dev/core/-/issues/2313

SearchKit - Default join conditions

Adds @ui_join_filters annotations to APIv4 entities which automatically shows that field when adding a join

https://lab.civicrm.org/dev/core/-/issues/2313

Merge pull request #19416 from eileenmcnaughton/dep

dev/financial#148 fully deprecate loadObjects function

Merge pull request #19427 from civicrm/dependabot/npm_and_yarn/socket.io-2.4.1

Bump socket.io from 2.3.0 to 2.4.1

Bump socket.io from 2.3.0 to 2.4.1

Bumps [socket.io](https://github.com/socketio/socket.io) from 2.3.0 to 2.4.1.
- [Release notes](https://github.com/socketio/socket.io/releases)
- [Changelog](https://github.com/socketio/socket.io/blob/2.4.1/CHANGELOG.md)
- [Commits](https://github.com/socketio/socket.io/compare/2.3.0...2.4.1)

Signed-off-by: dependabot[bot] <support@github.com>

SearchKit - Separate api calls for results & pager count in admin screen

https://lab.civicrm.org/dev/core/-/issues/2312

don't be random

Merge pull request #18779 from jaapjansma/dev_report_53

dev/report#53: search on relationship and case

Merge pull request #19395 from colemanw/afformTypes

Tabbed interface for organizing afforms by type

Bridge search displays and afforms

This allows an afform af-fieldset to function without a containing af-form or reference to af-entity
Instead it can directly contain an api-entities tag to assist filling field metadata.
The search extension then adds that tag based on the search parameters.
This also eliminates the need to pass in filters to the search display directly (though it still supports that),
as it will infer them from the presence of an af-fieldset.

Afform - convert afEntity and afField to components, standardize js filenames

Afform - Use new APIv4 format for option values

Afform - refactor admin page to use routing, add tabbed UI

Afform GUI - Rename service from afAdmin to afGiu

Afform - add option group for afform types

Greenwich - fix list margin added by some cms themes

Merge pull request #19419 from eileenmcnaughton/refs

[REF] Remove some of those old php4 &s

Merge pull request #19297 from eileenmcnaughton/pledge

Add v4 pledge api

Flag that Embedded Activity Revisions setting is unsupported and deprecated

Merge pull request #19420 from eileenmcnaughton/activity

Cleanup a couple of unnecessary references

Merge pull request #19418 from eileenmcnaughton/template

[REF] Move related code to be together

Cleanup a couple of unnecessary references

[REF] Remove some of those old php4 &s

I can't see any call to this that could be affected & I believe they are all well tested
In addition we can see that domain is not altered so it does not
need to be a reference in the 3 functions it is passed through

[REF] Move related code to be together

Puts both if (contactID) sections into the same if

dev/financial#148 fully deprecate loadObjects function

Merge pull request #19373 from eileenmcnaughton/valid

[REF] Simplify activity import validation

Merge pull request #19415 from seamuslee001/code_cleanup_dev_core_2307

[#REF] Cleanup recently added code fixing dev/core#2307 for the fact …

Merge pull request #19409 from eileenmcnaughton/template

dev/core#2303 Extract function to load the messageContent for a template

Merge pull request #19413 from christianwach/lab-core-2306

Fix rendering of Dashlet placeholder

[#REF] Cleanup recently added code fixing dev/core#2307 for the fact that  wasn't a variable before these lines

Merge pull request #19414 from spalmstr/Issue2307

Fix Issue 2307

Add in-code notes about the workflow_name

dev/core#2303 Extract function to load the messageContent for a template

This is a fairly straightforward extraction, seeking to break down the very-busy sendtemplate function
into the sub-functions within it. Essentially we load the template content, we swap out the tokens
and we send it.

Note that I have added a parameter to the output sent to alterMailContent
of 'workflow_name' which is more accurate - I will update the docs

[REF] Simplify activity import validation

There are 2 types of validation on importing
1) the correct fields have been mapped
2) the specific row has all the required fields

These generally live in different places
- field mapping is validated on the MapField::formRule validation function
- the specific row is validated in the (badly named) 'summary' function.

However, the second function is also doing some validation of the
mapping, checking that either activity_type_id or label is mapped
& ditto activity_date_time. This duplicates the MapField rule
and is hard to read, while making a larger cleanup difficult

Fix pledge to support v4 api

Add v4 pledge api

Fix Issue 2307

Fix rendering of Dashlet placeholder

Merge pull request #19402 from eileenmcnaughton/mem_form

[REF] Move instantiation of Order class to earlier in function

Merge pull request #19411 from eileenmcnaughton/deprecate_add

Deprecate Pledge::add() function

Deprecate Pledge::add() function

This function is no longer called from anywhere and we have settled on preferring 1 create function and
standardising on that. I have left the inner code there but added the deprecation notice

[REF] Move instantiation of Order class to earlier in function

This is part of making the Order object and it's methods available to the non-renewal
form which has the same amount calculation bug the class is used to fix.

Note the Order class is an internal class not the order api which we do wish to switch this
form to using

Merge pull request #19390 from civicrm/php_version_bump

dev/core#2204 Bump Minimum install PHP 7.2

Merge pull request #19400 from eileenmcnaughton/pledge_ref

Fix pledge to not use pass-by-reference

Merge pull request #19403 from eileenmcnaughton/mem_order

[REF] Fix Order class to not reload if already loaded

Merge pull request #19407 from eileenmcnaughton/anet

Remove call to loadRelatedObjects

Merge pull request #19410 from eileenmcnaughton/load

[REF] Move code closer to where it is used

Merge pull request #19408 from eileenmcnaughton/old_params

[NFC] Whitespace fix up

[REF] Move code closer to where it is used

Simply moves a few lines of code so the actual usage is apparent

Merge pull request #19404 from eileenmcnaughton/mem_deb

[Test] Extend new membership form to cover multi-line renew + minor cleanup

Merge pull request #19406 from eileenmcnaughton/param

[REF] Param simplification

Fix pledge to not use pass-by-reference

We have deprecated this out of BAO classes

[NFC] Whitespace fix up

Merge pull request #19401 from eileenmcnaughton/mem_test

[NFC] Preliminary cleanup

Merge pull request #19405 from eileenmcnaughton/inv

[REF] Minor cleanup on emailReceipt

Remove call to loadRelatedObjects

None of the ids loaded in this function are used anymore so we can remove the call to it
The only reference to ids after this line is to contribution page id, which is loaded earlier

[REF] Param simplification

[REF] Param simplification

On looking at this param it can only be reached if recur->id is set and the other criteria can be
squashed to an empty check

[REF] Minor cleanup on emailReceipt

Fixes 2 things
1) empty check on line items when it is never set
2) starts to move code out of the shared-function to the form function
when it is only possible when called via the form function. The sharing here
is one of those cases where only a small part of the shared code is actually shared

[REF] Minor cleanup

[Test] Extend new membership form to cover multi-line renew

[REF] Fix Order class to not reload if already loaded

[NFC] Preliminary cleanup

Merge pull request #19399 from seamuslee001/mysql8_rlike

REF Ensure that when doing RLIKE BINARY the field is cast as BINARY t…

REF Ensure that when doing RLIKE BINARY the field is cast as BINARY to prevent issue on MySQL 8

Merge pull request #19393 from civicrm/5.34

5.34

Merge pull request #19398 from seamuslee001/5.34

Add release-notes/5.33.2

Add release-notes/5.33.2

Merge pull request #19384 from totten/5.34-distmaker-wpver

(dev/wordpress#85) Fix version number reported within WordPress admin UI

(dev/wordpress#85) Fix version number reported within WordPress admin UI

Merge pull request #19388 from demeritcowboy/nulls-all-around

dev/core#2297 - E_NOTICE on contribution thankyou_date_time since time component no longer a separate field

Merge pull request #19383 from eileenmcnaughton/civi_sett

Add setting default to speed up cache flush

Add setting default to speed up cache flush

Detailed discussion in https://lab.civicrm.org/dev/core/-/issues/2031 but note
this does not add vendor in to make it easier to get it merged (as
there is more complexity there

Merge pull request #19339 from kainuk/dev_core_2288_sensible

dev/core#2288 - Alternative fix search range for select/radio custom fields - now disabling the combination

Merge pull request #19382 from totten/master-maxfile

"Admin => Misc" - Fix validation of "Maximum File Size"

Update Miscellaneous.php

Merge pull request #19352 from colemanw/afformRef

Afform - code refactoring for angular best practices

Bump Minimum install PHP 7.2

Merge pull request #19385 from civicrm/5.34

5.34

time component no longer a field

Simplify tracking of URLs

In the prior commit, it needed to conceptually change the behavior of
`getTrackerURL($oldURL): $trackedURL`, but it danced around it (adding
more classes and similar-sounding-methods, but distributed elsewhere).

This re-consolidates to keep a cleaner separation of responsibilities
between `{Html,Text}ClickTracker.php` (*parsing an HTML or text message for URLS*)
and `TrackableURL.php` (*translating the URL*).

Enable tracking of urls with tokens in Flexmailer

(NFC) BaseMailingSystemTest - Tweak debug message for assertion