CRM-18701 tidy up, do not pass variables around as a reference when they are not being altered

CRM-18701 tidy up, stop setting an array key & use a variable.

The array key is returned to the calling function but grepping did not find any more instances of it so we should limit it's scope to the function that uses it.

Really we want to pass in  as a non-reference or else it's too unpredictable but this is just a small step

Merge pull request #8426 from Anjali2906/crm_18560_validation

CRM-18560 - Validate either body text or body html

Merge pull request #8490 from eileenmcnaughton/trans

CRM-18661 Fix & add test for financial rows created by repeattransaction

Merge pull request #8504 from rollox/patch-2

CRM-18743 - Undefined variable $order

CRM-18661 further fix to ensure payment processor account is used

Merge pull request #8312 from colemanw/CRM-18344

CRM-18344 - Show attachments on CiviMail review panel

Merge pull request #8342 from rohankatkar/CRM18472NoticeFix

CRM-18472: Notice fix

CRM-18743 - Undefined variable $order

Fixes 3 notice/errors stating "Undefined variable: order in CRM_Contact_BAO_Query->prepareOrderBy()" on Drupal user page when the user record is linked to a Civi contact record. eg at path: user/1

Merge pull request #8499 from eileenmcnaughton/strings

CRM-10210 add test for international strings

Merge pull request #8475 from johanv/CRM-18696-addresses_in_reports_without_address_parsing

CRM-18696 - Street address filtering in reports for sites w/o address parsing

Merge pull request #8318 from AdamHillier/CRM-17657

CRM-17657 - Update ReCAPCHA validation text to reflect new style of captcha.

CRM-10210 add test for international strings

Fix wide date input in CiviMail

CRM-18344 - Show attachments on CiviMail review panel

Merge pull request #8408 from AdamHillier/bug-fix

Typo fix: remove space before question mark.

Merge pull request #8435 from totten/master-regen-cms

bin/regen.sh - Execute GenerateData.php after initializing other database structures

Merge pull request #8495 from seamuslee001/CRM-18652

CRM-18652 Show no results label in quick search

Merge pull request #8421 from ineffyble/correctly-update-test-payment-processor-urls

CRM-18625 - payment processors - correctly set test URLs from edit form

CRM-18751 - Add method for setting payment processor return URL (#8488)

* CRM-18751 - Add method for setting payment processor return URL

* CRM-18751 - Restore protected visibility of functions

Merge pull request #8491 from seamuslee001/CRM-18752

CRM-18752 Fix Custom multi value importer

Merge pull request #8142 from monishdeb/CRM-17779

CRM-17779: Authorize.Net payments held for fraud detection aren't logged in CiviCRM

Merge pull request #7797 from JKingsnorth/CRM-17977

CRM-17977 Usability of allowed countries/states

CRM-18661 Fix & add test for financial rows created by repeattransaction

CRM-18652 fix bug found in testing

CRM-18652 - Localize label for no results

----------------------------------------
* CRM-18652: Quick search should provide feedback to user if no matches are found
  https://issues.civicrm.org/jira/browse/CRM-18652

CRM-18652 - Show 'no matches' if quick search returns no results

----------------------------------------
* CRM-18652: Quick search should provide feedback to user if no matches are found
  https://issues.civicrm.org/jira/browse/CRM-18652

Merge pull request #8483 from colemanw/CRM-18742

CRM-18742 - Focus first text field on search forms

Merge pull request #8460 from colemanw/CRM-18660

CRM-18660 - Fix custom field year range selection

Merge pull request #8233 from PalanteJon/CRM-18468

CRM-18468 - Recurring Contributions - don't use mid for subscription …

Add in fixes for CRM-18708 as well as its needed on 4.7 and make minor fix as suggested by Coleman

CRM-18752 Fix Custom multi value importer

Merge pull request #7728 from andrew-cormick-dockery/CRM-17959

CRM-17959 - allow URL to be redirected if the queue ID is invalid but…

Merge pull request #8485 from colemanw/CRM-18744

CRM-18744 - Remember quicksearch field selection

Merge pull request #8484 from colemanw/actCustomSearch

Fix undeclared variables in custom activity search

Remove deprecated param

Merge pull request #8332 from TobiasLounsbury/CRM-17888-freq-unit

CRM-17888: Updated frequency unit to type T_STRING

Merge pull request #8472 from cividesk/CRM-10230-4.7

CRM-10230 Custom fields text type with limited length not validated

CRM-18744 - Remember quicksearch field selection

Fix undeclared variables in custom activity search

CRM-18742 - Focus first text field on search forms

Merge pull request #8482 from JKingsnorth/CRM-18739

CRM-18739: Display currency for canceled contribution amount

Merge pull request #8481 from yashodha/CRM-18699

CRM-18699: fix generated data for the typo

Merge pull request #8430 from yashodha/CRM-18655

CRM-18655: Selected id's not escaped properly

Merge pull request #8451 from yashodha/CRM-18139

CRM-18139: Notification needed when using Batch Update of Participant…

CRM-18739: Display currency for canceled contribution amount

CRM-18699: fix generated data for the typo

----------------------------------------
* CRM-18699: Wake Island is misspelled
  https://issues.civicrm.org/jira/browse/CRM-18699

Merge pull request #8477 from mpnordland/master

CRM-18699 Fixed Wake Island spelling

CRM-18699 fix in 4.7.9 now, better where statment

Merge pull request #8328 from mlutfy/master-crm18524

CRM-18524: Fix db error when exporting contributions (phone_type_id is not a real ID field).

Merge remote-tracking branch 'origin/4.7.8-rc' into 4.7.8-rc-master-2016-06-02-00-34-42

CRM-18699 Fixed Wake Island spelling

Merge pull request #8438 from yashodha/CRM-18659

CRM-18659: inappropriate place for upgrade script

CRM-18696 - Street address filtering in reports for sites without address parsing.

----------------------------------------
* CRM-18696: Street filter in Constituent Summary report broken without street address parsing
  https://issues.civicrm.org/jira/browse/CRM-18696

CRM-10230 Custom fields text type with limited length not validated

Test formatting and changed to base_convert

bin/regen.sh - Run DatabaseInitializer (first boot) before GenerateData

The DatabaseInitializer is called during first boot
(`CRM_Core_Config::singleton()`) and runs `system.flush session=1
triggers=1` which (among other things) enumerates modules (including CMS
modules).  In standalone boot protocol, that means it runs before
`CRM_Utils_System::loadBootstrap()` (which means that CMS modules
cannot be enumerated yet).

A simple work-around is to always boot the CMS first (before
`CRM_Core_Config::singleton()` or `DatabaseInitializer` runs).

bin/regen.sh - Execute GenerateData.php with a dummy CMS

1. This addresses the error [PHP Fatal error:  Call to undefined function db_query() in /opt/buildkit/build/mytestbuild/sites/all/modules/civicrm/CRM/Utils/System/DrupalBase.php](http://civicrm.stackexchange.com/questions/11972/regen-sh-call-to-undefined-function-db-query/11986#11986).

2.  Generally, the goal here is to produce `civicrm_generated.mysql`, which
is used on any/all CMS's to initialize the Civi database.  If the generation
were really dependent on Drupal 7, then we wouldn't be able to use that
output on other CMS's, right?  Thus I'm hopeful that we can generate
`civicrm_generated.mysql` with any `CIVICRM_UF`.  And the `UnitTests` UF has
fewer dependencies/interactions.

3. Since `GenerateData.php` is running in a dummy CMS, we can remove
these weird mock-drupal-functions.

4. To test, I ran `regen.sh` and compared the output with the old one.
All the changes looked like random-data (`civicrm_contact`, etal) and
not structural data (`civicrm_option_value`, etal).

Merge pull request #8452 from jitendrapurohit/CRM-18677

CRM-18677 - Bugs with multivalued custom data displayed Inline.

CRM_18560 Added test for empty body mailing component

Merge pull request #8441 from rajbdilip/CRM-17887_2

CRM-17887 - Add a new setting `Financial Batch date`

CRM-18660 - Fix custom field year range selection

This fixes the yearRange setting to be the responsibility of the crmDatepicker
widget instead of setting it from php. The php implementation was also buggy.

Merge pull request #8445 from eileenmcnaughton/merge_rc

CRM-18673 fix merge handling of identifal addresses

Merge pull request #8458 from tommybobo/select2css-fix

CRM-18691 Select2 Height being squashed when defult is blank

CRM-18691 Select2 Height being squashed when defult is blank

Merge pull request #8457 from yashodha/CRM-18676

CRM-18676: Incorrect status for pay later free membership

CRM-18676: Incorrect status for pay later free membership

----------------------------------------
* CRM-18676: Incorrect status for pay later free membership
  https://issues.civicrm.org/jira/browse/CRM-18676

CRM-18676: Incorrect status for pay later free membership

----------------------------------------
* CRM-18676: Incorrect status for pay later free membership
  https://issues.civicrm.org/jira/browse/CRM-18676

Merge pull request #8436 from civicrm/eileenmcnaughton-patch-1

Declaring required as false on timestamp fields causes allow null in …

CRM-17887 - Add a new setting `Financial Batch date`

https://issues.civicrm.org/jira/browse/CRM-17887

He's likely referencing the setting at Administer > Localization > Date Formats, Complete Date fields. Unfortunately, those settings don't include a convenient one for this purpose, since by default they include text names of months, etc. So, added a new field exclusively for Financial Batch.

CRM-18677 - Bugs with multivalued custom data displayed Inline.

string fixes

CRM-18139: Notification needed when using Batch Update of Participants via Profile (currently only when using Change Participant Status function)

----------------------------------------
* CRM-18139: Notification needed when using Batch Update of Participants via Profile (currently only when using Change Participant Status function)
  https://issues.civicrm.org/jira/browse/CRM-18139

CRM-18139: Notification needed when using Batch Update of Participants via Profile (currently only when using Change Participant Status function)

----------------------------------------
* CRM-18139: Notification needed when using Batch Update of Participants via Profile (currently only when using Change Participant Status function)
  https://issues.civicrm.org/jira/browse/CRM-18139

Merge pull request #8446 from totten/mollux-CRM-18510

CRM-18510 - Fix regression when searching for location types

Merge pull request #8443 from colemanw/CRM-17607

CRM-17607 - Add docx, odt, html document export functionality

CRM-17607 - Fix document showing up as activity type

CRM-17607 - Support multiple document formats for export

CRM-17607 - Change PDF activity type label

CRM_Contact_BAO_Query - Extract method `prepareOrderBy()` (via PHPStorm)

The fixes on `searchQuery` are making a bad function even harder to read.
In keeping with https://wiki.civicrm.org/confluence/display/CRM/Toxic+Code+Protocol , we
need to some cleanup at the same time.

This was generated using PHPStorm's automated refactoring, "Extract Method".

CRM_Contact_BAO_QueryTest - Update assertion on ORDER BY to use backtick

Order by clauses are now properly escaped.

CRM_Contact_BAO_Query::searchQuery - Preserve $order until last minute (EVIL)

Building on https://gist.github.com/mollux/7ffeb63f55ba0ba6180968d6d36f4745

This function does a lot of string-analysis on `$order`, and proper escaping
breaks this. However, the CRM_Utils_Type::escape() is quite clever about
validating and applying escaping.

CRM-18673 fix merge handling of identical addresses

CRM-17607 - Add phpoffice/phpword library

Fixed tests for loosened MysqlColumnNameOrAlias validation.

Fixed wrong dash.

Loosen the MysqlColumnNameOrAlias validation.

CRM-18659: inappropriate place for upgrade script

----------------------------------------
* CRM-18659: inappropriate place for upgrade script
  https://issues.civicrm.org/jira/browse/CRM-18659

Merge pull request #8437 from eileenmcnaughton/dedupe_regression

Dedupe regression

Readded escaping to CRM_Utils_Sort and fixed str_replace in CRM_Contact_BAO_Query->searchQuery()

CRM-18667 Dedupe regression bug (4.7.8rc) merge contacts action from task list fails with an error

Declaring required as false on timestamp fields causes allow null in new installs

Really this is kind of a GenCode bug - but this works...

Merge pull request #8434 from totten/master-regen-zipcodes

(NFC) bin/regen.sh - Drop-create table `zipcodes`

bin/regen.sh - Drop-create table `zipcodes`

The original `regen.sh` script drops the table `zipcodes` after creating it.
That works fine in normal execution (when `regen.sh` finishes without
erorr).  But if there is an error, then the table gets left in the DB -- and
creates a conflict when you try re-run `regen.sh`.

This patch does a 'drop-create' so that subsequent runs will work despite a
previous error.

Set version to 4.7.9

CRM-18655: Selected id's not escaped properly

----------------------------------------
* CRM-18655: Selected id's not escaped properly
  https://issues.civicrm.org/jira/browse/CRM-18655

CRM-18560 - Validate either body text or body html

Merge pull request #7586 from nielosz/configure-recent

[ready-for-core-team-review] Add settings to configure recently viewed items stack (Recent.php)

CRM-18625 - adding other missed fields (button, subject, api)

----------------------------------------
* CRM-18625: Authorize.net not saving testing URL
  https://issues.civicrm.org/jira/browse/CRM-18625

CRM-18625 - use test_url_site and test_url_recur if updating test processor

----------------------------------------
* CRM-18625: Authorize.net not saving testing URL
  https://issues.civicrm.org/jira/browse/CRM-18625