Merge pull request #14273 from eileenmcnaughton/test_dupe

[NFC] array formatting in CRM_Dedupe_MergerTest

Merge pull request #14254 from yashodha/add_dev_tab

add developer tab for summary report

[NFC] minor commits cleanup

[NFC] array formatting in CRM_Dedupe_MergerTest

Rename agpl-3.0.txt to License (#14255)

* Rename agpl-3.0.txt to License

This is the standard name for the license file and will improve this project's community score from GitHub.

* Rename License to LICENSE

Merge pull request #14270 from eileenmcnaughton/dedupe

[ref] Remove reference symbol from 2 variables

Merge pull request #14137 from eileenmcnaughton/pay_up

[REF][Test] Remove call to CRM_Contribute_BAO_Contribution::recordAdditionalPayment in favour of payment create

Remove reference symbol from 2 variables

I checked & these are not altered in the function - so no need

Remove call to CRM_Contribute_BAO_Contribution::recordAdditionalPayment in favour of payment create

Merge pull request #14260 from eileenmcnaughton/spec

[REF+test] clean up code for getting labels for merge screen, stdise

Merge pull request #14264 from colemanw/settingsOptions

[REF] Centralize logic for resolving settings pseudoconstants

Merge pull request #14259 from colemanw/cacheOptions

Remove wasteful double-caching of settings metadata

Merge pull request #14262 from francescbassas/patch-16

dev/core#964 Add start date and end date values on page contribution widget ajax response

Centralize logic for resolving settings pseudoconstants

dev/core#964 Add date start and end values on page contribution widget ajax response

Merge pull request #14263 from elisseck/dev/core/966

dev/core/966 skip urlencode for email in elavon xml

[REF] code cleanup on handling on presentation of merge fields for display

This code was really confusing - all that 'special values' stuff. Turned out it
didn't have to be....

Add unit test for handling of 'special values' when retrieving merge data

This is preliminary to a code cleanup.

m

Merge pull request #14261 from eileenmcnaughton/master

[REF] Remove extraneous variable

dev/core/966 skip urlencode for email in elavon xml

Remove wasteful double-caching of settings metadata

Settings metadata was being cached once per domain and one extra time for good measure.
hook_civicrm_alterSettingsMetaData() was invoked even when fetching cached data,
even though the hook-altered data was stored in the cache.

Now that the hook is invoked only once and the results cached, tests have been tweaked
to cache after setting the hook instead of before.

Merge pull request #14258 from eileenmcnaughton/tidy

[REF] Super Minor code readability fix

Remove extraneous variable

Delete Pair is only called from one place in the code & it does not pass the $entityTable
variable - so remove it & reduce complexity.

In addition the viceVersa variable is always true so we can remove this too.

[REF] Minor code readability fix

Merge pull request #14257 from civicrm/5.14

5.14

Jenkins style fix

Update 5.13.4.md

Update date of 5.13.3 release and add in APIv4 SA into 5.13.4

Update 5.13.4.md

WIP Release Notes for security release

Update Release notes based on Tim's Review and likely drop of 5.13.3

civicrm/file - Be forgiving about old image hyperlinks

Previous versions of Civi sometimes generated URLs for contact-images with incorrect `&mime-type` values:

http://dmaster.bknix:8001/civicrm/file?reset=1&filename=Hello_cca4153cb14beab37c68ab7f07162425.jpg&mime-type=image/jpg

The recent security update will generate an error if the mime-type is incorrect, so this patch relaxes it
to allow the old links to continue working.

Add whitelist back in and validate extension of file is permtted for the mime-type supplied and use mime-type from db if supplied with an fid and eid

Switch to different libary that is php5.6 compatable

Strength mime checking by comparing mime-type to the file path mime-type if we have entity_id and file id otherwise only permit image mime_types to be accepted if going via the filename route

Ensure mimetype is set in the case where we are passing it through and its valid

Remove Whitelisting of mime-types as not useful and only check mime-types if we have had one passed in

Resolve security/core#45 by validating the mimeType of the file with the supplied mime-type

Resolve security/core#52 by validating that an installtype is sensible

security/core#49 Ensure that only intergers are passed to the IN build options in address

Fix Rule checking and add a unit test

Add in unit test on building country_id options too

Add in a unit test for building county options with a state_province_id filter

Also escape subtype to fix POC#2 found by Patrick

escape alphanumeric/checkbox custom data

Fix security/core#51 by paramatising the Event Type part of the wuere clause

Resolve security/core#53 by updating tcpdf to latest version of 6.2.x

(security/core#50) Update jQuery to address prototype pollution

Don't expose condition to api.getoptions

Harden against serialization vulnerabilities (#46)

add developer tab for summary report

Merge pull request #14248 from civicrm/5.14

5.14

Merge pull request #14246 from seamuslee001/5.14-release-notes

5.13.3 Release Notes

contributor-key.yml, 5.13.13.md - Fix typo in credit

I saw a typo and asked Patrick for preferred text.

Update 5.13.3.md

Update 5.13.3.md

Update release notes

Add in Release notes for 5.13.3 Drop

Merge pull request #14239 from colemanw/haveACrack

Remove broken delete from customGroupCreate

Remove broken delete from customGroupCreate

Merge pull request #14181 from eileenmcnaughton/group_search

dev/core#926 [ref] adds a test & does a preliminary extraction

Merge pull request #14195 from mlutfy/dev932

dev/core#932 Fix dedupe contacts flip selection

Merge pull request #14160 from demeritcowboy/fix-change-status-warning

dev/core#896 - fix notice warning on closing a case

Merge pull request #14206 from mattwire/paymentforms_refactor

REF Refactor to reduce duplication on payment forms

Merge pull request #14226 from eileenmcnaughton/master

Update testGetActivityAccessCiviCRMEnough test for clarity

Merge pull request #14238 from civicrm/5.14

5.14

Merge pull request #14118 from eileenmcnaughton/no_fin_item

Fix financial acl permissions to respect check_permissions

Merge pull request #14236 from seamuslee001/activity_test

Fix Failing Activity Test

Fix Failing Activity Test

Merge pull request #14223 from eileenmcnaughton/5.14

dev/core#942 fix failure to render names for some activities

dev/core#942 fix failure to render names for some activities

Overview
----------------------------------------
Set limit for activity_contact retrieval to 0, allowing to retrieve more than 25 activity contacts when rendering the first 25 activities on the activity contact tab

Before
----------------------------------------

After
----------------------------------------

Technical Details
----------------------------------------
This moves the logic for retrieving the target contacts back into the getActivities function. We are stil not wanting to bypass the ACLs so still using the
api but strictly limiting the number of contacts we retrieve (at the cost of extra queries, but cheap ones).

Some tests added on the Bulk Mail activity.

Comments
----------------------------------------

Merge pull request #14233 from seamuslee001/participant_count_search_fix

dev/core#956 Fix hard breakage in find participants form when you do a filter on o…

Merge pull request #14220 from pradpnayak/core/issues/923

core/issues/923, Fixed notice error when creating/editing profile

dev/core#956 Fix hard breakage in find participants form when you do a filter on one event and n participant statues

Merge pull request #14216 from alifrumin/dontmunge

When creating relationship types don't munge names

Merge pull request #14227 from civicrm/5.14

5.14 to master

Merge pull request #14214 from jitendrapurohit/activity-input

Add select2 to input field of datepicker

Merge pull request #14221 from seamuslee001/activty_wrapper_date_picker_template

Add in Wrapper template around DatePickerRange template to have bette…

Update testGetActivityAccessCiviCRMEnough test for clarity

Add in Wrapper template around DatePickerRange template to have better layout of Activity Search Form

Allow for colspan and class to be set when adding in wrapper

Update the wrapper to be only one td and not all and fix activity template appropriately

Merge pull request #14219 from colemanw/hover

dev/core#950 Remove deprecated :hover jQuery selector

Remove deprecated :hover jQuery selector

core/issues/923, Fixed notice error when creating/editing profile

Merge pull request #14217 from prondubuisi/this-in-static-method

dev/core#945 - fix  use of $this error in static method

Merge pull request #14004 from mfb/set-utf8

Remove CIVICRM_TEMP_FORCE_UTF8; deprecate TempTable::setUtf8()

Merge pull request #14185 from mfb/watchdog

Pass the RFC 3164 severity level thru to drupal watchdog.

dev/core#945 - fix  use of $this error in static method

When creating relationship types thru the api Don't munge names

Add select2 to input field of datepicker

Merge pull request #14203 from yashodha/dev_705

(dev/core#705) Disabling Alphabetical Pager is not respected for events

Merge pull request #14183 from eileenmcnaughton/copy_dao

Fix CRM-21832 - Recurring activities don't carry over custom datas & add test provided by Agileware

Merge pull request #14205 from civicrm/5.14

5.14

Merge pull request #14210 from seamuslee001/5_13_2_release_notes

Add 5.13.2 release notes

Update 5.13.2.md

Update Release notes for dev/core#940

Include Dave D for review purposes

Add 5.13.2 release notes

Merge pull request #14208 from seamuslee001/dev_core_940_5_14

(dev/core#940) Fix regression involving Email Processor filing and ".unknown" attachments

Fix Regression in Email Processor filing all emails as .unknown attachments

Merge pull request #14198 from eileenmcnaughton/pay_activity

[REF+ test] start process of cleaning up payment activity handling

Merge pull request #14197 from eileenmcnaughton/cont_cancel

dev/core#927 [ref] Extract contribution.cancel function

Refactor to reduce duplication on payment forms

Merge pull request #14204 from totten/5.14-act-sort

(dev/core#934; followup) Fix escaping on new query code

Fix failure to copy custom fields & add test provided by Agileware

Per https://github.com/civicrm/civicrm-core/pull/13470 custom fields are
inconsistently copied where copying entities. This makes the code from
BAO_Event called from the genericCopy function.

I did a bit of an audit and the places where this is currently called from don't appear
to call the copyGeneric function with the 'custom' param that would have activated the old
code. I also consistently removed the & when it was being called so I could take it
out of the signature.

The original PR handled tags as well, but not in a generic way. I've left that out of scope
but the test is present, commented out, so it would be easy enough to revist

(dev/core#934; followup) Fix escaping on new query code

This updates a line which was added in the past day (#14194) to ensure that
the data is escaped.

(dev/core#705) Disabling Alphabetical Pager is not respected for events

Merge pull request #14199 from colemanw/relActive

Stop overriding is_active default when saving new relationship

Stop overriding is_active default when saving new relationship

For some obscure reason the relationsip BAO was forcing the is_active field
to default to 0 even though the schema sets a more sensible default of 1.