Merge pull request #12660 from pradpnayak/report-error

Fixed fatal error 'DB Error: no such field' on Contribution detail re…

Merge pull request #12691 from andrewpthompson/dev-core-343

dev/core#343 Fix long address lines overflowing label

Merge pull request #12509 from JMAConsulting/dev-mail-20

dev/mail#20 : Preview screen doesn't open until recipients list is built on mail compose screen

Merge pull request #12684 from mattwire/money_fn_update_payment_details

dev/core#299 Fix mishandling of non decimal currency on update payment details form

Merge pull request #12738 from mattwire/membershiptype_customdata_formsupport

Enable custom data display on MembershipType form

Merge pull request #12735 from colemanw/dev/core#351

dev/core#351 - Fix $.crmSnippet error with no url args

Merge pull request #12712 from seamuslee001/dev_core_352

dev/core#352 Ensure that contacts that are to be exluded are not adde…

dev/core#351 - Fix $.crmSnippet error with no url args

Enable custom data display on MembershipType form

Merge pull request #12701 from magnolia61/Scheduled_reminders-default-sort

dev/core#349 correct default sort for scheduled reminders list

Format money correctly for locale on UpdatePaymentDetails form

Merge pull request #12667 from mlutfy/translation-14

dev/translation#14 Fix InnoDB Advanced Logging

Merge pull request #12704 from colemanw/recent

Fix recent item deduping & call hooks

dev/core#352 Ensure that contacts that are to be exluded are not added incorrectly when adding recipients of past mailings

Add in DISTINCT clause as per Moonish's comment

Merge pull request #12662 from eileenmcnaughton/net_amount

Remove net_amount from the Back office contribution form.

Merge pull request #12678 from kewljuice/patch-1

Fix Undefined index: params in CRM_Event_BAO_Event::buildCustomProfile()

Merge pull request #12700 from vinuvarshith/BASW-240-membership-network-error-fix

BASW-240: 'Network Error' when creating membership type from contact membership tab

Merge pull request #12599 from lcdservices/dev-mail-21

dev/mail#21 trim test emails

Merge pull request #12679 from mlutfy/core-334

dev/core#334 Fix Contribution Page with Checksum and multiple Payment Processors

Merge pull request #12699 from chriscant/event-tell-a-friend-title

Get title for event tell a friend mail

Merge pull request #12703 from yashodha/CRM-21677

CRM-21677 - report clean up

Merge pull request #12716 from calbasi/patch-1

Wrap long labels onto multiple lines to prevent text being cut off

Merge pull request #12613 from JMAConsulting/core-issue-297

dev/core#297 : fix broken permission 'access my cases and activities'

Wrap long labels onto multiple lines to prevent text being cut off

Merge pull request #12726 from kainuk/issue-361

Corrected tag_id parameter description to refer cases

Merge pull request #12727 from pradpnayak/Pseudoconstant

Added Pseudoconstant for phone and website type

Merge pull request #12725 from civicrm/5.5

5.5

Added Pseudoconstant for phone and website type

Corrected tag_id parameter description to refer cases instead of activities (See https://lab.civicrm.org/dev/core/issues/361)

Merge pull request #12720 from selwyntcy/patch-1

(NFC) Fix incorrect reference of doTransferCheckout() in comments

Merge pull request #12718 from mattwire/custom_field_optiongroup_fix

Fix for issue editing custom fields with option groups after #12423

Merge pull request #12719 from seamuslee001/update_sql_generated

Update civicrm_generated mysql file following changes to custom mysql…

Merge pull request #12722 from eileenmcnaughton/report_instance

Fix ReportInstance BAO to call post hook after save (instead of calling pre twice).

Merge pull request #12708 from colemanw/summary

Contact summary - adjust block width when editing

Fix ReportInstance BAO to call post hook after save (instead of calling pre twice).

Also, remove one unused var

Fix inconsistent reference to function name

Fix inconsistent reference to function name

Fix inconsistent reference to function name

Update civicrm_generated mysql file following changes to custom mysql file

Fix for issue editing custom fields with option groups after #12423

Contact summary - adjust block width when editing

Merge pull request #12715 from mattwire/comments_2

Minor comment and typo cleanup

Comments

Fixed whitespace

Added comment explaining paddingLeft

Merge pull request #12705 from pradpnayak/Visibility

Added Pseudoconstant tag for visibility

Added Pseudoconstant tag for visibility

Call hooks when deleting recent items

Fix recent item deduping

Merge pull request #12702 from colemanw/urls

Navigation - Don't reprocess fully-formed urls

CRM-21677 - report clean up

Merge pull request #12692 from andrewpthompson/dev-joomla-2

dev/joomla#2 Joomla on Windows cli.php & cron.php fail due to incorrect path

Navigation - Don't reprocess fully-formed urls

Merge pull request #12696 from eileenmcnaughton/matt_1

NFC cleanup to Membership/Renewal form classes.

Merge pull request #12698 from yashodha/CRM-21677

CRM-21677 - report clean up

correct default sort for scheduled reminders list

BASW-240: Change Null Strings To Null Values As Null Strings Are Not Actually Null

Get title for event tell a friend mail

CRM-21677 - report clean up

NFC cleanup to Membership/Renewal form classes.

Use DIRECTORY_SEPARATOR in getBasePath()

Hard-coded forward slash resulted in incorrect path on Windows

Merge pull request #12628 from colemanw/Use

Use static cache for UFMatch lookups to improve performance

Fix long address lines overflowing label

generateLabel() - Width did not take into account paddingLeft

Merge pull request #12689 from eileenmcnaughton/money_fn

Add test for new rounded money format functions

Add test for new rounded money format functions

Note that there are also some protected functions added. I have left those to be teste via this wrapper as they are not yet exposed

Merge pull request #12626 from eileenmcnaughton/money_fn

dev/core/#119 Fix mishandling of non decimal currency on additional payment form.

Merge pull request #12686 from lcdservices/dev-core-341

dev/core#341 sybunt custom search date field fix

dev/core#341 sybunt custom search date field fix

Merge pull request #12652 from davejenx/core-322

dev/core/issues/322 - fix JS error on contribution page, completing form…

Merge pull request #12642 from eileenmcnaughton/recur

Code cleanup on Membership form when freezing fields in a recurring exists situation

Merge pull request #12685 from MegaphoneJon/fix-pledge-report-group-filter

Replicate fix on PR #12643 for pledge report

Replicate fix on PR #12643 for pledge report

Merge pull request #12664 from totten/master-prevnext-ifclean

(dev/core#217) PrevNext - Remove references to entity_table and entity_id2 from service. Add test.

Merge pull request #12683 from civicrm/5.5

5.5

Merge pull request #12671 from eileenmcnaughton/ex55

Fix + Add test for exporting location types with changed names

(dev/core#217) Improve test coverage of PrevNext::deleteItem

(dev/core#217) Add E2E_Core_PrevNextTest

(dev/core#217) PrevNext - Remove references to entity_table and entity_id2 from service

The PrevNext service/interface aims to be a replaceable component for use
the search-caching (but not deduping).

The interface that we produced from refactoring includes several references
to `entity_table` and `entity_id2` -- these values are part of the SQL
table, and they're needed for dedupe, but they don't seem to convey anything
meaningful for search-caching.  Including these fields makes the interface
more complicated -- which will make it hard to implement other variants.

The general gist of this commit is that we no longer fill those two columns,
and we no longer read them.

In a couple functions, we split the new OOP implementation
(`CRM_Core_PrevNextCache_Sql`) from the traditional static BAO
implementation (`CRM_Core_BAO_PrevNext`) so that we can omit these fields.

(dev/core#217) PrevNext - Make `entity_id2` optional (in SQL)

Merge pull request #12680 from mattwire/comments

NFC Add/update two comments

Merge pull request #12424 from alifrumin/editOwnEvents

 Ensure users with the perm "CiviEvent: access CiviEvent " can edit events they have created.

NFC comments

dev/core#334 Fix Contribution Page with Checksum and multiple Payment Processors

Event.php:1830, ControlSignature, Priority: High

Expected "if (...) {\n"; found "if(...) {\n"

Merge pull request #12663 from totten/master-prevnext-consistent

(dev/core#217) PrevNext - Use more consistent cache-keys while adjusting filters

Fix Undefined index: params in CRM_Event_BAO_Event::buildCustomProfile()

Added check to suppress notice: Undefined index: params in CRM_Event_BAO_Event::buildCustomProfile() which comes after returning to site from an online payment provider.

Merge pull request #12677 from civicrm/5.5

5.5 to master

Merge pull request #12674 from totten/5.5-upgr-tpl-msg

Upgrader - When updating message templates, identify them by name

Merge pull request #12670 from totten/5.5-perm-msg

Refine upgrade message for `save Report Criteria`

Fix + Add test for exporting location types with changed names

Merge pull request #12676 from civicrm/5.5

5.5 to master

Merge pull request #12673 from eileenmcnaughton/atest

Test fix for intermittant error

Upgrader - When updating message templates, identify them by name

Also: Fix a layout issue caused by a typo in the `<li>` tag.

Loosedn activityTest on country order.

The order is no considered significant, be open minded

(NFC) Upgrade - Improve example of an upgrade message for a new permission

@bgm pointed out that the permission name should be translated as well.
This left me wondering...  why do we bother with parameterizing the string
with a `%1` if it's all going to be translated anyway?

There is a plausible reason.  It's not super-critical, but it's plausible --
the permission name is used in more contexts (like the permission mgmt
screen in D7/WP *as well as* the upgrade screen).  This arrangement ensures
that the name appears the same in *all those contexts*, even in various
edge-cases of the translation lifecycle (e.g.  where the permission-names
have been translated but the upgrade-messages have not).  Again...  it's not
a big deal, but it is a slightly better default.

Upgrade messages - When going through 5.4.*, the permission name should be legible

Upgrade messages - When going through 5.4.*, only display permission message once

Merge pull request #12666 from jmcclelland/issue328

don't add query to developer tab if we are downloading the data

dev/translation#14 Fix InnoDB Advanced Logging.

don't add query to dev tab for each row

Adding the query to the dev tab for each row consumes a lot of resources
for the informational gain we get.