civicrm-core.git
4 years agoMerge pull request #18095 from demeritcowboy/no-db
Eileen McNaughton [Tue, 11 Aug 2020 08:03:17 +0000 (20:03 +1200)]
Merge pull request #18095 from demeritcowboy/no-db

dev/core#1926 - Towards supporting SSL for mysql connections - remove DB::connect that doesn't add anything

4 years agoMerge pull request #18122 from seamuslee001/master
Seamus Lee [Tue, 11 Aug 2020 00:18:06 +0000 (10:18 +1000)]
Merge pull request #18122 from seamuslee001/master

5.29

4 years agoMerge in 5.29
Seamus Lee [Tue, 11 Aug 2020 00:16:31 +0000 (10:16 +1000)]
Merge in 5.29

4 years agoMerge pull request #18116 from mattwire/phpnoticecontributionpage
Eileen McNaughton [Tue, 11 Aug 2020 00:09:35 +0000 (12:09 +1200)]
Merge pull request #18116 from mattwire/phpnoticecontributionpage

Fix PHP notice on contribution page

4 years agoMerge pull request #18119 from demeritcowboy/document-hidden-buttons
Seamus Lee [Mon, 10 Aug 2020 22:08:17 +0000 (08:08 +1000)]
Merge pull request #18119 from demeritcowboy/document-hidden-buttons

[NFC] Add code comments about hidden but still used buttons

4 years agoMerge pull request #18115 from sunilpawar/dev_1943
Seamus Lee [Mon, 10 Aug 2020 20:55:39 +0000 (06:55 +1000)]
Merge pull request #18115 from sunilpawar/dev_1943

dev/core#1943 add functionality to add civicrm log into Drupal access log

4 years agocomments about hidden but still used buttons
demeritcowboy [Sun, 9 Aug 2020 05:07:57 +0000 (01:07 -0400)]
comments about hidden but still used buttons

4 years agoMerge pull request #18117 from mattwire/529_master
Eileen McNaughton [Mon, 10 Aug 2020 19:53:31 +0000 (07:53 +1200)]
Merge pull request #18117 from mattwire/529_master

5.29 to master

4 years agoMore radical cleanup of button CSS
Andrew Hunt [Thu, 6 Aug 2020 22:17:31 +0000 (18:17 -0400)]
More radical cleanup of button CSS

4 years agoFix CSS for new buttons
Andrew Hunt [Thu, 6 Aug 2020 22:00:58 +0000 (18:00 -0400)]
Fix CSS for new buttons

4 years agoFix JS for input buttons that are now button buttons
Andrew Hunt [Thu, 6 Aug 2020 20:06:56 +0000 (16:06 -0400)]
Fix JS for input buttons that are now button buttons

4 years agoRemove deprecated submitOnce js function
Andrew Hunt [Thu, 6 Aug 2020 19:36:47 +0000 (15:36 -0400)]
Remove deprecated submitOnce js function

4 years agoNo need for crm-button wrappers for real buttons
Andrew Hunt [Thu, 6 Aug 2020 18:57:11 +0000 (14:57 -0400)]
No need for crm-button wrappers for real buttons

4 years agoGive buttons a value when the submitted value is used to identify them
Andrew Hunt [Thu, 6 Aug 2020 17:55:52 +0000 (13:55 -0400)]
Give buttons a value when the submitted value is used to identify them

4 years agoPut icons inside of button elements
Andrew Hunt [Thu, 6 Aug 2020 17:12:36 +0000 (13:12 -0400)]
Put icons inside of button elements

4 years agoSwap out button/submit inputs for button elements
Andrew Hunt [Thu, 6 Aug 2020 15:19:56 +0000 (11:19 -0400)]
Swap out button/submit inputs for button elements

4 years agoCorrection to change priority level
Sunil Pawar [Mon, 10 Aug 2020 13:30:45 +0000 (19:00 +0530)]
Correction to change priority level

4 years agoMerge pull request #18091 from agh1/xbutton-529
Matthew Wire [Mon, 10 Aug 2020 12:42:37 +0000 (13:42 +0100)]
Merge pull request #18091 from agh1/xbutton-529

Swap out button/submit inputs for button elements

4 years agoFix PHP notice on contribution page
Matthew Wire [Mon, 10 Aug 2020 12:31:07 +0000 (13:31 +0100)]
Fix PHP notice on contribution page

4 years agosmall correction
Sunil Pawar [Mon, 10 Aug 2020 12:12:20 +0000 (17:42 +0530)]
small correction

4 years agodev/core#1943 add functionality to add civicrm log into Drupal access log
Sunil Pawar [Mon, 10 Aug 2020 12:09:22 +0000 (17:39 +0530)]
dev/core#1943 add functionality to add civicrm log into Drupal access log

4 years agoMerge pull request #18112 from eileenmcnaughton/paytest
Eileen McNaughton [Mon, 10 Aug 2020 10:06:32 +0000 (22:06 +1200)]
Merge pull request #18112 from eileenmcnaughton/paytest

[NFC] [Test] Initial testing on event payment forms.

4 years ago[Ref] [Test] Update Authorize.net test to use the trait
eileen [Mon, 10 Aug 2020 05:38:30 +0000 (17:38 +1200)]
[Ref] [Test] Update Authorize.net test to use the trait

4 years ago[NFC] [Test] Intial testing on event payment forms.
eileen [Mon, 10 Aug 2020 04:12:02 +0000 (16:12 +1200)]
[NFC] [Test] Intial testing on event payment forms.

I'm trying to work on improving our testing of payment forms. I started with 3 event forms but there were
enough challenges that for this commit I'm just adding tests on one form. As noted in the code comments
it makes sense for this test to be in the extension but I want to work through the challenges on the
other forms before finalising any helper functions to the point where they are available to extensions.

This is actually a bit of a break through as it's the first time we have testing on a form flow - ie
submitting the first form and then the second. It was quite painful

4 years agoMerge pull request #18110 from eileenmcnaughton/xml
Seamus Lee [Mon, 10 Aug 2020 00:55:34 +0000 (10:55 +1000)]
Merge pull request #18110 from eileenmcnaughton/xml

Add Serialize key to payment_processor field for Event & Contribution Page

4 years agoMerge pull request #18109 from eileenmcnaughton/format
Seamus Lee [Sun, 9 Aug 2020 23:33:30 +0000 (09:33 +1000)]
Merge pull request #18109 from eileenmcnaughton/format

[NFC] Array formatting

4 years agoMerge pull request #18108 from eileenmcnaughton/paytest
Seamus Lee [Sun, 9 Aug 2020 23:33:13 +0000 (09:33 +1000)]
Merge pull request #18108 from eileenmcnaughton/paytest

Remove unused variable

4 years agoAdd Serialize key to payment_processor field for Event & ContributionPage
eileen [Sun, 9 Aug 2020 23:03:05 +0000 (11:03 +1200)]
Add Serialize key to payment_processor field for Event & ContributionPage

The DAO can handle arrays if it knows the format now...

4 years agoRemove unused variable
eileen [Sun, 9 Aug 2020 21:44:02 +0000 (09:44 +1200)]
Remove unused variable

I spotted this variable as defined but unused - let's remove

4 years ago[NFC] Array formatting
eileen [Sun, 9 Aug 2020 21:55:11 +0000 (09:55 +1200)]
[NFC] Array formatting

This is a straight up IDE reformat to get rid of old array syntax.

IDE had some whitespace thoughts with I'm neutral on - if Jenkins is too then all good IMHO

4 years agoMerge pull request #18104 from civicrm/5.29
Seamus Lee [Sat, 8 Aug 2020 07:04:10 +0000 (17:04 +1000)]
Merge pull request #18104 from civicrm/5.29

5.29

4 years agoMerge pull request #18101 from demeritcowboy/eventcart-settingcheck
Seamus Lee [Sat, 8 Aug 2020 04:27:00 +0000 (14:27 +1000)]
Merge pull request #18101 from demeritcowboy/eventcart-settingcheck

dev/event#40 - EventCart - Check legacy setting until extension is public

4 years agoMerge pull request #18100 from civicrm/5.29
Seamus Lee [Sat, 8 Aug 2020 01:35:44 +0000 (11:35 +1000)]
Merge pull request #18100 from civicrm/5.29

5.29

4 years agocheck legacy setting until extension is public
demeritcowboy [Sat, 8 Aug 2020 00:59:20 +0000 (20:59 -0400)]
check legacy setting until extension is public

4 years agoMerge pull request #18068 from seamuslee001/dev_wordpress_66
Seamus Lee [Sat, 8 Aug 2020 00:09:19 +0000 (10:09 +1000)]
Merge pull request #18068 from seamuslee001/dev_wordpress_66

dev/wordpress#66 Re-instate newer variables but with more support for…

4 years agoMerge pull request #18097 from demeritcowboy/mysqli
Seamus Lee [Sat, 8 Aug 2020 00:07:15 +0000 (10:07 +1000)]
Merge pull request #18097 from demeritcowboy/mysqli

There is no "I" in mysqli - oh wait there is

4 years agoMerge pull request #18098 from civicrm/5.29
Eileen McNaughton [Fri, 7 Aug 2020 22:59:56 +0000 (10:59 +1200)]
Merge pull request #18098 from civicrm/5.29

5.29 to master

4 years agoMerge pull request #18092 from seamuslee001/preimum_onbehalfof_fix
Eileen McNaughton [Fri, 7 Aug 2020 22:59:19 +0000 (10:59 +1200)]
Merge pull request #18092 from seamuslee001/preimum_onbehalfof_fix

[REF] Fix jquery validation for on behalf of fields when combined wit…

4 years agoMerge pull request #17919 from seamuslee001/nfc_locale_documentation
Eileen McNaughton [Fri, 7 Aug 2020 22:26:23 +0000 (10:26 +1200)]
Merge pull request #17919 from seamuslee001/nfc_locale_documentation

[NFC] Update locale over-ride documentation to mention that it is mos…

4 years agomysqli
demeritcowboy [Fri, 7 Aug 2020 22:02:54 +0000 (18:02 -0400)]
mysqli

4 years agounneeded connect call
demeritcowboy [Fri, 7 Aug 2020 16:05:13 +0000 (12:05 -0400)]
unneeded connect call

4 years agoMerge pull request #18093 from colemanw/expFix
Seamus Lee [Fri, 7 Aug 2020 04:18:17 +0000 (14:18 +1000)]
Merge pull request #18093 from colemanw/expFix

APIv4 Explorer - Fix possible undefined index

4 years agoMerge pull request #17934 from eileenmcnaughton/conttype
colemanw [Fri, 7 Aug 2020 02:59:08 +0000 (22:59 -0400)]
Merge pull request #17934 from eileenmcnaughton/conttype

[REF] Update subtypeInfo function to leverage getAllContactTypes

4 years agoAPIv4 Explorer - Fix possible undefined index
Coleman Watts [Thu, 6 Aug 2020 19:11:10 +0000 (15:11 -0400)]
APIv4 Explorer - Fix possible undefined index

4 years agoMerge pull request #18047 from eileenmcnaughton/ent_ref
Seamus Lee [Fri, 7 Aug 2020 02:18:16 +0000 (12:18 +1000)]
Merge pull request #18047 from eileenmcnaughton/ent_ref

[REF] Fix metadata label for contribution_page_id

4 years ago[REF] Fix jquery validation for on behalf of fields when combined with a preimum
Seamus Lee [Fri, 7 Aug 2020 02:09:55 +0000 (12:09 +1000)]
[REF] Fix jquery validation for on behalf of fields when combined with a preimum

4 years agoMerge pull request #18090 from civicrm/5.29
Seamus Lee [Fri, 7 Aug 2020 00:20:38 +0000 (10:20 +1000)]
Merge pull request #18090 from civicrm/5.29

5.29

4 years agoMerge pull request #17950 from lcdservices/dev-core-1895
Seamus Lee [Thu, 6 Aug 2020 22:53:24 +0000 (08:53 +1000)]
Merge pull request #17950 from lcdservices/dev-core-1895

dev/core#1895 fix first/last name adv search

4 years agoMore radical cleanup of button CSS
Andrew Hunt [Thu, 6 Aug 2020 22:17:31 +0000 (18:17 -0400)]
More radical cleanup of button CSS

4 years agoFix CSS for new buttons
Andrew Hunt [Thu, 6 Aug 2020 22:00:58 +0000 (18:00 -0400)]
Fix CSS for new buttons

4 years agoFix JS for input buttons that are now button buttons
Andrew Hunt [Thu, 6 Aug 2020 20:06:56 +0000 (16:06 -0400)]
Fix JS for input buttons that are now button buttons

4 years agoRemove deprecated submitOnce js function
Andrew Hunt [Thu, 6 Aug 2020 19:36:47 +0000 (15:36 -0400)]
Remove deprecated submitOnce js function

4 years agoNo need for crm-button wrappers for real buttons
Andrew Hunt [Thu, 6 Aug 2020 18:57:11 +0000 (14:57 -0400)]
No need for crm-button wrappers for real buttons

4 years agoGive buttons a value when the submitted value is used to identify them
Andrew Hunt [Thu, 6 Aug 2020 17:55:52 +0000 (13:55 -0400)]
Give buttons a value when the submitted value is used to identify them

4 years agoPut icons inside of button elements
Andrew Hunt [Thu, 6 Aug 2020 17:12:36 +0000 (13:12 -0400)]
Put icons inside of button elements

4 years agoSwap out button/submit inputs for button elements
Andrew Hunt [Thu, 6 Aug 2020 15:19:56 +0000 (11:19 -0400)]
Swap out button/submit inputs for button elements

4 years agoMerge pull request #18088 from agh1/forcebackend
Seamus Lee [Thu, 6 Aug 2020 22:35:44 +0000 (08:35 +1000)]
Merge pull request #18088 from agh1/forcebackend

dev/core#1905 force backend links for new "configure" buttons

4 years agoMerge pull request #18089 from colemanw/api4cv
Eileen McNaughton [Thu, 6 Aug 2020 22:33:45 +0000 (10:33 +1200)]
Merge pull request #18089 from colemanw/api4cv

APIv4 Explorer: Generate short cv code

4 years agoMerge pull request #18085 from totten/5.29-periodic
Seamus Lee [Thu, 6 Aug 2020 21:27:55 +0000 (07:27 +1000)]
Merge pull request #18085 from totten/5.29-periodic

dev/core#1932 - Make status-checks more polite during upgrade

4 years agodev/wordpress#66 Re-instate newer variables but with more support for legacy file...
Seamus Lee [Tue, 4 Aug 2020 20:59:25 +0000 (06:59 +1000)]
dev/wordpress#66 Re-instate newer variables but with more support for legacy file systems

4 years agoAPIv4 Explorer: Generate short cv code
Coleman Watts [Thu, 6 Aug 2020 18:36:10 +0000 (14:36 -0400)]
APIv4 Explorer: Generate short cv code

4 years agodev/core#1905 force backend links for new "configure" buttons
Andrew Hunt [Thu, 6 Aug 2020 18:31:40 +0000 (14:31 -0400)]
dev/core#1905 force backend links for new "configure" buttons

4 years agodev/core#1932 - Make status-checks more polite during upgrade
Tim Otten [Thu, 6 Aug 2020 09:04:36 +0000 (02:04 -0700)]
dev/core#1932 - Make status-checks more polite during upgrade

Before
------

If you happen to run a status-check (eg `showPeriodicAlerts()`) in the interim between
downloading code and running DB upgrades, then you'll get weird failures.

After
-----

The status-checks are able to complete.

Technical Details
-----------------

(1) I figure that this conditional is lightweight because it relies on data
that's read-once and then cached in memory:

```
CRM_Utils_System::version() !== CRM_Core_BAO_Domain::version()
```

(2) The easiest way to reproduce is to get a DB from before 4.7 and coerce
`CRM_Utils_Check::CHECK_TIMER`.

4 years agoMerge pull request #18027 from seamuslee001/pear_db_composer
Seamus Lee [Thu, 6 Aug 2020 06:43:13 +0000 (16:43 +1000)]
Merge pull request #18027 from seamuslee001/pear_db_composer

[REF] Deploy Pear DB package using composer

4 years agoSet version to 5.30.alpha1
CiviCRM [Thu, 6 Aug 2020 06:40:29 +0000 (06:40 +0000)]
Set version to 5.30.alpha1

4 years agoSet version to 5.29.beta1
CiviCRM [Thu, 6 Aug 2020 06:23:07 +0000 (06:23 +0000)]
Set version to 5.29.beta1

4 years agoMerge pull request #18080 from seamuslee001/dev_core_1928
Seamus Lee [Thu, 6 Aug 2020 03:54:01 +0000 (13:54 +1000)]
Merge pull request #18080 from seamuslee001/dev_core_1928

dev/core#1928 Fix HTML5 error due to required attribute being set swi…

4 years agoMerge pull request #18084 from civicrm/5.28
Seamus Lee [Thu, 6 Aug 2020 03:25:30 +0000 (13:25 +1000)]
Merge pull request #18084 from civicrm/5.28

5.28

4 years agoMerge pull request #18083 from totten/5.28-rn
Tim Otten [Thu, 6 Aug 2020 03:23:56 +0000 (20:23 -0700)]
Merge pull request #18083 from totten/5.28-rn

(NFC) 5.28.0.md - Describe last minute PR. Random copyedits.

4 years agoMerge pull request #18082 from seamuslee001/master
Seamus Lee [Thu, 6 Aug 2020 02:53:29 +0000 (12:53 +1000)]
Merge pull request #18082 from seamuslee001/master

5.28

4 years ago5.28.0.md - Describe last minute PR. Random copyedits.
Tim Otten [Thu, 6 Aug 2020 02:47:27 +0000 (19:47 -0700)]
5.28.0.md - Describe last minute PR. Random copyedits.

4 years agoMerge in 5.28
Seamus Lee [Thu, 6 Aug 2020 02:38:20 +0000 (12:38 +1000)]
Merge in 5.28

4 years agoMerge pull request #18079 from eileenmcnaughton/528
Seamus Lee [Thu, 6 Aug 2020 02:35:59 +0000 (12:35 +1000)]
Merge pull request #18079 from eileenmcnaughton/528

dev/core#1930 fix for move-related checkbox being overridden to true …

4 years agodev/core#1928 Fix HTML5 error due to required attribute being set switch to using...
Seamus Lee [Thu, 6 Aug 2020 01:18:33 +0000 (11:18 +1000)]
dev/core#1928 Fix HTML5 error due to required attribute being set switch to using a class as jquery.validation picks up the class as well

4 years agodev/core#1930 fix for move-related checkbox being overridden to true in form
eileen [Thu, 6 Aug 2020 00:50:43 +0000 (12:50 +1200)]
dev/core#1930 fix for move-related checkbox being overridden to true in form

4 years agoMerge pull request #18078 from eileenmcnaughton/directp
Seamus Lee [Wed, 5 Aug 2020 23:16:02 +0000 (09:16 +1000)]
Merge pull request #18078 from eileenmcnaughton/directp

dev/financial#135 Remove stub function from payflowPro

4 years agoMerge pull request #18077 from civicrm/5.28
Seamus Lee [Wed, 5 Aug 2020 22:14:57 +0000 (08:14 +1000)]
Merge pull request #18077 from civicrm/5.28

5.28

4 years agoMerge pull request #18000 from eileenmcnaughton/brn
Seamus Lee [Wed, 5 Aug 2020 21:20:04 +0000 (07:20 +1000)]
Merge pull request #18000 from eileenmcnaughton/brn

Fix button name on updated form

4 years agoRemove stub function
eileen [Wed, 5 Aug 2020 21:05:56 +0000 (09:05 +1200)]
Remove stub function

We used to do this, we don't now

4 years agoMerge pull request #18074 from agh1/5.28.0-releasenotes-final
Mathieu Lu [Wed, 5 Aug 2020 20:33:23 +0000 (16:33 -0400)]
Merge pull request #18074 from agh1/5.28.0-releasenotes-final

5.28.0 release notes final edits

4 years ago5.28.0 release notes: added late changes
Andrew Hunt [Wed, 5 Aug 2020 18:50:10 +0000 (14:50 -0400)]
5.28.0 release notes: added late changes

4 years ago5.28.0 release notes: misc edits
Andrew Hunt [Wed, 5 Aug 2020 16:04:32 +0000 (12:04 -0400)]
5.28.0 release notes: misc edits

4 years agoMerge pull request #18073 from seamuslee001/provider_test_php74
Eileen McNaughton [Wed, 5 Aug 2020 12:20:24 +0000 (00:20 +1200)]
Merge pull request #18073 from seamuslee001/provider_test_php74

[NFC] Fix provider unit test on PHP7.4

4 years agoMerge pull request #17981 from eileenmcnaughton/merge_form
Eileen McNaughton [Wed, 5 Aug 2020 12:18:52 +0000 (00:18 +1200)]
Merge pull request #17981 from eileenmcnaughton/merge_form

[REF] Move handling of form elements back to the Form

4 years agoMerge pull request #18071 from eileenmcnaughton/anet
Matthew Wire [Wed, 5 Aug 2020 10:26:48 +0000 (11:26 +0100)]
Merge pull request #18071 from eileenmcnaughton/anet

Do not pass-by-reference to recur function

4 years agoMerge pull request #18072 from eileenmcnaughton/manual
Matthew Wire [Wed, 5 Aug 2020 10:26:16 +0000 (11:26 +0100)]
Merge pull request #18072 from eileenmcnaughton/manual

dev/financial#135 Remove unreachable doDirectPayment from manual processor

4 years ago[NFC] Fix provider unit test on PHP7.4
Seamus Lee [Wed, 5 Aug 2020 10:14:03 +0000 (20:14 +1000)]
[NFC] Fix provider unit test on PHP7.4

4 years agoMerge pull request #18049 from christianwach/lab-core-1891
Matthew Wire [Wed, 5 Aug 2020 09:57:03 +0000 (10:57 +0100)]
Merge pull request #18049 from christianwach/lab-core-1891

Refactor "applyLocale" and remove references to "language" column in UFMatch table

4 years agodev/financial#135 Remove unreachable doDirectPayment from manual processor
eileen [Wed, 5 Aug 2020 06:35:45 +0000 (18:35 +1200)]
dev/financial#135 Remove unreachable doDirectPayment from manual processor

doDirectPayment is only ever called (deprecated) when doPayment is not overriden (for quite some time now). This is
unreachable & can be removed

4 years agoDo not pass-by-reference to recur function
eileen [Wed, 5 Aug 2020 06:25:53 +0000 (18:25 +1200)]
Do not pass-by-reference to recur function

This is called once, from the main function and the values are not used again

4 years agoMerge pull request #18065 from colemanw/cronAlert
Eileen McNaughton [Wed, 5 Aug 2020 03:51:33 +0000 (15:51 +1200)]
Merge pull request #18065 from colemanw/cronAlert

Show cron warning on Scheduled Jobs admin page

4 years agoMerge pull request #18069 from civicrm/5.28
Eileen McNaughton [Wed, 5 Aug 2020 02:32:59 +0000 (14:32 +1200)]
Merge pull request #18069 from civicrm/5.28

5.28

4 years agoAPIv4 - Fix potential PHP undefined index warning
Coleman Watts [Wed, 5 Aug 2020 01:22:43 +0000 (21:22 -0400)]
APIv4 - Fix potential PHP undefined index warning

Ensures the array key '0' exists in the where clause before accessing it.

4 years agoMerge pull request #18070 from seamuslee001/dev_core_1927
Seamus Lee [Wed, 5 Aug 2020 00:48:44 +0000 (10:48 +1000)]
Merge pull request #18070 from seamuslee001/dev_core_1927

dev/core#1927 Ensure that the contents of the database table are fixe…

4 years agoMerge pull request #18056 from pradpnayak/invopdf
Eileen McNaughton [Tue, 4 Aug 2020 23:38:57 +0000 (11:38 +1200)]
Merge pull request #18056 from pradpnayak/invopdf

Use correct pdf package to generate pdf file on invoice download/email activity

4 years agodev/core#1927 Ensure that the contents of the database table are fixed up before...
Seamus Lee [Tue, 4 Aug 2020 23:19:17 +0000 (09:19 +1000)]
dev/core#1927 Ensure that the contents of the database table are fixed up before changing the column type

4 years agoMerge pull request #18066 from seamuslee001/dev_drupal_131
Seamus Lee [Tue, 4 Aug 2020 22:34:17 +0000 (08:34 +1000)]
Merge pull request #18066 from seamuslee001/dev_drupal_131

dev/drupal#131 Ensure that the General class exists

4 years agodev/drupal#131 Ensure that the General class exists
Seamus Lee [Tue, 4 Aug 2020 20:48:47 +0000 (06:48 +1000)]
dev/drupal#131 Ensure that the General class exists

4 years agoMerge pull request #18005 from agh1/button-icon-placement
Seamus Lee [Tue, 4 Aug 2020 20:24:06 +0000 (06:24 +1000)]
Merge pull request #18005 from agh1/button-icon-placement

Fix buggy placement of icons on buttons

4 years agoMerge pull request #18058 from eileenmcnaughton/date
Seamus Lee [Tue, 4 Aug 2020 20:22:55 +0000 (06:22 +1000)]
Merge pull request #18058 from eileenmcnaughton/date

[REF] Even less variable variables