I think it's good to add 0/1 option for the default formats for a boolean field so you can use hide if empty because "Count the number 0 as empty" makes sense.

CommentFileSizeAuthor
#5 views-0_1_format_boolean_field-1791372-5.patch674 bytesyannickoo
#2 views-0_1_format_boolean_field-1791372-2.patch670 bytesyannickoo
#1 views-0_1_format_boolean_field-1791372-1.patch674 bytesyannickoo
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

yannickoo’s picture

yannickoo
Location Berlin
CreditAttribution: yannickoo commented
Status: Active » Needs review
FileSize
views-0_1_format_boolean_field-1791372-1.patch674 bytes
yannickoo’s picture

yannickoo
Location Berlin
CreditAttribution: yannickoo commented
FileSize
views-0_1_format_boolean_field-1791372-2.patch670 bytes

Ups, wrong order and we have to write them as integer and not as string.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Needs review » Reviewed & tested by the community

Cool!

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Version: 7.x-3.x-dev » 8.x-3.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Committed to 7.3-3.x, do you want to create a patch for 8.x-3.x or should i just do that?

yannickoo’s picture

yannickoo
Location Berlin
CreditAttribution: yannickoo commented
Status: Patch (to be ported) » Needs review
FileSize
views-0_1_format_boolean_field-1791372-5.patch674 bytes

Here is the patch for 8.x-3.x

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Needs review » Fixed

Perfect. Committed and pushed.

Status: Fixed » Closed (fixed)

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