When default value is set to None

CommentFileSizeAuthor
#7 date-2990189-7-set-default-value.patch1.01 KBjacob.embree
#3 date-2990189-3-set-default-value.patch1.5 KBsolotandem
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

theaxe created an issue. See original summary.

theaxe’s picture

theaxe CreditAttribution: theaxe commented
Issue tags: +bug
solotandem’s picture

solotandem CreditAttribution: solotandem commented
Status: Active » Needs review
FileSize
date-2990189-3-set-default-value.patch1.5 KB

An empty date field on an existing entity results in multiple PHP warnings in the form processing code. The attached patch creates an array structure and sets default values for the date value keys. It does not augment the tests (for which this is an obvious use case).

theaxe’s picture

theaxe CreditAttribution: theaxe commented

This fixes it, thank you.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Version: 7.x-2.10 » 7.x-2.x-dev
sonnykt’s picture

sonnykt
Location Melbourne, Australia
CreditAttribution: sonnykt at Salsa Digital for govCMS (Australian Government Department of Finance) commented
Status: Needs review » Reviewed & tested by the community

It works with 7.x-2.10.

jacob.embree’s picture

jacob.embree CreditAttribution: jacob.embree at St. Louis Integration commented
FileSize
date-2990189-7-set-default-value.patch1.01 KB

Reroll for 7.x-2.x.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Reviewed & tested by the community » Fixed
Issue tags: -bug
Parent issue: » #2867810: Plan for Date 7.x-2.11 release

Committed. Thanks.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.