Merge pull request #10768 from JMAConsulting/CRM-20947

CRM-20947, fixed notice error for deprecated function

CRM-20947, fixed notice error for deprecated function

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

CRM-20947, fixed notice error for deprecated function

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

Merge pull request #10764 from civicrm/4.7.23-rc

4.7.23 rc

Merge pull request #10750 from JMAConsulting/CRM-20947

CRM-20947, fixed notice error for deprecated function on new Bulk ent…

Merge pull request #10762 from seanmadsen/premium-images

Fix placement of call to simplifyURL() function

Merge pull request #10742 from eileenmcnaughton/dead

CRM-20953 Importing contacts with deceased_date not setting is_deceased

Merge pull request #10752 from seamuslee001/CRM-20961

CRM-20961 Fix issue where test on php7.1 was reporting that a non num…

Merge pull request #10761 from monishdeb/CRM-20821_CRM-20938

CRM-20821 & CRM-20938 : minor fix

Fix placement of call to simplifyURL() function

Before this commit, the URL simplification was performed within the
$params array used as the *default* values, so these values got
over-written by the submitted values with array_merge().

This is a follow-up to changes made in #10720 bb80d2f

CRM-20821 & CRM-20938 : minor fix

Merge pull request #10751 from seamuslee001/CRM-20960

CRM-20960 Upgrade dompdf to 0.8 for php7.1 compatability

Merge pull request #10747 from samuelsov/CRM-20954

CRM-20954 Fixing localized financial status problems

CRM-20961 Fix issue where test on php7.1 was reporting that a non numeric was being passed

CRM-20960 Upgrade dompdf to 0.8 for php7.1 compatability

CRM-20947, fixed notice error for deprecated function on new Bulk entry Batch

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

Merge pull request #10720 from seanmadsen/CRM-20821_CRM-20938

CRM-20821 & CRM-20938 - fix bugs in managing premium product images

Merge pull request #10723 from seamuslee001/CRM-20945

CRM-20945 Remove underline on active pager step

Merge pull request #10743 from JMAConsulting/CRM-20947

CRM-20947, fixed notice error for deprecated function on Bulk entry B…

Make the button a bit more prominiant

Merge pull request #10748 from seamuslee001/CRM-20943-2

CRM-20943 Fix subsequent issue found by Monish and Steven

CRM-20954 Initiate paid status id only once

CRM-20943 Fix subsequent issue found by Monish and Steven

Merge pull request #10686 from totten/master-sql

CRM_Utils_SQL_Select - Allow fluent query execution

CRM-20954 Fixing localized finantial status problems

Merge pull request #10653 from JKingsnorth/CRM-20863

CRM-20863 Improve report list ordering

CRM-20947, fixed notice error for deprecated function on Bulk entry Batch listing page

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

CRM-20953 Importing contacts with deceased_date not setting is_deceased

additional fix

Merge pull request #10732 from JMAConsulting/CRM-20947

CRM-20947, fixed warning for deprecated function while generating inv…

Merge pull request #10739 from civicrm/4.7.23-rc

4.7.23 rc merge to master

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 #10736 from eileenmcnaughton/reg

CRM-20950 Prt 2 Contact import mapping to wrong location type

CRM-20950 Prt 2 Contact import mapping to wrong location type

Merge pull request #10735 from eileenmcnaughton/reg

CRM-20950 Contact import mapping to wrong location type

CRM-20950 Contact import mapping to wrong location type

CRM-20947, fixed warning for deprecated function while generating invoice

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

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-20945 Remove underline on active pager step

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

CRM_Utils_SQL_SelectTest - Split into smaller test functions

CRM_Utils_SQL_Select::execute() - Improve docblock

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

CRM_Utils_SQL_Select - Allow fluent query execution

When I first wrote `CRM_Utils_SQL_Select`, I was a bit dogmatic about
loose-coupling and wanted the class to be entirely independent of the SQL
runtime. But this is a bit annoying in usage and training.

Before
======

To build and execute query, you had to pass the rendered SQL to the execute
function, eg

```php
$select = CRM_Utils_SQL_Select::from('mytable')
  ->select('...')
$dao = CRM_Core_DAO::executeQuery($select->toSQL());
while ($dao->fetch()) { ... }
```

After
=====

You can use more fluent style:

```php
$dao = CRM_Utils_SQL_Select::from('mytable')
  ->select('...')
  ->execute();
while ($dao->fetch()) { ... }
```

And you can chain with other DAO functions like `fetchAll()` or
`fetchValue()`.

```php
$records = CRM_Utils_SQL_Select::from('mytable')
  ->select('...')
  ->execute()
  ->fetchAll();
```

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