additional fix

Don't use array dereferencing (for PHP 5.3 compat)

CRM-20938 - Maintain aspect ratio in resizeImage()

Make CRM_Utils_File::resizeImage() preserve image aspect ratio by
default.

CRM-20821 - Use smarter logic to adjust img URL

This change is the crux of CRM-20821.

Before this change, editing a premium product would automatically
break the image URL. After this change, premium products can be edited
while retaining the image URL. Functionality is preserved which changes
the URL to a local URL when possible.

This commit adds two new Utils functions (along with tests) to handle
the logic of changing a supplied URL to a local URL.

CRM-20821 - Improve ManagePremiums formRule()

- Improve CRM_Contribute_Form_ManagePremiums::formRule()
- Require an image file, if "upload" is chosen
- Remove the commented-out code for a rule which required images to be
  within a certain size
- Minor refactoring to improve code clarity

CRM-20821 - Refactor ManagePremiums postProcess()

- Refactor CRM_Contribute_Form_ManagePremiums::postProcess()
- No major changes to functionality
- Eliminate deeply nested code (addressing "FIXME" comment)
- Split code into multiple functions
- Add more comments

CRM-20821 - Refactor PremiumProduct BAO add()

- Refactor CRM_Contribute_BAO_ManagePremiums::add
- Don't change any functionality
- Improve docblock
- Use a $defaults array

CRM-20821 - Refactor resizeImage()

- Clean up CRM_Utils_File::resizeImage()
- No major functionality changes
- Better docblock
- Better error handling
- Better variable names

CRM-20821 - Move resizeImage() to Utils class

- Move resizeImage() from CRM_Contribute_Form_ManagePremiums to
  CRM_Utils_File
- Make it public static
- Don't change anything about how it works

Merge pull request #10664 from jitendrapurohit/CRM-20875

CRM-20875 - Fix membership import to avoid notice error to be thrown

Merge pull request #10727 from JMAConsulting/CRM-20947

CRM-20947, fixed notice error deprecated CRM_Core_OptionGroup::getLabel

Merge pull request #10724 from seamuslee001/CRM-19941

CRM-19941 CRM-20943 Fix issue in PHP7.1 using  is non object mode in …

CRM-20947, ficed notice error deprecated CRM_Core_OptionGroup::getLabel

----------------------------------------
* CRM-20947: Remove Deprecation Notice for Option group
  https://issues.civicrm.org/jira/browse/CRM-20947

Merge pull request #10722 from seamuslee001/CRM-20944

CRM-20944 Remove call to deprecated CRM_Core_OptionGroup::getValue in…

CRM-20944 Remove call to deprecated CRM_Core_OptionGroup::getValue in ActivityLinks.php

CRM-19941 CRM-20943 Fix issue in PHP7.1 using  is non object mode in AtoZ Pager

Merge pull request #10715 from totten/master-jdebug

CRM-19575 - CRM_Utils_System_Joomla - Fix warning. Define JDEBUG

Merge pull request #10710 from jitendrapurohit/participantfix

CRM-17236 - Custom dates fields displayed as (01/01/1970) in Events confirmation receipt

Fix participant mails

Merge pull request #10704 from eileenmcnaughton/admin_form

CRM-20923 refactor tpl to be based on metadata not hard-coding

style fix

Merge pull request #10676 from fuzionnz/CRM-20884-remove_net_smtp_readme_symlink

Remove broken symlink from vendor/pear/net_smtp

Merge pull request #10668 from jmcclelland/CRM-20876

CRM-20876 - ensure honor block variable is passed to template

Merge pull request #10713 from mfb/CRM-20931

CRM-20931 - Allow contact custom fields to be added to Contribution Detail report

Merge pull request #10714 from totten/master-gitignore

(NFC) gitignore - Remove obsolete entries

CRM-20932 - CRM_Utils_System_Joomla - Fix warning. Define JDEBUG

Background
==========

When running `cv en <extname>` on Joomla, there are repeated warnings, "Use
of undefined constant JDEBUG", and each includes a backtrace.  This is hard
to read.

The bug appears to affect backend scripts; it's discussed with a solution in:

https://github.com/joomla/joomla-cms/issues/11512#issuecomment-244937966

Before
======

`CRM_Utils_System_Joomla::loadBootStrap()` loads various config files.
However, these may or may not set `JDEBUG`.

After
=====

`CRM_Utils_System_Joomla::loadBootStrap()` defensively sets `JDEBUG` if no
other part of the bootstrap has done so.

Merge pull request #10711 from colemanw/CRM-20929

CRM-20929 - Convey css classes to main page title from Angular

CRM-20929 - Simply add crm-page-title class

(NFC) gitignore - Remove obsolete entries

The `packages` directory is actually a separate git repo, with its own copy
of `.gitignore`.

These rules have no effect in there.

The `packages` directory is ignored as a whole with one entry in
`gitignore`.

CRM-20923 refactor tpl to be based on metadata not hard-coding

Change-Id: Iee2a7aa9fa93c5d20b95f3dbc45ec2f3cf7a216b

CRM-20931 - Allow contact custom fields to be added to Contribution Detail report

As a bonus also add nickname column to the report :)

Merge pull request #10430 from JMAConsulting/IIDA-144

CRM-20648, fixed validation to allow participant to register if their…

Merge pull request #10706 from totten/master-regen-finact

regen.sh - Fix stale function call in sql/GenerateData.php

CRM-20929 - Convey css classes to main page title from Angular

CRM-20648 Applied check exclusively to cancelled status

----------------------------------------
* CRM-20648: Allow registration in event with cancelled registration
  https://issues.civicrm.org/jira/browse/CRM-20648

CRM-20648, fixed validation to allow participant to register if their previous registration is cancelled

----------------------------------------
* CRM-20648: Allow registration in event with cancelled registration
  https://issues.civicrm.org/jira/browse/CRM-20648

regen.sh - Fix stale function call in sql/GenerateData.php

In [#10463](https://github.com/civicrm/civicrm-core/pull/10463), the
function signature of `getInstrumentFinancialAccount()` changed slightly -- in that
the parameter became mandatory.

Grepping for other calls to `getInstrumentFinancialAccount()` shows that one of the unit-tests
works with example input `4`, so this does the same.

Before
======

Running "regen.sh" fails -- because `getInstrumentFinancialAccount` is called without a parameter.

After
=====

Running "regen.sh" completes -- because `getInstrumentFinancialAccount` is called with `4`.

Acceptance Prompts
==================

 * The example data produced by `regen.sh` should contain suitable/similar values for financial records.
 * Determine whether `4` is a suitable value.

Merge pull request #10682 from JMAConsulting/CRM-20890

CRM-20890: Move recieve_date between Contribution status and Revenue Recognition date in Contribution backoffice form

Merge pull request #10660 from seamuslee001/CRM-20860-processor

CRM-20860 Allow payment processor field to use the Password QuickForm Field type

Set version to 4.7.24

Merge pull request #10683 from jitendrapurohit/CRM-20770

CRM-20891 : Separate membership pay later contribution sets the membe…

add test

Merge pull request #10697 from seamuslee001/4.7.22-rc

4.7.22 rc

Add 4.7.22 to release-notes.md

Add release notes for 4.7.22

CRM-20600 - Upgrader - Display warning if asset-caching is likely to fail

After releasing v4.7.21, there were several inquiries about sites in which
Angular pages failed to load.  In cases diagnosed so far, this has been a
result of inconsistent configurations or overzealous HTTP restrictions
applied to the `imageUploadDir`.

Ordinarily, we would address this kind of thing with a status check.
Unfortunately, the problem breaks that "Status" page...  so we have to
display a bit sooner.

Merge pull request #10690 from JMAConsulting/CRM-20902

[ready for review] CRM-20902 Fixed mailing opened query count for pie chart

Merge pull request #10689 from JMAConsulting/CRM-20901

[ready for core team to review]CRM-20901, Added submit credit card membership link

Merge pull request #10652 from jitendrapurohit/CRM-19017

CRM-19017 : Load default params for fragment query object and fix db error.

CRM-20902 Fixed mailing opened query count for pie chart

----------------------------------------
* CRM-20902: DB Syntax error on Mailing Opened Report
  https://issues.civicrm.org/jira/browse/CRM-20902

CRM-20901, Added submit credit card membership link

----------------------------------------
* CRM-20901: Add submit credit card membership link on membership form
  https://issues.civicrm.org/jira/browse/CRM-20901

Merge pull request #10649 from seamuslee001/CRM-20860

CRM-20860 Add in ability to create field type of password from metadata

CRM-20770 : Separate membership pay later contribution sets the membership to Completed

CRM-20890: Move recieve_date between Contribution status and Revenue Recognition date in Contribution backoffice form

Merge pull request #10627 from eileenmcnaughton/sett

CRM-20837 Make setting bug more explicit

Merge pull request #10662 from seamuslee001/CRM-20873

CRM-20873 : Fix issue where getTree try to load is_public field before it is added in upgrade

Minor fix

CRM-20873 FIx issue where getTree trys to load is_public field before it is added in upgrade

Merge pull request #10537 from PalanteJon/20754

CRM-20754 - Clear memory leak in CSV CLI import

Use the freeResult DAO method

Remove vendor/pear/net_smtp/README.rst also.

CRM-20884

Merge pull request #10675 from JMAConsulting/CRM-20848

CRM-20848: Upgrade to 4.7.19 sets price_field to inactive if default Financial Types are disabled

CRM-20848: Upgrade to 4.7.19 sets price_field to inactive if default Financial Types are disabled

Merge pull request #10599 from eileenmcnaughton/api_no_id

CRM-20847 Support custom api with composite primary keys

CRM-19017 : Load default params for fragment query object.

Merge pull request #10669 from seamuslee001/clear_all_messages

Clear all Messages when running clearMessages by default

Clear all Messages when running clearMessages by default

Merge pull request #10667 from mlutfy/master-crm20803-1

CRM-20803: Enable Farsi/Persian, Serbian and Ukranian.

CRM-20876 - ensure honor block variable is passed to template

Otherwise, it's not on the receipt.

CRM-20803: Enable Farsi/Persian, Serbian and Ukranian.

Merge pull request #10463 from JMAConsulting/CRM-20677

[ready for core team review] CRM-20677, used generalized function to retrieve financial account id

Merge pull request #10646 from JMAConsulting/deferred

[ready for core team review] CRM-20857: fixed Deferred Report for sql_mode=only_full_group_by

Fix membership import to avoid notice error to be thrown

Merge pull request #10659 from totten/master-crmuiselect

CRM-20871 - crmUiSelect - UI should stay in sync with model

Allow payment processor field to use the Password QuickForm Field type

CRM-20871 - crmUiSelect - UI should stay in sync with model

The `crmUiSelect` defines the function `$render()` -- whenever the model
changes, it should update execute `$render()` to update the UI.  But it doesn't.

This issue is discussed more in https://stackoverflow.com/questions/21084088/why-ngmodels-render-is-not-called-when-the-model-changes-in-angularjs

== Before ==

The `$render()` function for directive `crm-ui-select` is not called because the `select`
directive overrides it.

== After ==

The `$render()` function for directive `crm-ui-select` is called, even if
used on a `select` element.

== Acceptance Criteria ==

 * The test page in https://github.com/totten/org.example.angpage (`civicrm/a/#/play`)
   keeps elements synchronized.
 * Existing screens which use `crm-ui-select` continue to work.

Merge pull request #10618 from colemanw/CRM-20830

CRM-20830 - Improve handling of overdue activities

Merge pull request #10656 from JMAConsulting/fatalerror

 [ready for core team to review]CRM-20867, fixed fatal error

CRM-20867, fixed fatal error

----------------------------------------
* CRM-20867: Class not found
  https://issues.civicrm.org/jira/browse/CRM-20867

Code cleanup

CRM-20830 - Trinary activity status types

CRM-20830 - Improve handling of overdue activities

Merge pull request #10586 from colemanw/CRM-20794

CRM-20794 - Colors for case status

CRM-20860 Add in ability to create field type of password from metadata

CRM-20857, fixed report for sql_mode=only_full_group_by

----------------------------------------
* CRM-20857: Deferred Revenue report -  DB Error: unknown error
  https://issues.civicrm.org/jira/browse/CRM-20857

fixed style

Merge pull request #9784 from monishdeb/CRM-19702

CRM-19702: Fatal error when merging contact records with custom file fields

Merge pull request #10637 from mickadoo/CRM-20849-allow-duplicate-psr4-prefix

CRM-20849: Allow duplicate PSR-4 prefixes

CRM-20851 php notice (#10639)

 Fix php notice when is_primary is not set

CRM-19702: Fatal error when merging contact records with custom file fields

Merge pull request #10638 from mattwire/CRM-20772_statusbounce

CRM-20850 Replace fatal with statusBounce

Merge pull request #10604 from jitendrapurohit/CRM-20809

CRM-20809 : Modify extension cache file to get created in uploads directory

Merge pull request #10642 from civicrm/revert-10370-CRM-20321

Revert "CRM-20321, CRM-20652 : Changing membership type should change related contribution and generalise changeFeeSelection function to be used for any entity"

Revert "CRM-20321, CRM-20652 : Changing membership type should change related contribution and generalise changeFeeSelection function to be used for any entity"

modify fix to place cache dir in upload dir

test failure fixes

CRM-20677, updated getInstrumentFinancialAccount() to use api

----------------------------------------
* CRM-20677: Use generalised function to retrieve financial account
  https://issues.civicrm.org/jira/browse/CRM-20677

CRM-20677. more changes

----------------------------------------
* CRM-20677: Use generalised function to retrieve financial account
  https://issues.civicrm.org/jira/browse/CRM-20677

CRM-20677, used generalized function to retrieve financial account id

----------------------------------------
* CRM-20677: Use generalised function to retrieve financial account
  https://issues.civicrm.org/jira/browse/CRM-20677

Merge pull request #10431 from JMAConsulting/NoticeFixes-1

[ready for core team to review] CRM-20649, Fixed notice error on form rule

Merge pull request #10450 from JMAConsulting/CRM-20667

[ready for core team to review] CRM-20667, fixed addtional line creation when multiple membership is …

Replace fatal with statusBounce