Merge pull request #11725 from tschuettler/fix-field-required

CiviReport - Check for trueish values of the parameter 'required'

Merge pull request #11642 from JKingsnorth/CRM-21739

[NFC] CRM-21739 Test improvements for mailing getRecipients

Merge pull request #11729 from lcdservices/CRM-21810

CRM-21810 changelog panel UI tweaks

CRM-21810 changelog panel UI tweaks

CRM-21739 [NFC] Refactoring for SMS recipient test

CRM-21739 Test for getRecipients in/exclude mailing groups

Check for trueish required values

Merge pull request #11696 from mydropwizard/phpword-0.14.0

Update to PHPWord 0.14.0

Merge pull request #11723 from mukeshcompucorp/CRM-21805-fix-fields-and-labels-in-find-participants

CRM-21805: Fixed fields and labels in find participants

Merge pull request #11571 from magnolia61/CRM-21693_Show_Name_in_OnlinePayNow_UI

Crm 21693 show Display Name in online pay now UI

Update composer.lock for PHPWord update

Update to PHPWord 0.14.0

CRM-21805: Fixed fields and labels in find participants

Merge pull request #11719 from eileenmcnaughton/fatal

Throw exception instead of using fatal

CRM-21804 improve error handling on duplicate contribution.

Duplicate contributions are a really not handled well anywhere in the form layer.
The CRM_Core_Error winds up using the fatal template, which has the same end
result for form users as if an exception were thrown, let's switch.

Note that this appears to be the only scenario when an error would
be returned so we can simply drop that handling

Merge pull request #11718 from eileenmcnaughton/website

CRM-21803 Fix another entity to be standardised & support custom data

Fix another entity to be standardised & support custom data

Merge pull request #11688 from eileenmcnaughton/custom

Fix more entities to support custom fields (via api)

Merge pull request #11713 from yashodha/update-year

(NFC) Update copyright header for 2018

Merge pull request #11717 from seamuslee001/nfc_renewal_civilint_Fix

(NFC) MembershipRenewal.php - Fix civilint error

Merge pull request #11714 from mukeshcompucorp/CRM-21797-change-structure-of-campaign-search-pages

CRM-21797: Fixed structure for campaign search pages

Merge pull request #11694 from eileenmcnaughton/website

Move towards standardising website.create function.

NFC fix civilint error on MembershipRenewal.php

Move towards standardising website.create function.

CRM-21797: Fixed structure for campaign search pages

fix header

Fix more entities to support custom fields (via api)

Merge pull request #11154 from agileware/CRM-20421

CRM-20421: Inherited memberships are converted to individual memberships when the "parent" for the membership is merged with another contact

Merge pull request #11711 from seamuslee001/variable_rename_filter_fields

(NFC) Rename fiterable fields param in _civicrm_api3_basic_array_get …

Merge pull request #11709 from totten/master-extget

CRM-21792 Extension.get API - Fix regression (4.7.13) in result filtering

(NFC) Rename fiterable fields param in _civicrm_api3_basic_array_get to be more explicit about what it is for

api_v3_ExtensionTest::testExtensionGet - Be a little less flaky

The test has a magic number which seems to be based on the number of
extensions in the `drupal-clean` build profile -- which is basically the
smallest number anyone might have.  However, it gives false-negatives if you
have any other extensions around.  Make it a little less flaky.

Extension.get API - Only apply special filters once (key/keys/full_name)

Filtering extensions by key is a good idea -- so good we have three ways to do it! ;)

All of these fields accept a `string|array<string>` input, which is different from
the typical filtering supported by `_civicrm_api3_basic_array_get()` (which only
matches on values)...  If you happen to choose a basic filter that passes both
filters (i.e.  a simple string-match), then it works.  But if you choose a filter
where they behave differnetly (i.e.  an array), then you get weird results.

Extension.get API - Fix regression (4.7.13) in result filtering

Overview
--------

When one calls the `Extension.get` API, it should support filtering.  For
example, these commands would return only installed extensions or only
uninstalled extensions (respectively):

```
cv api Extension.get status=installed
cv api Extension.get status=uninstalled
```

Before
------

The implementation of `Extension.get` passes a list of all extensions down to
`_civicrm_api3_basic_array_get()` which is supposed to interpret any APIv3
options (such as filter-values/offsets/limits).  However, it confuses the
`return` list (i.e.  the names of any columns *that the user wants to see*) with the
*filterable* list (i.e.  the names of any columns *for which filtering is
permitted*).

After
-----

The implementation of `Extension.get` passes a list of all extensions down to
`_civicrm_api3_basic_array_get()`.  It also passes a fixed list of columns
which one might reasonably want to filter.

It might be nice to support filtering on all fields, but (a) several fields
wouldn't be sensible (because they're nested arrays) and (b) this provides a
smaller surface-area to maintain.

Comments
--------

The doc-block for `_civicrm_api3_basic_array_get()` describes the purpose of this field:

https://github.com/civicrm/civicrm-core/blob/4.7.0/api/v3/utils.php#L2390-L2391

Since ~v4.7.13, this line of code appears to have gone through several revisions, e.g.

 * The regression was originally introduced by 517dfba8b
 * Subsequent partial-fixes were d4c44c700, 9776f417, 34239e81, 525ccb68

CC @lucianov88  @twomice @tschuettler @seamuslee001 for review

Merge pull request #11706 from JMAConsulting/CRM-21789

CRM-21789: 'Find Case' group by issue

Merge pull request #11681 from jitendrapurohit/CRM-21778

CRM-21778 - Contact image uploaded from drupal webform don't render o…

test failure fix

Merge pull request #11707 from jitendrapurohit/CRM-21771

CRM-21771: error when viewing event registration with linked contribu…

CRM-21771: error when viewing event registration with linked contribution

CRM-21789: 'Find Case' group by issue

Merge pull request #11699 from totten/master-i18n-sql

(civicrm-setup/1) CRM_Core_I18n - Loosen coupling to DB layer

Merge pull request #11705 from totten/master-upgrade-ui-ver

/civicrm/upgrade - On finish screen, display actual version number

/civicrm/upgrade - On finish screen, display actual version number

This screen was hard-coded to always display the message "Thank you for
upgrading to 4.7...". This patch makes it dynamic, plugging in the
actual version number.

Merge pull request #11701 from samuelsov/tsfix

(NFC) remove starting whitespace in ts about installments

Merge pull request #11700 from JMAConsulting/CRM-21787

CRM-21787 : Simplify CRM_Utils_System::version() to fetch version directly from xml/version.xml

Merge pull request #11695 from JMAConsulting/civicrm-setup#11

[civicrm-setup#11] Remove templates/CRM/common/version.tpl

(NFC) remove starting whitespace in ts

CRM-21787: Keep  up-to-date without running GenCode on all builds

add version.tpl in checklist

Remove templates/CRM/common/version.tpl

add version.tpl in checklist

Merge pull request #11671 from eileenmcnaughton/nfc

Deduper - Pass arrays rather than strings to construct URLs

Merge pull request #11698 from reneolivo/patch-1

Fix wrong tag defintion

(civicrm-setup/1) CRM_Core_I18n - Loosen coupling to DB layer

Overview
--------

The `ts()`/`{ts}` feature from `CRM_Core_I18n` has an option to escape output
(e.g.  `{ts escape="sql"}Activities{/ts}`).  However, SQL encoding is subjective
(depending on the connection details). For `{ts}` to support this feature, it
must have a dependency on the DB subsystem (e.g. it eventually calls `CRM_Core_DAO`).

For https://github.com/civicrm/civicrm-setup/issues/1, we have an issue
where expressions like `{ts escape="sql"}Activities{/ts}` fail during
installation because `CRM_Core_DAO` is not fully available.

This change loosens the coupling, so that we can use `{ts escape="sql"}Activities{/ts}`
without needing `CRM_Core_DAO` per se.

Before
------

`ts`/`CRM_Core_I18n` is tightly coupled to `CRM_Core_DAO`. There is no way
to use `{ts escape=sql}Activities{/ts}` without spinning-up `CRM_Core_DAO`.

After
-----

`ts`/`CRM_Core_I18n` *defaults* to calling `CRM_Core_DAO`.  However, this can
be overriden by manipulating a property.

Comments
--------

* I feel a little dirty keeping any coupling between i18n and the DB.
  However, changing this would mean removing support for the
  `{ts escape=sql}` option, and that would be a clear compatibility-break.
* Arguably, there may be a microsecond-level penalty in using
  `call_user_func($SQL_ESCAPER)` rather than a specific class/function.
  However, it's only incurred if you actually call `{ts escape=sql}`
  while setting `$SQL_ESCAPER`, and that's pretty rare circumstance.
  The typical runtime use-cases for `ts()` are unaffected.

Fix wrong tag defintion

This notification template has a wrong HTML definition which makes it break a bit .

<img src="https://user-images.githubusercontent.com/1642119/36434765-36c7bf66-1636-11e8-84a5-2ffb75ef5eab.png" />

Merge pull request #11689 from eileenmcnaughton/groupOrg

Fix GroupNesting, GroupOrganization, Domain to work with singleValueAlter

Remove templates/CRM/common/version.tpl

Merge pull request #11692 from eileenmcnaughton/form

CRM-21391 Convert Activity to use core Task class

Merge pull request #11693 from eileenmcnaughton/cont_form

CRM-21391 Convert Contribute to use core Task class

Merge pull request #11682 from totten/master-i18n-preboot

(civicrm-setup#1) CRM_Core_I18n - Don't require immediate bootstrap

PHP doesn't allow abstract static functions.

Convert Contribute to use core Task class

Convert Activity to use core Task class

Merge pull request #11690 from mattwire/CRM-21781_dont_fatal_on_contact_view

CRM-21781 Don't crash CiviCRM when missing contact ID

Don't crash CiviCRM when missing contact ID

Fix GroupNesting, GroupOrganization, Domain to work with singleValueAlter

Merge pull request #11687 from totten/master-topsort

composer.json - De-fork dependency, marcj/topsort

composer.json - De-fork dependency, marcj/topsort

When this dependency was originally added, we needed a few patches (for PHP 5.3 compatibility)
and initially used a forked version of library. Of course, it's undesirable to use a fork
in the long term (e.g. harder to apply upgrades; harder to merge into other build processes).

In the intervening period, upstream has merged the patches for PHP 5.3, and
we've politely asked downstream to get over PHP 5.3, so we're covered on
both ends.  Let's get back on the mainline branch!

Merge pull request #11685 from colemanw/4-7-28-up

Ignore errors when upgrade step already took place

Merge pull request #11686 from civicrm/4.7.31-rc

4.7.31 rc

Merge pull request #11411 from mattwire/CRM-21558_case_batch_task

CRM-21558 Add batch update via profile for cases

Merge pull request #11676 from seamuslee001/CRM-21759

CRM-21759 Remove template variable as not needed and causing issues o…

Merge pull request #11670 from eileenmcnaughton/warn

CRM-21766 quash reload warning on dedupe conflicts screen

Merge pull request #11608 from eileenmcnaughton/mailing_custom

CRM-21140 extend support for custom data to Mailing api

Merge pull request #11496 from magnolia61/CRM-21639-NoIndex_when_event_is_not_public

CRM-21639 NOINDEX Event Info page when event is non-public or in the past

Ignore errors when upgrade step already took place

Merge pull request #11665 from JMAConsulting/CRM-21760-rc

CRM-21760: Manager Name is missing in 'Find Cases' search list

Remove unused var

Merge pull request #11677 from totten/master-xml-install

(civicrm-setup#1) Facilitate more direct installation

Merge pull request #11678 from mukeshcompucorp/CRM-21770-change-position-of-help-div-in-export-participant

CRM-21770: Change position of help div

(civicrm-setup#1) CRM_Core_I18n - Don't require immediate bootstrap

For civicrm/civicrm-setup#1, the general goal is to allow installing the
database schema without needing to run `GenCode`.

The current draft is crashing because the SQL does translation using `ts()`.
But if you try to use `ts()` in a pre-boot environment, it will attempt to
boot automatically so that it can read `$config->customTranslateFunction.

This is a chicken-egg situation.  We haven't yet reached the phase where the
installer can boot up Civi...  because we don't have the SQL...  but the SQL
can't be generated (translated) because Civi hasn't been booted.

The aim of this patch is to loosen the coupling between `ts()`
and `CRM_Core_Config` so that `ts()` can be used on its own.

> Aside: You might ask how this works today -- basically, `GenCode` does a
> database-less-boot, which placates `ts()`.  However, the `civicrm-setup`
> will eventually need to do full-boot, and AFAIK we don't have any
> situations where one transitions from database-less-boot to full-boot; I
> have a gut fear that such a transition would be its own slipper slope.

Merge pull request #11252 from mattwire/CRM-21405_outboundsms_mobile_notprimary

CRM-21405 Allow "Outbound SMS" when Mobile is not primary phone number

CRM-21778 - Contact image uploaded from drupal webform don't render on summary page

Allow "Outbound SMS" when Mobile is not primary phone number

Merge pull request #11675 from totten/master-phonecaps

Add/Edit Contact - Fix inconsistent capitalization

CRM-21770: Change position of help div

xml/GenCode.php - Only allow CLI execution

If we're going to bundle the `xml` folder so that it can be used by the
installer, then we don't this particular file to be web-accessible.

distmaker - Include xml schema in tarballs

CRM_Core_CodeGen_Specification - Allow use from civicrm-setup

For https://github.com/civicrm/civicrm-setup/issues/1, this change allows
`civicrm-setup` to read the XML metadata in the same way as GenCode -- which
helps us to install the current schema without any intermediate files.

CRM-21759 Remove template variable as not needed and causing issues on Joomla in 4.7.30

Add/Edit Contact - Fix inconsistent capitalization

In this context, there are several links: "Add another phone number", "Add
another IM", "Add another website". They should all be capitalized according
the same rules (e.g. standard sentence case).

Merge pull request #11667 from JMAConsulting/CRM-21765

CRM-21765: Commit files to git which are static but are generated from CodeGen like DAO files

Merge pull request #11585 from mukeshcompucorp/fix-page-structure

CRM-21708: Fixed page structure by adding/removing classes

CRM-21639 NOINDEX Event Info page when event is non-public

[NFC] pass arrays rather than strings to construct urls in dedupe code

When we pass a query it is urlencoded and any quotes in the string are not subsequently htmlentity encoded plus
I think the url construction is generally cleaner

CRM-21765: Commit files to git which are static but are generated from CodeGen like DAO files

CRM-21766 quash reload warning

Merge pull request #11622 from omarabuhussein/CRM-21733-membership-override-improvments

CRM-21733: Adding the ability to override membership until specific date

Merge pull request #11498 from magnolia61/CRM-21639_NOINDEX_Print_Preview_pages

CRM-21639_NOINDEX_Print_preview_pages

CRM-21760: Manager Name is missing in 'Find Cases' search list