Customizable "true"/"false" Views output for booleans

Thanks for the patch! We should add test coverage for this.

Yeah, looks like a pretty good idea, Tim's right, we need tests for this too. Also, it would be good if The custom functionality was turned on/off with a checkbox too maybe?

Sorry, you have this dependent on the custom format being selected. Awesome! I'll write some tests now.

Here are some tests to go with your patch, also, I changed the check_plain, to filter_xss_admin... Is it possible people will want tags? Should we even let them?

I don't think we need just the tests, they will always fail because this is testing new functionality that only exists in the patch above but not expose anything broken :)

It's cool, imo we generally add just the tests to expose a current bug, they are there no, so there is no harm in that I guess :)

Setting this back top needs review as the patch in #5 is fine, not #6.

+++ b/core/modules/views/lib/Drupal/views/Plugin/views/field/Boolean.phpundefined
@@ -81,7 +104,10 @@ function render($values) {
diff --git a/core/modules/views/lib/Drupal/views/Tests/Handler/FieldBooleanTest.php b/core/modules/views/lib/Drupal/views/Tests/Handler/FieldBooleanTest.php

+++ b/core/modules/views/lib/Drupal/views/Tests/Handler/FieldBooleanTest.phpundefined
@@ -84,7 +84,6 @@ public function testFieldBoolean() {
-

Seems to be out of scope but sure that's okay ;)

#5: 1869124-5.patch queued for re-testing.

#5: 1869124-5.patch queued for re-testing.

#1: views-8.x-boolean_custom_output.patch queued for re-testing.

#5: 1869124-5.patch queued for re-testing.

#6: 1869124-5-tests_only.patch queued for re-testing.

#6: 1869124-5-tests_only.patch queued for re-testing.

Rerolled.

This should be back to RTBC.

Committed/pushed to 8.x, thanks!