Merge pull request #10559 from civicrm/4.7.21-rc

4.7.21 rc

Merge pull request #10556 from devarun/4.7.21-rc

CRM-20644: Column names on Advanced search results displayed on contacts are out of order.

Merge pull request #10536 from yashodha/CRM-20753

CRM-20753 - Net amount doesn't respect localization

CRM-20644: Column names on Advanced search results displayed on contacts are out of order.

Merge pull request #10545 from totten/master-ui-bootstrap

CRM-20756 - Add Angular `ui.bootstrap` library

Merge pull request #10541 from JMAConsulting/CRM-20411

CRM-20411: mailing tab listing: MySQL 5.7 group by error

Merge pull request #10498 from JMAConsulting/CRM-20722

CRM-20722: 'Lock wait timeout exceeded' error triggered on smart group cache rebuild

Merge pull request #10407 from JMAConsulting/CRM-19653

CRM-19653: Custom field checkboxes migrated incorrectly on merge (part deux)

Merge pull request #10298 from colemanw/CRM-20387

CRM-20387 - Add invoice_number field for human readable contribution invoice

Merge pull request #10549 from jitendrapurohit/CRM-20761

CRM-20761 : Formrule error when priceset is selected on contribution page

CRM-20387 - Add invoice_number field for human readable contribution invoice

Merge pull request #10547 from eileenmcnaughton/extract

[NFC] Extract submit in mapFields to start adding unit testing (towards CRM-20759)

Extract submit in mapFields to start adding unit testin (CRM-20759)

Merge pull request #10544 from eileenmcnaughton/deprecated

CRM-20758 remove call to deprecated function & add test

CRM-20761: Formrule error when priceset is selected on contribution page

CRM-20758 remove one more deprecated call

Merge pull request #10542 from seamuslee001/CRM-20720

CRM-20720 CRM-20757 Fix issue where Price Options were not being sort…

CRM-20756 - Add Angular `ui.bootstrap` library

This adds the library
[ui.bootstrap](https://angular-ui.github.io/bootstrap/), which makes it
easier to include the functional elements of Bootstrap in AngularJS pages.

In the past, adding this was problematic, but now the situation has
improved:

1.  civicrm-core has upgraded to AngularJS 1.5.  (In the past, civicrm-core
used AngularJS 1.3, which was incompatible.)

2.  civicrm-core allows optional AngularJS modules.  (In the past, all
modules were loaded unconditionally, which could result in loading
new/unnecessary code for legacy users.  Now, you can opt-out of this
behavior with `basePages=>array()`.)

Merge pull request #10274 from eileenmcnaughton/activity2

CRM-19821 performance improvement on the activity search

CRM-19821 turn action links into relative links.

This is to fix the test in the first instance but discussed with Seamus & Monish & both
felt that relative links are more correct

CRM-19821 unit test

CRM-19821 performance improvement on the activity search.

This commit removes 2 bad joins from the activity search
- priority_id - this is just removed from the returnProperties as it is not rendered
- activity_status_id - this is handled as a pseudoconstant rather than the bad join

activity_type_id is partially handled, however, there is a component_id filter
that needs to be fixed before the bad join on this
field can be removed.

CRM-20758 remove call to deprecated function & add test

CRM-20720 CRM-20757 Fix issue where Price Options were not being sorted by weight

CRM-20411: mailing tab listing: MySQL 5.7 group by error

Merge pull request #10306 from aydun/CRM-20525

CRM-20525 Fix version of mail sent for pay later mails from webform

Merge pull request #9777 from jitendrapurohit/CRM-19745fix

CRM-19745: Image URL field doesn't show up on CiviEvent Additional Pa…

Merge branch '4.7.21-rc' into master

Merge pull request #10540 from colemanw/CRM-20091

CRM-20091 - Fix return value in api.customValue.gettree

CRM-20091 - Fix return value in api.customValue.gettree

CRM-20753 - Net amount doesn't respect localization

CRM-19745: Image URL field doesn't show up on CiviEvent Additional Participants Profile

Merge pull request #10534 from seamuslee001/CRM-20670

CRM-20670 (NFC) Style alter as per Eileen

Merge pull request #10515 from yashodha/CRM-20741

CRM-20741 - Cancellation message shown as error

CRM-20670 (NFC) Style alter as per Eileen

Merge pull request #10455 from JO0st/CRM-20670

    CRM-20670 fix editing membership types with a lot of members

Merge pull request #10487 from mfb/CRM-20713

CRM-20713 - CiviMail - do not set mode = 'sms' if sms_provider_id = 'null'

Merge pull request #10449 from jmcclelland/CRM-20666

CRM-20666 - allow filenames up to 255 characters

fixes editing membership type for membershiptypes with a lot of members

Merge pull request #10532 from monishdeb/CRM-20488_soft_credit_organization

CRM-20488: remove unused variable _honoreeProfileType

CRM-20488: remove unused variable _honoreeProfileType

Merge pull request #10528 from tschuettler/CRM-20746

CRM-20746 - CiviMail - Use body_text for text part of resubscription …

CRM-16395: Installation with localized default settings (4) (#10518)

CRM-16395: Installation with localized default settings

This makes it possible to have a file in "civicrm/l10n/xx_YY/settings.default.json" with default settings that correspond to the locale during installation (ex: default country, currency, date formats, etc).

Merge pull request #10432 from francescbassas/patch-11

CRM-20650 Translate strings

Merge pull request #10424 from twomice/CRM-20401_subscription_urls

Fix CRM-20401: use native CiviCRM recurring links

Merge pull request #10530 from davejenx/CRM-20443-master

CRM-20443 - SQL syntax error creating logging triggers if column name…

Merge pull request #10419 from twomice/CRM-20488_soft_credit_organization

Fix CRM-20488: remove restrictions on soft credit contact type.

Merge pull request #10447 from JMAConsulting/CRM-19478

CRM-19478: Handle p=null oddity for Paypal Pro Recurring payments

minor fix

Additional for CRM-20488: remove useless preProcess() method.

Fix CRM-20488: remove restrictions on soft credit contact type.

Merge pull request #10524 from jitendrapurohit/CRM-20745

CRM-20745: Post date of recur not respected in credit card pledge pay…

Merge pull request #10521 from JMAConsulting/CRM-20495

CRM-20495 : 'Contribution amounts section' checkbox setting on contribution pages always shows as checked

Merge pull request #10457 from jitendrapurohit/CRM-20675

CRM-20675: Fix incorrect renewal activity created when membership upd…

Minor fix at unit test

CRM-20443 - SQL syntax error creating logging triggers if column name is reserved word - 4.7 version

Merge pull request #10467 from JMAConsulting/CRM-20682

CRM-20682: Include human readable contribution's custom field label in token widget for Thankyou letter

CRM-20746 - CiviMail - Use body_text for text part of resubscription confirmation mails

----------------------------------------
* CRM-20746: CiviMail - text part of resubscribe confirmation mail contains html
  https://issues.civicrm.org/jira/browse/CRM-20746

Add test

Merge pull request #10526 from civicrm/4.7.21-rc

4.7.21 rc

Merge pull request #10523 from seamuslee001/animated_image

(NFC) re add in animated-overlay.gif is no longer provided by jquery-ui

Merge pull request #10525 from seamuslee001/contribution_test_rigger_fix

(NFC) Fix clearing messages to ensure test accuracy

(NFC) Fix clearing messages to ensure test accuracy

CRM-20745: Post date of recur not respected in credit card pledge payment

Merge pull request #10522 from colemanw/CRM-20743

CRM-20743 - Check tag permissions in UI and BAO

Apply mlutfy changes

(NFC) re add in animated-overlay.gif is no longer provided by jquery-ui

Apply mlutfy changes

Remove stray semicolon

CRM-20743 - Check tag permissions in form layer

Merge pull request #10429 from JMAConsulting/CRM-20622

CRM-20622 , CRM-20673 : tags and groups panel layout/styling

QA fixes

CRM-20743 - Check tag permissions in UI and BAO

CRM-20673: Tag and group edit form: implement Select2 for tags

CRM-20622: contact edit: tags and groups panel layout/styling

CRM-20495 : 'Contribution amounts section' checkbox setting on contribution pages always shows as checked

Merge pull request #10441 from JMAConsulting/CRM-20621

CRM-20621 : manage tags: the tag usage count is not accurate

Set version to 4.7.22

Merge pull request #10085 from totten/master-assets

CRM-20600 - AngularJS - Allowing hooking and static-caching

CRM-20600 - AssetBuiler - Add separate setting, `assetCache`

The default value (`auto`) behaves the same as the current behavior: enable
caching in production mode, disable caching in debug mode.

But it also allows you to enable/disable caching independent of the
prod/debug mode.

Merge pull request #9790 from fliespl/patch-3

CRM-19896 - customTranslateFunction not mapped

Merge pull request #10519 from johanv/attribution_chirojeugd

(NFC) Attribution Chirojeugd Vlaanderen

CRM-20600, CRM-20112 - AngularLoader - Add helper for base-pages and injections

This commit extracts the `Civi\Angular\Page\Main::registerResources()` and
creates a new utility-class, `AngularLoader`.  The `AngularLoader` can be
used in new Angular base-pages, e.g.

```php
class Example extends CRM_Core_Page {
  public function run() {
    $loader = new \Civi\Angular\AngularLoader();
    $loader->setPageName('civicrm/foo/bar');
    $loader->setModules(array('crmApp', '...'));
    $loader->load();
    return parent::run();
  }
}
```

The `AngularLoader` only loads the resources or assets (JS/CSS/HTML files).
To start Angular, you'll need to call `ng-app` or `angular.bootstrap(...)`.
One way to do this is to define a page-template:

```html
<!-- Example.tpl -->
<div ng-app="crmApp">
  <div ng-view></div>
</div>
```

Or you can reuse the existing template:

```php
  public function getTemplateFileName() {
    return 'Civi/Angular/Page/Main.tpl';
  }
```

Note: This is framed as a utility-class which loads the Angular resource
files.  It's not a page-class/base-class.  This approach allows us to call
the utility from other situations -- e.g.  inject AngularJS onto an
pre-existing page via hook.  Doing that may or may not be wise, but the
class-hierarchy shouldn't be the issue.

CRM-20600 - Civi\Angular\Manager - Add resolveDependencies(), resolveDefaultModules()

CRM-20600 - CRM_Core_Resources - Add hook for settings data

CRM-20600 - Civi\Angular\Manager - Add hook for modifying partials and dependencies

CRM-20600 - Civi\Angular\ChangeSet - Implement a utility for modifying HTML snippets via phpQuery

CRM-20600 - composer.json - Add phpQuery

CRM-20600 - Angular deps - Add missing/implicit dependencies

Several modules depended on `crmResource` but didn't declare it.

CRM-20600 - Angular deps - Move from *.js to *.php

If third-parties are allowed to alter the HTML content, then they may
introduce new dependencies.  This means that they'll need to delcare those
dependencies.

This commit actually moves the declarations for any in-house modules to PHP.

CRM-20600 - Angular deps - Allow declaration in PHP code

If third-parties are allowed to alter the HTML content, then they may
introduce new dependencies.  This means that they'll need to delcare those
dependencies.

Tangentially, this will also make it easier to construct more optimized
base-pages (e.g.  "make a base-page with module X plus all its
dependencies").

CRM-20600 - civicrm/a - Avoid dirty caches for permissioned modules

The default behavior is to include a `bundleUrl` for a bundle of "all
Angular modules".  However, some Angular modules like `crmMailing` are only
displayed if the current user has permission to access them.  This means
that the list of "all Angular modules" can be different (depending on who
you are; albeit in practice there's a small# of permutations).

This patch ensures that two users who access a different list of modules
will see a different `bundleUrl`.

CRM-20600 - Civi\Angular\Manager::getStrings() - Scan raw HTML strings instead of *.html files

Rationale: If the raw HTML can be modified via hook, then the HTML files are
not truly representative of the final output.  (e.g.  a hook might add a new
string to some HTML.)

Note: This patch means that we no longer benefit from the cache in
`CRM_Core_Resources_Strings`.  However, the typical user still pulls data
from a cache because we're using `AssetBuilder` to manage
`angular-modules.json`.

Note: `getStrings()` now requires a call to `getPartials()`, which is
expected to become more expensive (firing a hook).  To avoid building the
partials twice, we introduce a private cache.

CRM-20600 - PartialSyntaxTest - Ensure that Angular partials are well-formed

CRM-20600 - Civi\Angular - Generate modules via AssetBuilder

In the typical use-case, AssetBuilder can avoid an extra CMS+Civi bootstrap
along with any build steps.

CRM-20600 - Implement AssetBuilder - See extended docblock in AssetBuilder.php

CRM-20600 - Move Angular metadata to civix-style files

This PR moves the metadata about Angular modules out of Civi\Angular\Manager
and into standalone files (`ang/*.ang.php`).  This uses the same structure
as `civix`.  Also, this makes it easier to navigate between the
JS/HTML/CSS/PHP content.

To test that these changes were safe, I ran the following command before and
after this commit:

$ cv ev 'var_dump(Civi::service("angular")->getModules());'

The results were substantively identical.

Merge pull request #10520 from seamuslee001/CRM-20744

CRM-20744 Use images in the bower_components version of datatables plugin rathe…

Use images in the bower_components version of datatables plugin rather than packages

Merge pull request #10495 from seamuslee001/CRM-20243-phpword

CRM-20243 Remove outdated sample jquery.min.js from phpword package