Merge pull request #14934 from civicrm/5.16

5.16

Merge pull request #14932 from eileenmcnaughton/export_fn_move

[REF] [Export] Move writeToTable fn to exportProcessor

Merge pull request #14931 from eileenmcnaughton/mysql516

mysqli_get_client_info() does not have a parameter - fixes https://lab.civicrm.org/dev/core/issues/1136

Merge pull request #14917 from eileenmcnaughton/zero_ex

[REF][Event]  Extract calculation of 'zero-ness' in form rule

Merge pull request #14881 from eileenmcnaughton/cont_payment_method_import

dev/financial#36 [IMPORT] fix & test mishandling on payment_instrument labels

Move write to table to exportProcessor

mysqli_get_client_info() does not have a parameter - fixes https://lab.civicrm.org/dev/core/issues/1136

Merge pull request #14877 from eileenmcnaughton/hist_rep

dev/core#1141 remove unused deprecated sql_calc_rows

Merge pull request #14914 from eileenmcnaughton/export_test

[Export] Convert testGender Export to new test format.

Merge pull request #14913 from eileenmcnaughton/export

[REF] [Export] Stop passing header rows around

Merge pull request #14925 from civicrm/5.16

5.16

Merge pull request #14924 from colemanw/menubar5.16

Backport menubar fixes to 5.16

Merge pull request #14926 from eileenmcnaughton/conf

dev/core#1084 Contribution confirmation page: Translation won't work

Update Confirm.tpl

Merge pull request #14920 from eileenmcnaughton/url_search

Fix accidental move of sort_name field on advanced search

Merge pull request #14904 from eileenmcnaughton/activ_516

dev/core#1042 fix  duplicate activities on custom data block

Menubar - get rid of hardcoded pixel values in crm.menubar.js

Menubar - Use class instead of id for flexibility

Fix accidental move of sort_name field on advanced search

The code that made this field url-enterable ALSO re-ordered them by accident - this fixes

Merge pull request #14922 from civicrm/5.16

5.16 to master

Merge pull request #14918 from totten/5.16-prevnext

Revert "[REF] use generic loadStandardSearchOptionsFromUrl". Fix search selections.

Revert "[REF] use generic loadStandardSearchOptionsFromUrl". Fix search selections.

Overview
-------------------

This reverts commit a5ca2ad8e16b145edf3a5604fab923d1e2386b37
and fixes a bug in management of the prev/next cache. The
bug is visible when following this procedure:

1. In browser, use "Quick search" to search for text "am"
2. In browser, select checkboxes next to two items
3. In MySQL CLI, run `select * from civicrm_prevnext_cache where is_selected =1;`
4. In browser, continue to next page. Observe that there are no more selection.
5. In MySQL CLI, re-run SQL query. Observe that the selections have disappeared.

Before
-------------------

Proceeding to another page of search results causes loss of selections

After
-------------------

Selections are preserved

[REF] Extract calculation of 'zero-ness' in form rule

Merge pull request #14911 from seamuslee001/ref_prev_next_cleanup_service

[REF] Add in cleanup function to prevnext service and utilise in clea…

Merge pull request #14915 from eileenmcnaughton/ex_test

[REF] [Export] [Test] Update 2 more tests to use new helper

Test conversion

Convert testGender Export to new test format.

This includes a tangental cleanup & fix whereby the test covers a label longer than 16 char & the code for
prefix_id & suffix_id is consolidated by handling the special casing in the metadata declaration
rather than in multiple places.

[REF] Add in cleanup function to prevnext service and utilise in cleanup cache situation

Set TTL on prevNextSQL Cache to be 2 days so cleanup code works as previously

Switch to using a class constant for the number of cache days

Stop passing header rows around

Merge pull request #14912 from eileenmcnaughton/export

[REF] [Export] Remove now redundant param

Remove now redundant param

Merge pull request #14784 from kirk-circle/1050-delete-repeating-activities

dev/core#1050 - Delete repeat activities that are selected for deletion

Merge pull request #14907 from eileenmcnaughton/force_pay

[REF] remove never-set, mispelt parameter

Merge pull request #14887 from mattwire/membership_deprecateids

Further deprecate use of $ids array in membership functions

Merge pull request #14747 from mattwire/contributionapi_getsoftperformance

Improve performance on getSoftContribution details - only run one query instead of one per contribution

Merge pull request #14909 from eileenmcnaughton/net_amount

Add unit test for net_amount when fee_amount is set

Merge pull request #14898 from eileenmcnaughton/export_rel

[REF] [Export] Move fetch Relationship details to processor

Merge pull request #14910 from JKingsnorth/core-1153

Remove unused standalone-mode code

Merge pull request #14906 from eileenmcnaughton/event

[NFC] comment fixes, function mis-casing fix

Test the activity form's deletion of single and repeating activities

Delete repeat activities that are selected for deletion

Add unit test for net_amount when fee_amount is set

This just improves testing & ensures fee_amount is used to set net amount.

I got some odd results on financial_trxn & financial_item but just adding checks for
what I found atm

Remove more use of ids array on CRM_Member_BAO_Membership::create

dev/core#1153 Remove unused standalone-mode code

Merge pull request #14889 from demeritcowboy/logging-report-clarification

dev/core#1149 - Make it clearer which record the logging report is displaying

Merge pull request #14902 from eileenmcnaughton/mem

Fix membership end date on confirming a pending contribution

Merge pull request #14908 from seamuslee001/master

5.16

Merge in 5.16

[REF] remove never-set, mispelt parameter

[NFC] comment fixes, function mis-casing fix

Fix membership end date on confirming a pending contribution

https://github.com/civicrm/civicrm-core/pull/13706

dev/core#1042 fix  duplicate activities on custom data block

Merge pull request #14900 from eileenmcnaughton/regress

Fix unreleased regression - fatal on financial account screen

Add handling & test for translated contribution status

dev/financial#36 [IMPORT] fix & test mishandling on payment_instrument labels

Merge pull request #14880 from eileenmcnaughton/cust_field_label

[Test] [Import] Add test to demonstrate bug that turns out not to exist

Merge pull request #14899 from eileenmcnaughton/format

[NFC] reformat class

Fix unreleased regression - fatal on financial account screen

Merge pull request #14534 from pradpnayak/EventTypeQuery

Fixed event type id fetch

[NFC] reformat class

Merge pull request #14870 from eileenmcnaughton/import_except

[IMPORT] [code-quality] Remove instances of CRM_Core_Error::fatal from first import form

Merge pull request #14893 from eileenmcnaughton/date

Fix support for relative dates in urls

Move fetch Relationship details to processor

Merge pull request #14885 from aydun/location_defaults

Add default location for API v3 creates of Address, IM, OpenID and Phone

Merge pull request #14760 from eileenmcnaughton/unsub

dev/core#1108 [REF] use CRM_Core_DAO::executeQuery instead of ->query()

Merge pull request #14886 from mattwire/membership_payment

Switch create MembershipPayment to use API

Merge pull request #14620 from mattwire/entitypagetrait_defaultcontactid

EntityPageTrait: Set logged in contact ID as default if one is not specified - this allows permission checks etc. to work properly.

Switch create MembershipPayment to use API

Improve performance on getSoftContribution details - only run one query instead of one per contribution

Merge pull request #14896 from eileenmcnaughton/event_count

Fix php7.x warning on count

Merge pull request #14874 from eileenmcnaughton/export_1

[REf] [Export] Remove deprecated componentPaymentFields function

Merge pull request #14875 from eileenmcnaughton/export_2

[Ref] [Export] Remove exportComponent function - it's not adding much here

Fix php7.x warning on count

Fixes a warning when php 7.2 tries to count on a null value

Merge pull request #14854 from eileenmcnaughton/validate

Do not launch raw js alert  jqueryValidation fails

Merge pull request #14895 from mattwire/optiongroup_checkcrash

Don't let optiongroup check crash

Don't let optiongroup check crash

Small test fixes/standardisations.

Fix support for relative dates in urls

This improves support for relative dates for fields that have been converted.

For example

http://dmaster.local/civicrm/contribute/search?reset=1&receive_date_relative=this.year&force=1

Now works to render donations 'This Calendar Year'

Add default location for API v3 creates of Address, IM, OpenID and Phone.

Has tests.

Default location was recently added for APIv3 Email.create
(https://github.com/civicrm/civicrm-core/pull/14489)
and for Address, Email, IM, OpenID and Phone in APIv4
(https://github.com/civicrm/org.civicrm.api4/pull/162)
so this brings consistency of behaviour for these entities between each other
and between APIv3 and APIv4

Remove some static caches that are crashing tests

[Test] Add test to demonstrate bug that turns out not to exist

I thought there was a bug in import handling in custom data so I wrote a test to try to capture it - tests pass & I'm off to re-check
my data & see if the bug was more of a pebkac error.

Anyway test is good

make it clearer what the record is

Merge pull request #14879 from eileenmcnaughton/import_gender_test

[REF] [Test] Add test to cover handling of 'gender_id' on import, remove unused code.

Merge pull request #14883 from totten/master-daotest

CRM-20455 - Allow CRM_Core_DAOTest to pass w/packages#259

Merge pull request #14884 from mlutfy/event6

dev/event#6 Follow the 'same email' participants config setting for backend participants

dev/event#6 Follow the 'same email' participants config setting for backend participants

DAOTest - Use docblock for type

This patch is going in as part of some cleanup/refactoring in a mid-cycle
change. Using the docblock allows the type to be communicated without
breaking on mid-cycle contract change.

CRM-20455 - Allow CRM_Core_DAOTest w/packages#259

Merge pull request #14672 from seamuslee001/load_smart_cache_after_deleting_group_contact_records

Invalidate smart group cache for group following deletion of group_c…

Merge pull request #14882 from totten/master-query-event

CRM-20455 - Add specialized event for queries

Remove the js alert message when jqueryValidation fails on frontend pages

It's really really ugly & as discussed without it
a) extensions can still opt to display the errors themselves and
b) the inline errors still appear.

Note this will have no real affect on existing code - the ugliness of this has been
a blocker to getting into front end validation & at this stage only the
omnipay extension is trying to work in this space.
https://github.com/eileenmcnaughton/nz.co.fuzion.omnipaymultiprocessor/pull/128

I think ideally with Omnipay it DOES makes sense to display messages
near the checkout button - which might be a future step

CRM-20455 - Add specialized event for queries

Overview
--------

The recently merged PR for CRM-20455, https://github.com/civicrm/civicrm-packages/pull/180, uses the
GenericHookEvent for modifying all SQL queries.  GHE is flexible, but it
comes with a slight performance penalty (juggling a few internal arrays).
The penalty is usually too small to care if you're firing one hook. However, with CRM-20455,
we can foresee the event firing hundreds or even thousands of times within a given page-view.

This PR adds a smaller/simpler class for use with `modifyQuery`. There will be a different PR to use it.

Before
------

* Don't have `QueryEvent`

After
-----

* Do have `QueryEvent`

Add test to cover handling of 'gender_id' on import, remove unused code.

I'm just venturing into import to fix a bug & wrote this test to step through the code. Committing as it's passing.

In the process I found that it's impossible to set 'gender' as an import field & only 'gender_id' is
available as a field so removing 'gender'

Merge pull request #14861 from eileenmcnaughton/anet

[test] Catch A.net exception & ignore

Merge pull request #14833 from seamuslee001/ids_ip_logging_improvements

Ensure that if present the HTTP_X_FORWARDED_FOR IP address is used in…

Merge pull request #14878 from civicrm/5.16

5.16 to master

Merge pull request #14876 from eileenmcnaughton/is_front

dev/core#378 Determine front end pages in drupal

Merge pull request #14718 from totten/herb-private-hier

Move log and compilation dirs from "Runtime" to "Paths"

Invalidate smart group cache for group following deletion of group_contact row

Add in function to invalidate smart group cache rather than building it up again when contact removal is deleted

Remove if check

dev/core#1141 remove sql_calc_rows

It's gone for mysql8 & I just checked & in this case the count is never used so it can just go