Merge pull request #28868 from civicrm/5.69

5.69 to master

Merge pull request #28867 from eileenmcnaughton/report_move

Move report test to civi_report

Merge pull request #28722 from eileenmcnaughton/auto_renew

CiviMember - Fix checkbox for optional auto-renewals

Move report test to civi_report

Merge pull request #28866 from eileenmcnaughton/report_instance

Minor cleanup in ReportInstance BAO

Merge pull request #28641 from colemanw/fullScreenDashlet

Dashboard Fullscreen dashlet improvements

Merge pull request #28865 from civicrm/5.69

5.69

Merge pull request #28860 from colemanw/deprecateDate

CoreForm - Add noisy deprecation to addDate & addDateTime functions

Merge pull request #28861 from seamuslee001/grant_report_recieved_fixed

[REF] Fix grant_report_received column to be not null and default to 0

Merge pull request #28769 from eileenmcnaughton/amountu

Declare amount private - use magic method to provide access to the old value

Merge pull request #28850 from colemanw/fkColumn

APIv4 - Add fk_column to getFields metadata

Merge pull request #28862 from seamuslee001/grant_search_notices

[REF] Fix Smarty Notices on Find Grants search

Minor cleanup in ReportInstance BAO

Merge pull request #28848 from eileenmcnaughton/notice

Fix notice in workflow message examples when no price set present

Merge pull request #28847 from eileenmcnaughton/date_non_us_time_2

Fix time handling for dd_mon_yy format date strings

[REF] Fix grant_report_received column to be not null and default to 0

Fix form handling

APIv4 - Add fk_column to getFields metadata

Before: getFields contained incomplete info about foreign keys, so trying to map the schema was unnecessarily difficult.

After: No need to use extra DAO methods like getReferenceColumns() to find out about FKs. All the info is returned by getFields.

[REF] Fix Smarty Notices on Find Grants search

CoreForm - Add noisy deprecation to addDate & addDateTime functions

Merge pull request #28859 from eileenmcnaughton/table

Fix notice on civi-report display

Merge pull request #28819 from eileenmcnaughton/no_line

(dev/core#4877) CiviContribute - Re-calculate prices when using "Go Back"

Merge pull request #28820 from eileenmcnaughton/amount

CiviContribute - Do not delete price_set_entity record while page supports payment

Merge pull request #28858 from eileenmcnaughton/no_array

dev/core#4881 Do not fatal if params is not an array

Fix notice on civi-report display

Merge pull request #28757 from totten/master-oauth2-google

(dev/core#3581) composer.json - Allow newer version of oauth2-google

Reload submitted line items on re-submit of Main form

(dev/core#3581) composer.json - Allow newer version of oauth2-google

I looked at the diffs for v3.0.4 and 4.0.0.

* They raised the requirements to PHP 7.3/8.0 and added some type-hints.
* They raised the test-env to PHPUnit 8/9
* They updated some metadata to say all that.
* The main contracts are provided by `league/oauth2-client` -- which
  hasn't changed. It's still the same v2.x requirement.
* I don't believe that Civi directly extends/overrides any of the Google Auth stuff.

Merge pull request #28519 from MegaphoneJon/pcp-enumerate

[REF] Rewrite PCP enumeration to avoid caching bug

Do not fatal if params is not an array

Fix autorenew checkbox with memberships

Merge pull request #28854 from alifrumin/5.69lc

[NFC] 5.69 release note late changes

Merge pull request #28857 from civicrm/5.69

5.69 to master

Merge pull request #28856 from eileenmcnaughton/569

Correct jcalendar.tpl

Correct jcalendar.tpl

Merge pull request #28851 from colemanw/ruleGroupTestCleanup

[NFC] Cleanup dummy DAO in ruleGroupTest class

Merge pull request #28852 from colemanw/checkAccessHelp

[NFC] Fix docblock reference in _checkAccess functions

[NFC] 5.69 release note late changes

Merge pull request #28540 from twomice/3912_sk_manage_premiums

Add "Manage Premiums" to AdminUI, toward  dev/core#3912 .

Fix docblock reference in _checkAccess functions

Cleanup dummy DAO in ruleGroupTest class

The getReferenceColumns() is no longer needed and can just use the parent function.

Update preview path in Product.xml

Merge pull request #28849 from eileenmcnaughton/test_clean

Minor cleanup in test class

Fix time handling for dd_mon_yy format

Standardise processor set up, variable usage

Minor cleanup in test class

Fix notice in workflow messages when no price set present

Handle time in dd_mm_yyyy format

Merge pull request #28846 from colemanw/extReferenceColumns

Regenerate ext DAO files using updated civix

Merge pull request #28844 from eileenmcnaughton/date_non_us

Fix handling of non-padded day in import date in dd/mm/yyy format

Regenerate ext DAO files using updated civix

Uses the update from https://github.com/totten/civix/pull/328 to add FKColumnName metadata.

Merge pull request #28843 from civicrm/5.69

5.69

Merge pull request #28827 from colemanw/getReferenceColumns

DAO - Return reference columns based on field metadata

Fix handling of non-padded month in dd_mon_YY format

Fix time handling for us-style

Add consistent time-format support for US formats, add test for our magic year-handling

Merge pull request #28842 from MegaphoneJon/price-field-weights

Price field and value weights not respected

Merge pull request #28647 from eileenmcnaughton/delete

Move implementation of financial type acl out of core (leverage existing extension hooks)

fix price field and value weights

Remove duplicate check

Do not call core function from extension

Move financial type check from event register page

Move link handling to extension

Move check permissioned line items to acl now it is not used in core

Move implementation of financial type acl out of core (leverage existing extension hooks)

Merge pull request #28668 from colemanw/utilsArrayCollect

REF - Swap CRM_Utils_Array::collect with vanilla array_column

Merge pull request #28828 from eileenmcnaughton/date_iso

Fix time validation for iso date format

Merge pull request #28834 from colemanw/phpStormGetLinks

[NFC] Add 'getLinks' action to phpStorm extension

Merge pull request #28837 from colemanw/group_notice2

Fix e-notice on contact create for low-permissioned user

Merge pull request #28786 from colemanw/oauthLinks

dev/core#4674 - Add Oauth links for AdminUi MailSettings

Merge pull request #28838 from spalmstr/Issue_4879

Issue 4879 second attempt

Following suggested additional changes.

Fix e-notice on contact create for low-permissioned user

This cleans up some very messy code which was emitting e-notices when the 'group' key was missing.
This can happen when the current user does not have permissions to add contacts to groups.

Issue 4879

Merge pull request #28831 from eileenmcnaughton/template_vars_sample

Remove smarty2 functions from sample code

Merge pull request #28829 from eileenmcnaughton/template_vars

Use getTemplateVars in smarty help function

Merge pull request #28830 from eileenmcnaughton/template_smarty

Replace getTemplateVars() in test, assignTemplateVariables

Merge pull request #28621 from artfulrobot/issue-4857

standalone: implement cms-prefixed permissions as concrete ones

Merge pull request #28833 from artfulrobot/standaloneusers-dependency

Remove missingStandaloneExtension() checks which were pre-alpha scaffold

Add 'getLinks' action to phpStorm extension

Remove missingStandaloneExtension() checks which were pre-alpha scaffold

Merge pull request #28784 from mlutfy/floatLeftButton

Merging as no objections

Remove smarty2 functions from sample code

Replace getTemplateVars() in test, assignTemplateVariables

Use getTeplateVars in smarty help function

Combine convertToDefaultDate back into only calling function

It is not a logical separation of concerns & makes code more complex
rather than simpler

Fix time validation for iso date format

As this is a very simple format strtotime will get it right
so we just fix our validation & simplify

Merge pull request #28826 from eileenmcnaughton/act_workflow

Use distinct test workflow message for workflow message tests

Disable check temporily for PR juggling

dev/core#4674 - Add Oauth links for AdminUi MailSettings

This fixes https://lab.civicrm.org/dev/core/-/issues/4674 by adding a LinksProvider
for the new GetLinks action from #27973.

It uses the CRM_Core_BAO_MailSettings::getSetupActions() function to generate the extra links,
but there is a mismatch between the two functions:

- getSetupActions() returns a fully-formed url
- API::getLinks() returns a path

However, I noticed that while in theory hook_civicrm_mailSetupActions could return various urls,
in practice they are all the same: a generic civicrm/ajax/setupMailAccount redirect.
So this returns that as the path and it works.

Perhaps we should deprecate the option to return a url from hook_civicrm_mailSetupActions
(or just remove it, since it appears to be unused & YAGNI).

Regenerate Ext DAO files without boilerplate getReferenceColumns()

Regenerate Core DAO files without boilerplate getReferenceColumns()

DAO - Return reference columns based on field metadata

This deletes getReferenceColumns() from the DAO boilerplate and adds all the necessary field metadata
so that they can be generated by the parent class.

Merge pull request #27973 from colemanw/getLinks

SearchKit - New getLinks API action for advanced link handling

Merge pull request #28789 from eileenmcnaughton/message_template

Update `MessageTemplates` generation to work with Smarty3

Use distinct test workflow message for workflow message tests

The tests for the workflow message subsystem use an instance of a test
case_activity template attached the real message workflow template. They
test the text version - meaning that removing it, like we have with
others, causes various fails. In addition adding a real template that
does not match the test version causes mayhem.

In this case we want to ensure we keep all the tests for the subsystem
functionality but undo the tethering to a particular
template to allow that template to have updates without
having to go deep into this code

Extract date regex validation

Merge pull request #28825 from eileenmcnaughton/no_ref

Remove function use of reference

Merge pull request #28822 from eileenmcnaughton/date_formats_test

Extend date formats being tested

Merge pull request #28823 from eileenmcnaughton/date_const

Use constants for date formats

Merge pull request #28556 from eileenmcnaughton/trait_meta

Fully deprecate MetadataTrait

Deprecate passing in month