Merge pull request #20906 from eileenmcnaughton/api4

Add test & fix for failing OR clause in apv4

Merge pull request #20900 from colemanw/angularTraits

[REF] Refactor SearchKit displays to inherit traits from a common base

Add test for failing OR clause

This fails because when the field is being concatenated like
`a`.`first_name`='x' OR`a`.`last_name`='x'

The lack of a space between OR and `a` is fugly but it
is parsed by mysql. However, when the value requires utf8mb4
to be supported to do the comparison it returns '0=1' if the
database does not support utf8mb (or it thinks it doesn't per

0 = 1 OR0 = 1

Merge pull request #20870 from totten/master-tokensmarty

(REF) Extract TokenSmarty::render() from MessageTemplate::renderMessageTemplate()

Merge pull request #20895 from demeritcowboy/test-upper

[NFC/Unit test] - Test for #20892 - New smarty plugin crmUpper

Merge pull request #20887 from demeritcowboy/lazytests

[NFC/Unit test] ReportTemplateTest - fix tests that assume setup from a separate test

Merge pull request #20892 from seamuslee001/utf8mb4_smarty_upper

[REF] Add in smarty modifier to replcae the upper smarty modifier to …

Merge pull request #20902 from colemanw/testNoNewOptionGroups

APIv4 - Add test to ensure superfluous option groups are not created

Merge pull request #20901 from demeritcowboy/fix-api4test

[NFC/Unit test] dev/core#2699 - Fix intermittent api4 test (part 1)

APIv4 - Add test to ensure superfluous option groups are not created

fix intermittent api4 test

Refactor SearchKit displays to inherit traits from a common base

This refactors searchDisplayUtils to searchDisplayBaseTrait service,
which acts like a PHP trait, controllers use it via angular.extend().

Merge pull request #20899 from mattwire/recaptchanotice

Fix PHP notice in recaptcha lib

Fix PHP notice in recaptcha

Merge pull request #20897 from yashodha/typo-fix

some typo fixes

Merge pull request #20875 from totten/master-field-spec

(REF) APIv4 FieldSpec - Extract various traits (Civi\Schema\Traits\*)

Merge pull request #20601 from kartik1000/pcpshortcode

Added setup pcp page wordpress shortcode

some typo fixes

Merge pull request #20869 from eileenmcnaughton/acl3

[REF] Fix functions to be protected

Merge pull request #20880 from colemanw/searchKitTokenSelect

SearchKit - Support all fields as tokens

Merge pull request #20893 from colemanw/fixAPIComparisonFunctions

APIv4 - Fix and add tests for comparison SQL functions

Merge pull request #20896 from colemanw/groupConcatOutputValue

APIv4 - Better handling of output value using GROUP_CONCAT

SearchKit - Allow tokens in link text

SearchKit - Support all fields as tokens

Previously, only fields present in the SELECT clause could be tokens.
Now the SearchDisplay::Run api will add any fields used as tokens to the SELECT automatically.

APIv4 Explorer - Fix display of bridge entity fields for symmetric bridge

Symmetric bridge entities such as RelationshipCache which joins Contact to Contact
need to have the second entity id availble to join on.

APIv4 - Validate number of args passed to SQL functions

APIv4 - Better handling of output value using GROUP_CONCAT

Merge pull request #20894 from civicrm/5.40

5.40

test upper

Merge pull request #20891 from demeritcowboy/timestamp-update

[NFC/Unit test] - Failing test demonstrating blank timestamp updates

APIv4 - Fix and add tests for comparison SQL functions

Merge pull request #20890 from demeritcowboy/ignore-log-upgrade-alt

dev/core#2666 - (Alternate) Don't repeatedly log about crm-l10n.js during upgrade

Merge pull request #20865 from totten/master-getset-trait

(REF) Civi/Schema - Extract MagicGetterSetterTrait. Add test coverage.

[REF] Add in smarty modifier to replcae the upper smarty modifier to better handle umlouts / accents

(NFC) AbstractAction - Pithiness

failing test for timestamp updates

don't log about missing l10n for every upgrade task

fix tests that assume setup from a separate test

Merge pull request #20850 from eileenmcnaughton/int

Use type hinting for id

Merge pull request #20848 from eileenmcnaughton/dep

Remove deprecated code

Merge pull request #20853 from eileenmcnaughton/auth

Remove never-true-if

Civi/Schema - Extract MagicGetterSetterTrait. Add test coverage.

Merge pull request #20881 from eileenmcnaughton/batch2

Remove non-variable variables

Merge pull request #20879 from colemanw/fixOnCrmUiSelect

Fix onCrmUiSelect to use current scope and apply to the digest cycle

Merge pull request #20878 from colemanw/fixSearchKitCount

SearchKit - Fix pager count return value

Remove non-variable variables

Fix onCrmUiSelect to use current scope and apply to the digest cycle

Evaluates expression in current scope rather than parent scope,
and uses $scope.$apply.

Merge pull request #20871 from eileenmcnaughton/val

[Ref] Remove never-passed parameter

SearchKit - Fix pager count return value

Merge pull request #20874 from eileenmcnaughton/batch

[REF] Remove some non-variable variables

Merge pull request #20864 from eileenmcnaughton/tok

dev/core#2650 Add support for contribution_status_id to the token processor

Merge pull request #20801 from eileenmcnaughton/upg

Enotice fix on extensions page

Merge pull request #20873 from eileenmcnaughton/remove_dep

Remove deprecated lines of code

dev/core#2650 Add support for contribution_status_id to the processor

Per https://lab.civicrm.org/dev/core/-/issues/2650 the goal is to get the
token processor (for each entity, in this case contributions) to be processing fields
in the same way as the legacy processor does and for this to be thoroughly tested.

This allows us to expose the token processor classes to message template processing
and use them interchangeably, with an eventual goal of migrating over.

It turns out that althought 'Contribution Status ID' is exposed in the UI
for scheduled reminders it did not work. This left us open to simply add the
token contribution.contribution_status_id and add tests.

Per 2650 the handling of pseudoconstants is inconsistent but our preferred goal
is that they would follow apiv4 style syntax and that the actual field name
would hold the actual field value.

Although the token processor exposes a value 'status' - it seems this is likely never
actually exposed to the user and is probably unused.

We can come back to how to deprecate & remove

Merge pull request #20876 from eileenmcnaughton/cust

Api test cleanup fixes

Api test cleanup fixes

[REF] Remove some non-variable variables

This previously shared function has parameters that are hard set to a specific value.
Most notably pending is set to FALSE - this removes all references to pending from the function

(REF) ConformanceTest - Ensure that field-specs are consistent when converting to/from array

Schema - Add PhpDataTypeSpecTrait

(NFC) FieldSpec - Docblock for $type

(REF) APIv4 FieldSpec - Extract ArrayFormatTrait. Add loadArray().

(REF) APIv4 FieldSpec - Extract OptionsSpecTrait (options, optionsCallback)

(REF) APIv4 FieldSpec - Extract DataTypeSpecTrait (dataType, serialize, fkEntity)

Also: Update `setSerialize()` to accept types a string.  This is useful if
the info comes from an annotation.

(REF) APIv4 FieldSpec - Extract SqlSpecTrait (tableName, columnName, operators, sqlRenderer, sqlFilters)

(REF) APIv4 FieldSpec - Extract GuiSpecTrait (label, inputType, inputAttrs, helpPre, helpPost)

(REF) APIv4 FieldSpec - Extract BasicSpecTrait (name, title, description)

Merge pull request #20791 from eileenmcnaughton/batch_renew

[REF] Simplify isRenew handling on batch for membership

Remove deprecated lines of code

[Ref] Remove never-passed parameter

this function is called from only one place & it does not pass in financialTrxnValues

Merge pull request #20820 from eileenmcnaughton/tpl

dev/core#2684 Fix filters tpl in civi reports to permit more than one  table in a grouping

(REF) Extract TokenSmarty::render() from MessageTemplate::renderMessageTemplate()

Most Civi message-templates have been written with a hybrid notation based on
mixing tokens (eg `{contact.first_name}`) and Smarty (eg `{$foo}` or
`{if}{/if}`).  The notion here is to acknowledge that Token-Smarty is a
distinct/de-facto templating language and provide the kind of `render()` API
that you would expect from any templating language (i.e.  "Give me your
template, give me your data, and let me return to you a string!").

```php
$rendered = CRM_Core_TokenSmarty::render($template, $tokenData, $smartyData);
```

None of this is new functionality.  It's just a refactoring which extracts
code from `renderMessageTemplate()`, hardens it a bit more, and adds some more
testing.

This is a step toward removing `renderMessageTemplate()`.  The problem with
`renderMessageTemplate()` is that only handles `$contactID` -- you can't
pass-through other data to the token layer.

Before
------

* Support `CRM_Core_BAO_MessageTemplate::renderMessageTemplate()`, which accepts
  `bool $disableSmarty` and `int $contactID`.
* There is no way to pass other IDs (e.g. activity and contribution IDs) to the
  token-processor.

After
-----

* Support `CRM_Core_TokenSmarty::render()`. This largely the same as `renderMessageTemplate()`, except that:
    * You're not specifically tied to `subject` or `text` as the template names. The list of message-templates will accept/preserve whatever keys you input (e.g. give it `html`/`text` for current compatibility; or give `msg_html`/`msg_text` to match actual DB fields)
    * `bool $disableSmarty` and `int $contactID` are combined into one `array $tokenContext`, which can pass through more fields
    * If there's an exception during process, `Smarty->pop()` will still do cleanup (`try/finally`).
    * `Smarty->push()` and `Smarty->pop()` only run if needed
* Add test coverage from some of the peculiar ways these notations are mixed.

[REF] Fix functions to be protected

I did some poking around & these function are not called outside the class

Once we can see that we can consider updating to save 0 to the table, allowing
a lot of simplification

Merge pull request #20829 from eileenmcnaughton/order

[REF] Further order api cleanup

Merge pull request #20863 from eileenmcnaughton/api4

Fix v4 api to do custom field delete in tearDown

Fix v4 api to do custom field delete in tearDown

This existing efforts are wrong because
1) the delete has to go in the tearDown not the clean up
2) it is rigid about the custom group table name - which
will get us into problems as the name can be set and we
should be ensuring that it can in our tests

Merge pull request #20578 from eileenmcnaughton/admin_data

Give administer CiviCRM data access to administer custom groups

dev/core#2684 Fix filters tpl in civi reports to permit more than one table in a grouping

Merge pull request #20860 from civicrm/5.40

5.40 to master

Merge pull request #20859 from agh1/5.40.0-releasenotes-initial

5.40.0 release notes initial run

5.40.0 release notes: added boilerplate

5.40.0 release notes: raw from script

Merge pull request #20840 from colemanw/membershipTypeAutoRenewFix

APIv4 - Fix output of membershipType auto_renew field

Merge pull request #20852 from eileenmcnaughton/paypal

[REF] [towards dev/core#2693] Use getter rather than passing variable

Merge pull request #20849 from civicrm/5.40

5.40

Remove never-true-if

This code is in the recur() function. the lines immediately before this function ensure contribution->contribution_status_id
is unset if it is 1 - meaning this if can never be true

Merge pull request #20847 from eileenmcnaughton/ref

Minor simplification - use casting

Merge pull request #20718 from eileenmcnaughton/ipn

[REF] Minor auth.net ipn simplification

Merge pull request #20844 from demeritcowboy/report-rectype

dev/core#2691 - On logging detail civireport show words instead of numbers

[REF] Use getter rather than passing variable

txnType is a required parameter for recurring & unused for single. This switches to retrieving
it as needed rather than passing it around. It removes one check for whether it is
set because the abort will exit if not

Use type hinting for id

This clarifies that the id must be an integer (0 for anonymous)

Function is only called from one place

Merge pull request #20846 from colemanw/fixCkEditorExtension

CKEditor - Fix .tpl insertion on every form

Remove deprecated code

We deprecated NOT passing an id into the function in 2018 - this removes

Note I double checked - if you pass in '2' to a strict casting of int
it accepts it and casts to int

Minor simplification

Merge pull request #20843 from demeritcowboy/smartyreptable

Smarty notices - Missing type on logging civireport

Merge pull request #20768 from eileenmcnaughton/mem_type_acl

dev/core#2115 Switch membership type browse page to use v4api for get, thus using acls properly

CKEditor - Fix .tpl insertion on every form

Merge pull request #20842 from in2part/add-to-contributors-file

Add to contributors file

Merge pull request #20821 from colemanw/afformBeta

Mark afform extensions as beta

APIv4 - Fix output of membershipType auto_renew field.

The field had been declared a "boolean" in the schema, which at a SQL level
is exactly the same as "tinyint" but it was resulting in incorrect API output.

show words instead of numbers