Merge pull request #18511 from totten/master-aff-test

Afform Tests - Fix extension tests when run via `civi-test-run`

Merge pull request #18506 from totten/master-bundle-array

CRM_Core_Resources::addBundle() - Fix handling of array inputs

Merge pull request #18507 from eileenmcnaughton/test_fix

Test fix - use valid membership type

afform_clear() - Fix error when running on non-cached container

After switching the tests from an effective policy of `CIVICRM_CONTAINER_CACHE=auto`
to an effective policy of `CIVICRM_CONTAINER_CACHE=never`, this started to fail:

```
1) api_v4_AfformTest::testGetUpdateRevert with data set #0 ('mockPage', array('', '', 'civicrm/mock-page', 'access Foobar'))
Symfony\Component\DependencyInjection\Exception\BadMethodCallException: Setting service "angular" for an unknown or non-synthetic service definition on a compiled container is not allowed.

/home/me/bknix/build/dmaster/web/sites/all/modules/civicrm/vendor/symfony/dependency-injection/ContainerBuilder.php:532
/home/me/bknix/build/dmaster/web/sites/all/modules/civicrm/ext/afform/core/afform.php:520
/home/me/bknix/build/dmaster/web/sites/all/modules/civicrm/ext/afform/core/Civi/Api4/Afform.php:99
/home/me/bknix/build/dmaster/web/sites/all/modules/civicrm/Civi/Api4/Generic/BasicBatchAction.php:88
/home/me/bknix/build/dmaster/web/sites/all/modules/civicrm/Civi/Api4/Generic/BasicBatchAction.php:68
/home/me/bknix/build/dmaster/web/sites/all/modules/civicrm/Civi/Api4/Provider/ActionObjectProvider.php:68
/home/me/bknix/build/dmaster/web/sites/all/modules/civicrm/Civi/API/Kernel.php:150
/home/me/bknix/build/dmaster/web/sites/all/modules/civicrm/Civi/Api4/Generic/AbstractAction.php:238
/home/me/bknix/build/dmaster/web/sites/all/modules/civicrm/ext/afform/mock/tests/phpunit/api/v4/AfformTest.php:53
/home/me/bknix/extern/phpunit6/phpunit6.phar:570
```

Container - Disable caching in as many unit-tests as we can

Merge pull request #18503 from colemanw/removeHookCiviCRMTabs

Remove long-deprecated hook_civicrm_tabs

Merge pull request #18488 from eileenmcnaughton/loc_block

dev/core#1973 Fix Email & Phone storage issues in event location

Test fix - use valid membership type

Addresses cause of https://github.com/civicrm/civicrm-core/pull/18427 fail

CRM_Core_Resources::addBundle() - Fix handling of array inputs

(This fixes a small bug in a new function added during this release -- part of #18247.)

The signature of `addBundle()` optionally accepts an array/iterable -- if
given, then it should add all the items from the array. For example:

```php
Civi::resources()->addBundle(['foo', 'bar']);
```

Before
------

It adds `foo` but then bails out on `bar`.

After
-----

It adds both `foo` and `bar`.

Merge pull request #18501 from eileenmcnaughton/acltest

dev/core#2039 Fix test to use set up method with more data integrity

dev/core#1973 Fix Email & Phone storage issues in event location

Merge pull request #18492 from colemanw/moveDaoFunctionsToBaseClass

Ensure DAO base class contains functions to be removed from generated files

Merge pull request #18490 from eileenmcnaughton/emailadd

Remove deprecated code lines

Remove deprecated hook_civicrm_tabs

This hook has been deprecated for 5 years. Time to remove it.

Merge pull request #18498 from eileenmcnaughton/opennp

dev/core#2039 Fix OpenID::add to ensure is_primary is set

dev/core#2039 Fix test to use set up method with more data integrity

Using the api rather than CRM_Core_DAO::createTestObject to create the phone ensures it is created with is_primary.

We might need to fix CRM_Core_DAO::createTestObject but for this case simply using the api suffices

dev/core#2039 Fix OpenID::add to ensure is_primary is set

In my efforts to determine the the non-performant line of code that makes sure IM is set is
not required I edited tests to track down when is_primary is not correct & determined it
is missing from the add function - same as https://github.com/civicrm/civicrm-core/pull/18489 for IM

Merge pull request #18489 from eileenmcnaughton/im

dev/core#2039 Add check to ensure all have a primary address, fix identified but in IM.add

Merge pull request #18497 from eileenmcnaughton/open_primary

dev/core#2039 [NFC] Code cleanup in test class

Merge pull request #18496 from eileenmcnaughton/nfc

dev/core#2039 [NFC] Minor cleanup in tests

Merge pull request #18495 from eileenmcnaughton/email_add

dev/core#2046 Rationalise add vs create on email BAO

[NFC] Code cleanup in test class

Merge pull request #18494 from eileenmcnaughton/test_clean

dev/core#2039 [NFC] Test cleanup to stop calling Email::add action

[NFC] Minor cleanup in tests

Merge pull request #18484 from eileenmcnaughton/ref

dev/core#2043 remove pass-by-ref

Merge pull request #18491 from totten/master-ref-gendata

(REF) GenerateData - Make it possible to call this via PHP

dev/core#2046 Rationalise add vs create on email BAO

This folds the 2 functions into one, deprecating add

[NFC] Test cleanup to stop calling Email::add action which does not handle primary

Note I feel like we should further
1) move the contents of email add into create
2) make add a deprecated wrapper for create

Whatever the vision of different add vs create functions it didn't turn out in practice and seems to
only generate confusion. We recommend apiv4 as the preferred create methodology

Ensure DAO base class contains functions to be removed from generated DAO files.

The functions fieldKeys(), getLog(), and getTableName() are redundant in the generated DAO files,
we don't need 50 copies of each function.

This preliminary commit ensures they are present in the parent class, and in the future we can
regenerate the DAOs without them.

dev/core#2043 remove pass-by-ref

Also shortening of if clause

Merge pull request #18470 from JKingsnorth/dev/core#2027-2

dev/core#2027 Add/update to UK county list

Merge pull request #18423 from eileenmcnaughton/afform

dev/core#2000 Move afform into core extensions

dev/core#2027 Add/update to UK county list

(REF) GenerateData - Allow external control randomness

(REF) GenerateData - Use overrideable 'echo'. Use exceptions.

This removes elements of GenerateData which perform direct output to the user. there
are basically two cases:

1. Outputting advisory/status info (e.g. "echo"). This now goes through a method, which
   can be overriden/replaced.

2. Displaying errors from DAO-calls. Now-a-days, DAO-calls can raise their own exceptions,
   so we don't need to sprinkle it without error-outputs.

(REF) GenerateData - Extract method 'generateAll()'

GenerateData - Make the generated sample data more stable (wrt shuffle+array_rand)

GenerateData - Make the generated sample data more stable (wrt time)

GenerateData - Make the generated sample data more stable (wrt mt_rand)

(REF) GenerateData - Extract method 'randomInt()'

This it easier to have one random number generator used for all 'randomFoo()' functions.

(REF) Extract `CRM_Core_CodeGen_GenerateData` - Fix path reference

When this class moved to a new folder, the relative paths changed.

Remove deprecated code lines

Add check to ensure all have a primary address, fix identified but in IM.add

Merge pull request #18486 from eileenmcnaughton/part

dev/event#42 [Ref] Separate url path calculation into the relevant classes

(REF) Extract `CRM_Core_CodeGen_GenerateData`

This moves the class from `sql/GenerateData.php` to `CRM/Core/CodeGen/GenerateData.php`, which makes
it easier to reference in other workflows.

Merge pull request #18480 from eileenmcnaughton/dom_org

dev/core#2039 Set is_primary to 1 on email, address for domain contacts

Merge pull request #18487 from civicrm/5.30

5.30

[Ref] Separate url path calculation into the relevant classes

Merge pull request #18482 from seamuslee001/5.30

release-notes/5.29.1.md

Merge pull request #18478 from seamuslee001/api_v4_fin_acl_test

[NFC] Enable APIv4 testing on the Fin ACL Extension Line Item test

Merge pull request #18464 from eileenmcnaughton/urlparamms

[REF] Start the process of separating the search action from the participant form

Merge pull request #18463 from eileenmcnaughton/searchhhook

dev/core#2017 remove unused SearchTaskHookSample.php

Regenerate sql

dev/core#2039 Set is_primary to 1 on email, address for domain contacts

I'm digging into the places where the code handles is_primary due to duplicate handling/ queries
- see https://lab.civicrm.org/dev/core/-/issues/2039

One distraction is that the domain contact's address & email are not marked is_primary. This
doesn't make sense to me as they ARE a contact in the DB

Phpcs fix

Remove commented out line

Remove more commented out code that is upsetting CI

Remove more commented out function

Style fixes

I removed the commented out functions rather than figuring out how to style them right

Misc style fix

Re-run civix upgrader

Re-run civix

Update API factory functions with checkPermissions

Use standard api for getFields

Update API docblock.

Use built-in php function to do what we were trying to do with str_replace

Code comment

Revert "Reinstate parsing of data from the form"

This reverts commit d31206f9e08703ae4f11b7e34673d7ece92b1630.

Support is_primary fields and boolean options.

Civi 5.23 changes boolean options to be real true and false instead of '1' and '0'
so added version bump to keep in step.

More efficient select options handling

The afField.getOptions function was causing infinite recursion by recreating the boolean options array every time.
Passing this variable by refernce solves the problem, and adding "track by" to the ng-repeat follows best-practices for efficiency.

Reinstate parsing of data from the form

This means that data can be set in an embedded form e.g

 <af-entity data='{name_a : options.name_a,  name_b : options.name_b, is_name_b_nickname: options.is_name_b_nickname, is_name_b_inferior: options.is_name_b_inferior}' .... />

In this case the options.name_as comes from the directive that instantiates this afform

Increment suggested version to reflect info.xml

Arguably afform still works on older versions but breaking changes have happened & if people
use it on older you want a buyer-beware type feeling

Release v0.4

Loosen restrictions on file names - allow underscores

Use api to get directive/module names

Rename custom blocks and optimize api get by file/module/directive name

Api - output module_name & directive_name

Load join blocks primary first

GUI - improve menu button appearance

GUI - simplify menus; only add items from palette

Improve drag-n-drop placeholder & validation

Fix loading fields for contact type blocks

Rename blocks & require form name

GUI - Add block fields to field palette

GUI - Implement Save Block feature

Implement Address block with chainSelect state/county

GUI - present Individual/Organization/Household as seperate entities

Also adds per-contact-type default name blocks.

Refactor af-block directive into af-repeat and af-join

This allows mix-n-match between blocks and repeatable things, allowing
entity fieldsets to be repeated, as well as arbitrary non-repeatable blocks.

Implement afform blocks for multi-value custom groups

Implement afform blocks for Email, Phone, IM, Website

Blocks are reusable, repeatable afforms that extend other entities

Update git clone url

Note other updates around installation still needed

Fix #10 add prefix to new afforms to prevent name conflicts

Remove unnecessary registerField process

GUI - set default permission for new forms

Update docs to reflect less metadata in default afform scope

AfformScanner - Just use the long cache

The long cache didn't exist when this class was first created, but it seems
better suited. #knockonwood

AfformScanner - Simplify cache. Fix CLI/web sync bug.

Example bug:

1. Find a packaged (non-overriden) form - e.g. `mockPage`
2. Run `cv api4 Afform.update +w name=mockPage +v permissions='*always allow*'
3. Run `curl http://localhost/civicrm/mock-page`
4. Observe: the page-load fails even though the permissions say it should work

The call to `Afform.update` should trigger a cache-clear, but it only fixes
on the CLI...  because the CLI and web have different runtime IDs.

The runtime-id was included preventively in anticipation that oddball
multisite arrangements might need to store these things separately.  But I
think it's overdone, because the runtime-ID factors in things like
`SCRIPT_FILENAME` which has no bearing on the whether to use the same cache.

Let's KISS and *if* there's any kind of multisite issue, then we can revisit.

Toward #6 - use API instead of scanner to retrieve afforms

GUI - dropdown select for permissions field

(NFC) Expand docblock for afform_civicrm_permission_check()