Switch userDashboard from using selector object to using the api to get contributions to display.

Note that preliminary tests for this were written & merged some time ago.

Reasons for the change are

1) readability - most devs are much more comfortable with reading the api code than the selector
2) performance - the contribute selector code is deeply unperformant - mostly due to the summary function which in
this case is somewhat mitigated by the limit of 12 but we are still doing something slow for no reason
3) test stability - the test for this turned out to have poor stability & hopefully this will help
4) preliminary cleanup - there are 2 existing PRs that attempt to add new buttons to this & moving towards
a cleaner tpl / php layer will help with those. In addition there is a serious performance issue to
address on the contribution summary function. Reducing use of that function should help with the cleanup effort

Merge pull request #13523 from eileenmcnaughton/def_country

dev/core#691 Make default country optional on setting form

Merge pull request #13583 from eileenmcnaughton/user_dash

[REF] switch from (undeclared) class property to local variable.

Merge pull request #13562 from eileenmcnaughton/payment_test

[REF] Minor readability cleanup

Merge pull request #13342 from mattwire/activity_submitonce

Remove activitystatus js. Add submitOnce handler for activity create

dev/core#691 Make default country optional on setting form

This includes removing the default of 'United States'.... ahem!

Merge pull request #13580 from colemanw/noMoreFourTwo

Bump minimum upgradable ver to 4.2.9

Merge pull request #13585 from civicrm/5.11

5.11 to master

Merge pull request #13581 from colemanw/dev/core#715

dev/core#715 - Fix delete action on RelationshipType form

REF minor code readability improvements

[REF] switch from (undeclared) class property to local variable.

We either needed to declare the variable on the class & make it a local variable. Switching
to a local var reflects the fact it is never accessed from outside thie function & improves readability.

grepping for _userOptions returns nothing after this.

I also removed an extraneous config singleton call. I can't see a strong case that either of
these changes will affect the intermittent fails but ... maybe?

dev/core#715 - Fix delete action on RelationshipType form

Merge pull request #13577 from mattwire/codecleanup

Fix typo and space

Bump minimum upgradable ver to 4.2.9

Code cleanup

Merge pull request #13576 from civicrm/5.11

5.11 to master

Merge pull request #13575 from seamuslee001/5_10_1_release_notes_11

Release notes for 5.10.1 release

Merge pull request #13572 from eileenmcnaughton/cc_511

Fix regression whereby making receive_date required breaks back offic…

Release notes for 5.10.1 release

Merge pull request #13564 from seamuslee001/remove_free_campaign_case

Remove hurty free calls from campaign and case

Fix regression whereby making receive_date required breaks back office cc

Regression from https://lab.civicrm.org/dev/core/issues/680
led to https://civicrm.stackexchange.com/questions/28410/unable-to-take-credit-card-payment-after-civi-update

Remove activitystatus js. Add submitOnce handler for activity create

Merge pull request #13569 from colemanw/Fix

Fix contact ID help on advanced search

Fix contact ID help on advanced search

Remove hurty free calls from campaign and case

Merge pull request #13558 from eileenmcnaughton/balance_test

Add unit test on getContributionBalance fn (#13187)

Merge pull request #13560 from seamuslee001/free_activity_member

dev/core#562 Remove free calls from Activity and Member sections of CRM

Merge pull request #13327 from pradpnayak/SerarchView

Used buildoptions function to get all groups

Merge pull request #13404 from pradpnayak/ReportActions

Show Add to group on  create new report after refresh of result

Add unit test on getContributionBalance fn (#13187)

Remove free calls from Activity and Member sections of CRM

Minor readability cleanup

Merge pull request #13287 from mattwire/eventregister_rename

(REF) Rename variables and adjust variable definitions for Event Register form

Merge pull request #13187 from JMAConsulting/payment_function

Optimize CRM_Core_BAO_FinancialTrxn::getTotalPayment

Merge pull request #13555 from colemanw/loadScript

CRM.loadScript improvements

Add angular util crmLoadScript

Append cache-buster to dynamically loaded scripts

Merge pull request #13549 from pradpnayak/Qfill

dev/core/issues/700, Show Qill when searched using contact id

Merge pull request #13333 from mattwire/buttons_submitOnce

Make submitOnce() button js into a button parameter

Merge pull request #13529 from greenpeace-cee/fix-api-entity-permission

dev/core#690 - Civi\API - Fix entity permission check for trusted calls

Set version to 5.12.alpha1

Set version to 5.11.beta1

Merge pull request #13552 from seamuslee001/master

Update 5.10.0.md

Update 5.10.0.md

Merge pull request #13550 from civicrm/5.10

5.10

Merge pull request #13494 from agh1/5.10.0-releasenotes

5.10.0 release notes

Merge pull request #13544 from pradpnayak/ContactImage

core/issues/698, send correct MIME type for .jpg files

dev/core/issues/700, Show Qill when searched using contact id

5.10.0 release notes: late changes, credits, and last-minute fixes

Add release-notes/5.9.1.md

5.10.0 release notes: bulk of edits

* pulling out misc and organizing lines with gitlab issue numbers

* linking to gitlab issues

* update lines with gitlab issues to have the gitlab titles

* progress

* second pass

* finished first pass of notes

* deleting rows related to merge

* reorganizing sections

5.10.0 release notes: added boilerplate

5.10.0 release notes: raw from script

Merge pull request #13548 from civicrm/5.10

5.10 to master

Merge pull request #13536 from eileenmcnaughton/export_sc

dev/core#651 Fix group by on export soft credits (possible recent regression, clearly wrong).

Merge pull request #13547 from eileenmcnaughton/master

Re-run gencode for fix on PriceField DAO

Re-run gencode for fix on PriceField DAO

Merge pull request #13546 from civicrm/5.10

5.10 to master

Added few more mime type that needs over-ride

Merge pull request #13403 from pradpnayak/FieldLength

Donot allow to enter more than max length of the field

dev/core#651 Fix group by on export soft credits (possible recent regression, clearly wrong).

As pointed out by the reporter the group by is being calculated as if it were a string but it's an array, this fixes.

This code has been touched recently so it might be a recent regression. 5.10 is the first release in a long time where
export is working in some mysql / output configs after a big refactor to get rid of wide temp tables

core/issues/698, send correct MIME type for .jpg files

dev/core#690 - Civi\API - Add a check on entity_table existing

This adds a check in DynamicFKAuthorization to verify that the
entity_table in the API request actually exists as a table.

Additionally, this changes a test case in api_v3_AttachmentTest to
enable permission checking. This is necessary because the change to
DynamicFKAuthorization means trusted API calls can now attach files
to *any* entity unless check_permissions is set, in which case it's
only possible for allowed delegates.

CRM/Core - Fix isCoreTable to check for tables instead of classes

This fixes CRM_Core_DAO_AllCoreTables::isCoreTable to check for the
existence of tables instead of DAO classes. The method is used only
once in core, and the intention there is to look for table names,
so there should not be much risk of breakage.

Use addField to generated form fields on Note form

Merge pull request #13537 from eileenmcnaughton/cont_comp

dev/core#680 make receive_date required on backoffice contribution form

Merge pull request #13390 from MegaphoneJon/core-609

fixes core#609 - view 'Advanced Search' links without 'view all conta…

dev/core#680 make receive_date required on backoffice contribution form
and api

Discussion on https://lab.civicrm.org/dev/core/issues/680#note_13205

Merge pull request #13535 from civicrm/5.10

5.10 to master

Merge pull request #13533 from eileenmcnaughton/prev_next

Prev next

Disable full group by mode whe groupContacts is TRUE

This prevents an error when using the cache and allows us to move
past this without untangling the messy way other tables are joined back in
for now.

Note it became necessary due to query changes off the cache but the query is no
different in accuracy

dev/core#695 - Make custom searches slightly less fragile

dev/core#679 fix loss of search results when including tags or groups.

The bug as described was a bit tricksy to replicate as it is inconsistent and the
code is not actually outputting the tags/ groups. I think these are pre-existing issues

I was able to replicate by having 2 search profiles - one with only tags & one one with only groups & switching back & and force on output -  once I replicated I found that there was a regression due to

https://github.com/eileenmcnaughton/civicrm-core/commit/2ca46d4d5a8cd15929ac0939ca2bb380a3de027e#diff-e54381bfdf51e31cab376c71ca0d66ffR4967

whereby the groupBy had been dropped, causing the results to be squashed into a single row

Merge pull request #12834 from pradpnayak/preRelHook

Respect pre hook for relationship to alter id in $params

Merge pull request #13526 from yashodha/number_widget

use number widget for weight and other numeric fields in more forms

Merge pull request #13530 from pradpnayak/FieldLength-Activity

Used field name defined in DAO file for Activity Subject

Merge pull request #13532 from pradpnayak/xmlChanges

Add html type for civicrm_note.privacy field

Merge pull request #13489 from totten/master-ro-cache

(dev/core#635) Deprecate CRM_Core_BAO_Cache for I/O. Optionally redirect I/O to Redis or Memcache.

Merge pull request #13522 from eileenmcnaughton/activity

Performance fix for alternate getActivity listing function

Remove redundant css class from activity subject field

This is now being automatically assigned thanks to the change in 6e9c7c7

Merge pull request #13527 from JMAConsulting/core-693

dev/core#693 : On contact summary page, on submitting a 'New Case' form doesn't redirect to 'Manage Case' screen

CRM_Core_BAO_Cache - When delegating to an adapter, don't expect it to return a reference

This is following up on the code-review comments from
https://github.com/civicrm/civicrm-core/pull/13489#discussion_r252070543

use name and label to retrive ID

Merge pull request #13525 from civicrm/5.10

5.10

Add html type for civicrm_note.privacy field

Donot show group until refresh result

Added api test to support fix

Used buildoptions function to get all groups

Used field name defined in DAO file for Activity Subject

dev/core#690 - Civi\API - Fix entity permission check for trusted calls

This changes the permission check in DynamicFKAuthorization to
short-circuit on trusted API calls (check_permissions = 0) so that
no exception is thrown when it's used with entities that are not
allowed delegates.

dev/core#693 : On contact summary page, on submitting a 'New Case' form doesn't redirect to 'Manage Case' screen

use number widget for weight fields in more forms

Merge pull request #13524 from eileenmcnaughton/search_rc

Revert fix causing unreleased regression

Revert "Assign the translated formValues to itself"

This reverts commit e8ef090642f0ff5639166b847a31f4e50eec3dd6.

Revert "Rewrite testHiddenSmartGroup"

This reverts commit 7810fa53a7f5d8be82ebda7fde122d0fd349d358.

Performance fix for alternate getActivity listing function

We have an alternate function to render the activiy listing on the contact tab. It is
getActivities whereas the other is deprecatedGetActivities.

It was developed in order to replace the other and we have tests that compare the results of the 2. It is better in that it
1) performs better (on a  WMF contact with many activities this is 'snappy' while the current deprecated one gives a  white screen time out) and
2) calls the selectWhereClause hook, allowing hook alteration and respecting preferred architecture.

However, we didn't go live with it in core because it
1) has a remnant performance bugs (this PR fixes the last of these)
2) implements ACLs differently - it uses generic functions whereas the deprecated one
applies more limited permissioning. This is something to clarify & resolve separately.

This PR fixes the last remaining performance issue - best described as
'When one of the activities to be displayed has many targets the activity listing is slow to load'

The reason for the slowness is that when 'target_contact_name' is passed to the api
the api does a call for each contact to fetch the contact's sort_name. For a bulk mailing that went to 50,000 people that equates to 50,000 extra queries.

However the actual display shows the first contact name and then gives a number for how many more should be retrieved. This PR hence does not ask the api for the display name, but rather does the check itself, but
only for 1 target contact rather than ALL

Note that a similar logic might be considered for assignee - I left that out of scope as I'm not
aware of situations where a large number of assignees would be assigned to a single activity.

The unit test ensures the output matches the deprecated function.

Merge pull request #13401 from PeaceWorksTechnologySolutions/master

dev/mail#32 - Further checks - making mailing test email non-case-sensitive

Optimize CRM_Core_BAO_FinancialTrxn::getTotalPayment

Merge pull request #1 from civicrm/master

Update master from upstream

Merge pull request #13520 from yashodha/number_widget

use number widget for weight fields in all forms