Merge pull request #11927 from samuelsov/NFC-taxts

(NFC) make (includes taxes) translation work

(NFC) make (includes taxes) translation work

Merge pull request #11885 from eileenmcnaughton/report_filter_eq

Handle eq (equals) operator in the report url

Merge pull request #11921 from civicrm/5.0

5.0 merge to master

Merge pull request #11917 from eileenmcnaughton/cont_report

CRM-21831 & dev/report/issues/1 Fix regressions in contribution detail report relating to soft credits

Merge pull request #11919 from eileenmcnaughton/top

Fix fatal on topDonor report

Merge pull request #11918 from eileenmcnaughton/export

Extract fetchRelationshipDetails function

Fix fatal on topDonor report

Merge pull request #11814 from eileenmcnaughton/report

CRM-21677 - Report improvements

Extract fetchRelationshipDetails function

Merge pull request #11914 from seamuslee001/CRM-21244-doc

CRM-21244 Update documetnation so it makes sense when setting is disabled

Fix bugs in contribution detail report relating to soft credits

Merge pull request #11915 from civicrm/5.0

5.0

Merge pull request #11906 from JMAConsulting/dev-42

(dev/mail/6) On multilingual mode, choosing mailing group doesn't affect recipient count and list

Merge pull request #11905 from seamuslee001/CRM-21244-consistancy

CRM-21244 Ensure consistancy with previous behavior where user emails …

Merge pull request #11909 from mattwire/membershiptype_comments

NFC Minor comments cleanup on MembershipType

Merge pull request #11668 from elisseck/CRM-21763

CRM-21763 Util to subtract currencies using integers for precision

Merge pull request #11911 from JMAConsulting/dev-7

(dev/mail/7) On 'New Mailing' review page, it doesn't show recipients count

Update documetnation so it makes sense when setting is disabled

CRM-2144 Ensure consistancy with previous behavior where user emails are first then system from emails.

Add unit test to try to verify order of emails

Fix issue where email was not being used as array key causing wrong email to be used

(dev/mail/7) On 'New Mailing' review page, it doesn't show recipients count

(dev/mail/6) On multilingual mode, choosing mailing group doesn't affect recipient count and list

Minor comments cleanup on MembershipType

Merge pull request #11904 from totten/5.0-preview-regr

(dev/mail/5) "New Mailing" - Previews should not schedule real blasts

CRM-21677 - Report improvements

CRM-21677 - fix DB error and more clean up

CRM-21677 - generalised function

minor fix

CRM-21677 - keep the naming consistent

style fix

Merge pull request #11898 from eileenmcnaughton/min_report

CRM-21677 reduce unnecessary joins in reports

(dev/mail/5) "New Mailing" - Previews should not schedule real blasts

In `CRM_Mailing_BAO_Mailing::create` and `Mailing.create` API, there is a
(*ahem*) special behavior where setting the `scheduled_date` will
immediately trigger scheduling.  One shouldn't submit `scheduled_date` for a
preview.

This was not symptomatic before #11142/v4.7.31 because all preview
operations were wrapped in a transaction and rolled back.  But now previews
are allowed to have side-effects, so we need some other means to prevent.

This copies the workaround from `crmMailingMgr.save()` and applies
it to `crmMailingMgr.preview()` (etal).

Merge pull request #11878 from mattwire/CRM-21791_financial_batch_pseudoconstant

CRM-21791 Use name for comparison instead of label (alternative)

CRM-21677 reduce unnecessary joins in reports

Merge pull request #11889 from eileenmcnaughton/min_report

CRM-21677 reduce unnecessary joins in reports

Merge pull request #11890 from otetard/bugfix/fix-deceases-membership-processing

Fix CRM-19920 regression: “Deceased” status could be translated.

Fix CRM-19920 regression: “Deceased” status could be translated.

Find “Deceased” `status_id` by using `CRM_Core_PseudoConstant::getKey()` and check use this status is enabled by using this `status_id` instead of its label (which could be translated).

Merge pull request #11703 from eileenmcnaughton/export

Remove cloning hack from export and add unit tests

Merge pull request #11895 from totten/master-ext-check

CRM_Utils_Check - Soften messages for read-only extensionsDir

CRM_Utils_Check_Component_Env::checkExtensions - Fix typo

CRM_Utils_Check_Component_Env - Soften messages for read-only extDir

There are competing schools of thought on whether extension folders should be web-writable:

* Sometimes, the most active (or the only) admins are web-based. Making the folder writeable lets them keep extensions up-to-date through the web UI.
  This includes applying security-fixes for extensions. Thus, sites with writeable extdir are harder to attack (more secure).
* Sometimes, the most active (or the only) admins don't use the web-based admin UI, and they don't trust any web-based users to do administration.
  They don't want the folder to be writeable.
* If there's a flaw that allows writing to the filesystem, it could be escalated to writing+executing code. Thus, sites with read-only
  extdir are harder to attack (more secure).

This commit tries to accept each scenario as valid -- but communicate
better.  Instead of flatly describing the read-only dir as erroreous,
present a warning with some choice/trade-off.

CRM_Utils_Check_Component_Env - Remove redundant check for extensionsDir

When you have a non-writeable extensions directory, *two* status checks will
copmlain about it (`checkDirsWritable`, `checkExtensions`).  Between the two,
`checkExtensions` is smarter.

Merge pull request #11891 from nielosz/Add-activities-to-recent-items

CRM-21434: Add activities to recent items stack

Merge pull request #11888 from mattwire/formrule_typo

NFC Fix function name typo

CRM-21763 add money util to subtract currency floats with precisely and implement in CRM_Core_BAO_FinancialTrxn::getPartialPaymentWithType

Add items to recents items

Merge pull request #11886 from eileenmcnaughton/varName

[NFC] Fix mispelled variable and simplify a couple of other vars

CRM-21677 reduce unnecessary joins in reports

Merge pull request #11880 from eileenmcnaughton/min_report

 CRM-21677 reduce unnecessary joins in Case, Grant, Survey reports

Fix function name typo

 CRM-21677 reduce unnecessary joins in Case, Grant, Survey reports

[NFC] Fix mispelled variable and simplify a couple of other vars

Merge pull request #11882 from eileenmcnaughton/report_filter

Fix report form isTableSelected to treat relative date filters as filters

Handle eq (equals) operator in the report url

Merge pull request #11807 from lyyana/patch-1

"hold_date" not updated if civimail_multiple_bulk_emails is true + test

Merge pull request #11336 from eileenmcnaughton/live

CRM-21489 document new constant in civicrm.settings.php

Merge pull request #11863 from eileenmcnaughton/payment_view

Tweak layout on contribution view screen to make payments clearer

Fix report form isTableSelected to treat relative date filters as filters.

isTableSelected looks to see if the filter has been passed in. Without this it looks in ->_params[{}_value] &
for {}_op to be null or 'not null' but misses relative date filters.

Merge pull request #11879 from MegaphoneJon/CRM-21842

CRM-21842 - Fix links to documentation

Merge pull request #11874 from eileenmcnaughton/cust_data

[NFC] remove unused vars

CRM-21791 Use name instead of label

Merge pull request #11876 from yashodha/report_improv

CRM-21677 reduce unnecessary joins in mailing reports

CRM-21842 - Fix links to documentation

CRM-21677 reduce unnecessary joins in mailing reports

Merge pull request #11875 from eileenmcnaughton/min_report

CRM-21677 reduce unnecessary joins in pledge reports

CRM-21677 reduce unnecessary joins in pledge reports
And walklist report:

Pledge/Detail
pledge/pbnp
pledge/summary
walklist/walklist

Merge pull request #11872 from totten/master-test-trait

CiviUnitTestCase - Extract Api3TestTrait and Api3DocTrait

CiviUnitTestCase - Extract Civi\Test\Api3DocTrait

This makes `CiviUnitTestCase` more readable, and it allows other tests to
benefit from these helpers (even if they don't use everything from
`CiviUnitTestCase`).

CiviUnitTestCase - Extract Civi\Test\Api3TestTrait

This makes `CiviUnitTestCase` more readable, and it allows other tests to
benefit from these helpers (even if they don't use everything from
`CiviUnitTestCase`).

Merge pull request #11697 from mattwire/CRM-21784_recur_customdata

CRM-21784 Display custom data when viewing recurring contributions

[NFC] remove unused vars

Merge pull request #11826 from eileenmcnaughton/karing

Use try catch when deleting groups (dev/core/issues/22 )

Merge pull request #11867 from eileenmcnaughton/min_report

CRM-21677 reduce unnecessary joins in membership reports

Merge pull request #11864 from colemanw/upgradeVersion

Increment minimum upgradable version

Merge pull request #11869 from mukeshcompucorp/fix-template-structure-issues

Fix template structures

Fix template structures

Merge pull request #11609 from andrewpthompson/CRM-21203-andrewpthompson

CRM-21203 revisited - Joomla 3.8 extra fixes for cli.php

CRM-21677 reduce unnecessary joins in membership reports

Merge pull request #11086 from agileware/CRM-21277

CRM-21277: Suppress warnings while finding install directory.

Merge pull request #11087 from yashodha/CRM-19850

CRM-19850 - Add a offline membership without 'record payment' option,…

Merge pull request #11825 from mattwire/CRM-21682_commenting

Add commenting to explain +1/-1 days for membership dates calc

Merge pull request #11816 from eileenmcnaughton/fix

core/issues/15 Line item fix with attempt to determine how it is hit

Merge pull request #11824 from mattwire/CRM-21682_membership_renewal_tests

CRM-21682 Membership Renewal Tests

Merge pull request #11829 from eileenmcnaughton/rules

Fix regression on CiviRules search due to exception handling

Merge pull request #11836 from seamuslee001/dev_core_25

Dev/Core 25 Wrap the mailing split_job function in a transaction to e…

Merge pull request #11828 from eileenmcnaughton/revert

Revert "CRM-8140: Not possible to select fields for export when using Custom Searches"

Merge pull request #11862 from mukeshcompucorp/fix-template-structure-issues

Fix template structure

Merge pull request #10496 from jitendrapurohit/CRM-20719

CRM-20719 : Show warning on system status page if reply_id for mailing is not set to any default

Merge pull request #11857 from eileenmcnaughton/cust_data_any

Consolidate CustomData code into shared tpl

Merge pull request #11861 from eileenmcnaughton/task

Fix bugs with hook params, functionality on contact searches

Merge pull request #11856 from eileenmcnaughton/cust_multi

Fix ajax error on rendering multi custom data tab when field limit reached

Merge pull request #11859 from eileenmcnaughton/import_test

[nfc] Add test for thousand separators on import

Dev/Core 25 Wrap the mailing split_job function in a transaction to ensure that there is no possibility of duplicate maiing jobs created

Include the split_jobs in the transaction used to update the parent job

Increment minimum upgradable version

Fix colspan

Merge pull request #11298 from otetard/feature/merge_memberships

CRM-19151: Add the ability to merge memberships without data loss

Tweak layout on contribution view screen to make payments clearer

Fix bug where search does not load if print is disabled.

This was picked up in QA on search task changes & I was able to replicate - this fixes

Fix template structure

Merge pull request #11860 from eileenmcnaughton/test_fixes

[nfc] Minor test tidy ups

Fix regression (intra-rc) where entity not passed to hook, and fix CRM-21841 as by-product.

The self::objectType is passing NULL - the object type on the parent.

Switching to static::objectType uses the child.

https://stackoverflow.com/questions/1912902/what-exactly-are-late-static-bindings-in-php

Consolidate CustomData code into shared tpl

 The only change that I found in function was that the delete icon was originally in a
 table on contact edit but afterwards it was not. However the UI looked the same

core/issues/15 Line item fix with attempt to determine how it is hit

Merge pull request #11855 from eileenmcnaughton/report_min

CRM-21677 - reduce unnecessary address joins on activity & contact detail reports

[nfc] Minor test tidy ups