Set trxn_id & invoice_id in tests as needed rather than by default.

This causes work-arounds & flakiness by tests using this fn.
Better to add as required.

Merge pull request #13145 from civicrm/5.8

5.8 to master

Merge pull request #13137 from eileenmcnaughton/user58

dev/core#534 fix failure of print invoice to display on settings page

Fix missing Pay now link

Disable function that doesn't work on jenkins (but does locally) for not

Remove static var from env function.

It is causing a test fail and is only saving us from calling a cached function....

Test fixes, better cleanup

Use function to determinie if invoicing is enabled which accounts for historical weirdness

Use onToggle feature for setting user_dashboard_options when toggling invoicing, remove from form

Add test for user dashboard

Merge pull request #13135 from civicrm/5.8

5.8

Merge pull request #13131 from eileenmcnaughton/5.8

dev/core#534 Fix intra-rc regression mis-saving of invoicing setting

Merge pull request #13132 from eileenmcnaughton/php72

Fix php 7.2 notice

Fix php 7.2 notice

Fix saving of invoicing.

Invoicing has setting metadata but this setting is ignored in various places in core in favour of using a hacky method. Don't break when
hackiness ensues

Merge pull request #13124 from eileenmcnaughton/export_fgb

Fix Export when full group by mode is used

Merge pull request #13087 from eileenmcnaughton/lybunt

Lybunt report - improve developer support for debugging this report

Merge pull request #13130 from civicrm/5.8

5.8 to master

Merge pull request #13128 from colemanw/jstree

dev/core#507 - Fix wrong version of jstree in search popop

dev/core#507 - Fix wrong version of jstree in search popop

Fix a further fatal error when full group by is enabled

Merge pull request #13122 from eileenmcnaughton/export_merge

Export code cleanup - remove silly function

Merge pull request #13121 from pradpnayak/BatchSearchFix

dev/core#532 Batch search fix

Fix Export when full group by mode is used

After upgrading locally to mysql 5.7 I found that the exportIM test
was taking so long locally that I was killing it rather than finding
out how long it would take.

Digging into it I found that we were changing the query so that
attempts to group by contact ID were being nullified when full group by
is enabled. This meant that we were winding up with
7776 rows being retrieved to reflect a grid of permutations, which was
being iterated down to 1 in php.

In general the practice of altering the groupby to meet this new
standard is one that we determined to be
causing problems and we discontinued / rolled back

Remove silly manipulateHeaderRows function

Export code cleanup -  Extend testing so we can get rid of silly manipulate fn

Remove header handling that I think is unreachable

Merge pull request #13117 from eileenmcnaughton/export2

Export code clean up - extract build row & getTransformed row off to ExportProcessor

Merge pull request #13120 from eileenmcnaughton/less_primary

Export class cleanup - remove some unnecessary code

Merge pull request #13098 from eileenmcnaughton/monish_pay

dev/core#153: Pending Pay Later /w Custom payment method

Modified unit test

format form values for multi-select fields

Query building shouldn't be iterated

Merge pull request #13115 from eileenmcnaughton/noah

Minor Contact BAO code cleanup

Remove provider_id & duplicate key additions from export 'primary' (AKA all) fields

In testing all these fields except for provider_id are already in the primary field.

Provider ID is a psynonym for im_provider. Possibly it is the better name choice
but it has a lot of hacks associated with it that can go by removing it

Merge pull request #13119 from civicrm/JoeMurray-patch-4

Update name of icon from pencil to wrench in EventInfo help

Merge pull request #13118 from civicrm/JoeMurray-patch-3

Fix Configure Event help grammar

Merge pull request #12650 from JMAConsulting/dev-core-321

dev/core#321: Prevent duplicate entries in civicrm_entity_file

Minor Contact BAO code cleanup

Merge pull request #13113 from mattwire/fix_delete_participant

Regression: Fix delete event participant fatal error and display name if no email

Update name of icon from pencil to wrench in EventInfo help

Fix grammar

Move buildRow to processor class

Move getTransformedFieldValue to exportProcessor

Don't pass il8n object, just use singleton

Extract buildRow

Merge pull request #13110 from eileenmcnaughton/profile_translate

dev/core#527 Permit translation of Public Title field for Profiles

Merge pull request #13082 from eileenmcnaughton/export

Export code cleanup - Use getComponentPaymentFields from processorClass

Fix delete event participant fatal error and display name if no email

Merge pull request #13111 from eileenmcnaughton/php72Countable

Fix php7.2 notices on trying to count null

Merge pull request #13112 from jitendrapurohit/core-528

dev/core#528 - Advanced Search -> Contribution Tab and Contribution D…

dev/core#528 - Advanced Search -> Contribution Tab and Contribution Dashboard returns a fatal error

Lybunt report - improve developer support for debugging this report

By using the createTemporaryTable to create our temporaray table we
help developers out by
a) adding the query to the developer tab
b) supporting CIVICRM_TEMP_FORCE_DURABLE
(see debugging section in dev docs)

This fix updates a function used by reports that have been marked with

groupFilterNotOptimised = FALSE

The reports with optimised group filtering construct a temp table
of contacts in the groups & use those to inner join / limit
the contacts in the report.

They are tested via a bunch of tests in api_v3_ReportTemplateTest such as

testReportsWithNonSmartGroupFilter

m

Change-Id: I1102b43a643760320a4b011c7a11146c8d4f380f

Fix php7.2 notices on trying to count null

Merge pull request #13101 from systopia/issue_525

fix for issue-525: Extraneous br-tags in rendered note-fields

Always create temp table at the start.

The saving of not creating an empty file occassionally doesn't warrant the code complexity

Move wrangling payment header outside main row iteration

Refactor out getPaymentHeaders function

Use getComponentPaymentFields from processorClass

Further the conversion to deal with non-entity pseudofield

Partial converstion of Profile form to be an entity form, fixes translatability for frontend title

Merge pull request #13107 from eileenmcnaughton/php72Countable

Fix warning error on php 7.2

Merge pull request #13109 from civicrm/5.8

5.8

Merge pull request #13108 from seamuslee001/5.8

Port Use tab title in feedback message

Use tab title in feedback message

Fix warning error on php 7.2

Merge pull request #13106 from seamuslee001/5.8

5.8

Replace deprecated event function

Retrieve tab key from form attributes even when action is urlencoded

Merge pull request #13093 from eileenmcnaughton/event_deprecated

Replace deprecated event function

Merge pull request #13097 from eileenmcnaughton/website_type

CRM-21427 - Add form validation to make it clear we only allow a single website of each type

Merge pull request #13100 from eileenmcnaughton/env

dev/cloud-native/issues#18: Soften messages for read-only extensionsDir

Merge pull request #13102 from christianwach/issue-526

Use tab title in feedback message

Merge pull request #13099 from christianwach/issue-wp-12-tab

Retrieve tab key from form attributes even when action is urlencoded

Use tab title in feedback message

Replace deprecated event function

dev/cloud-native/issues#18: Soften messages for read-only extensionsDir

fix for issue-525: Extraneous br-tags in rendered note-fields

Retrieve tab key from form attributes even when action is urlencoded

dev/core#153: Pending Pay Later /w Custom payment method

Merge pull request #13095 from eileenmcnaughton/case_date

Add format type to Case Start and End Dates

CRM-21427: Allow multiple websites without type and only allow a single website of specified type

Add format type to Case Start and End Dates

Merge pull request #13091 from omarabuhussein/dev/core#193-v2

dev/core#193: Ensure that tax amount is calculated when checking for order API line item

Merge pull request #13094 from civicrm/5.8

5.8 to master

Merge pull request #13084 from colemanw/menuCleanup

dev/core#487 Menu cleanup in preparation for switch to SmartMenus library

Merge pull request #12557 from eileenmcnaughton/activity

Activity tab code refactor

dev/core#193: Ensure that tax amount is calculated when checking for order API line items

Merge pull request #13081 from colemanw/mergeBtn

Add merge link to duplicate contact popup

Merge pull request #12659 from kenwest/strip-html-tags-from-joblog-description

Fix Job Log entries containing links so they will display properly

Merge pull request #13085 from colemanw/scroll

Fix accidental scroll when saving/deleting CiviMail draft

Add unit test for CRM_Core_BAO_Navigation::checkPermission()

Merge pull request #12942 from twomice/CRM-19751_search_on_hold_form

Toward CRM-19751: conditionally change On-Hold criteria to select on …

Fix accidental scroll when saving/deleting CiviMail draft

Toward CRM-19751: conditionally change On-Hold criteria to select on Advanced Search form.

Merge pull request #12990 from mattwire/event_copy_posthook

Trigger post hook on Event Copy

Menu code cleanup in preparation for switch to SmartMenus library

- Add a couple client side variables
- Fix access to functions that were private for no reason
- Extract permissions check into its own function
- Fire crmLoad event when menu loads

Add merge link to duplicate contact popup

Merge pull request #13076 from colemanw/dev/core#503

dev/core#503 - More accurate custom field checking before htmlEncode

Merge pull request #13083 from eileenmcnaughton/export_huh

Export code cleanup -remove meaningless code

Export code cleanup -remove meaningless code