From dbfddeb48ae0d96fee279598885e4ce59ada2249 Mon Sep 17 00:00:00 2001
From: Paul Holden <paulh@moodle.com>
Date: Tue, 19 Dec 2023 11:36:13 +0000
Subject: [PATCH] MDL-74500 profilefield_checkbox: override show/empty class
 methods.

---
 user/profile/field/checkbox/field.class.php   |  25 ++++
 .../tests/profile_field_checkbox_test.php     | 122 ++++++++++++++++++
 2 files changed, 147 insertions(+)
 create mode 100644 user/profile/field/checkbox/tests/profile_field_checkbox_test.php

diff --git a/user/profile/field/checkbox/field.class.php b/user/profile/field/checkbox/field.class.php
index a6e0885f8ec..6ddfc03a5b7 100644
--- a/user/profile/field/checkbox/field.class.php
+++ b/user/profile/field/checkbox/field.class.php
@@ -39,6 +39,31 @@ class profile_field_checkbox extends profile_field_base {
         }
     }
 
+    /**
+     * Override parent {@see profile_field_base::is_empty} check
+     *
+     * We can't check the "data" property, because if not set by the user then it's populated by "defaultdata" of the field,
+     * which can also be 0 (false) therefore ensuring the parent class check could never return true for this comparison
+     *
+     * @return bool
+     */
+    public function is_empty() {
+        return ($this->userid && !$this->field->hasuserdata);
+    }
+
+    /**
+     * Override parent {@see profile_field_base::show_field_content} check
+     *
+     * We only need to determine whether the field is visible, because we also want to show the "defaultdata" of the field,
+     * even if the user hasn't explicitly filled it in
+     *
+     * @param context|null $context
+     * @return bool
+     */
+    public function show_field_content(?context $context = null): bool {
+        return $this->is_visible($context);
+    }
+
     /**
      * Display the data for this field
      *
diff --git a/user/profile/field/checkbox/tests/profile_field_checkbox_test.php b/user/profile/field/checkbox/tests/profile_field_checkbox_test.php
new file mode 100644
index 00000000000..d008e859519
--- /dev/null
+++ b/user/profile/field/checkbox/tests/profile_field_checkbox_test.php
@@ -0,0 +1,122 @@
+<?php
+// This file is part of Moodle - http://moodle.org/
+//
+// Moodle is free software: you can redistribute it and/or modify
+// it under the terms of the GNU General Public License as published by
+// the Free Software Foundation, either version 3 of the License, or
+// (at your option) any later version.
+//
+// Moodle is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+// GNU General Public License for more details.
+//
+// You should have received a copy of the GNU General Public License
+// along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
+
+namespace profilefield_checkbox;
+
+use advanced_testcase;
+use profile_field_checkbox;
+
+/**
+ * Unit tests for the field class
+ *
+ * @package     profilefield_checkbox
+ * @covers      \profile_field_checkbox
+ * @copyright   2024 Paul Holden <paulh@moodle.com>
+ * @license     http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+class profile_field_checkbox_test extends advanced_testcase {
+
+    /**
+     * Load required test libraries
+     */
+    public static function setUpBeforeClass(): void {
+        global $CFG;
+        require_once("{$CFG->dirroot}/user/profile/lib.php");
+    }
+
+    /**
+     * Data provider for {@see test_is_empty}
+     *
+     * @return array[]
+     */
+    public static function is_empty_provider(): array {
+        return [
+            'No value' => [
+                [],
+                true,
+            ],
+            'Value equals 0' => [
+                ['profile_field_check' => 0],
+                false,
+            ],
+            'Value equals 1' => [
+                ['profile_field_check' => 1],
+                false,
+            ],
+        ];
+    }
+
+    /**
+     * Test field empty state
+     *
+     * @param array $userrecord
+     * @param bool $expected
+     *
+     * @dataProvider is_empty_provider
+     */
+    public function test_is_empty(array $userrecord, bool $expected): void {
+        $this->resetAfterTest();
+
+        $this->getDataGenerator()->create_custom_profile_field([
+            'datatype' => 'checkbox',
+            'name' => 'My check',
+            'shortname' => 'check',
+        ]);
+
+        $user = $this->getDataGenerator()->create_user($userrecord);
+
+        /** @var profile_field_checkbox[] $fields */
+        $fields = profile_get_user_fields_with_data($user->id);
+        $fieldinstance = reset($fields);
+
+        $this->assertEquals($expected, $fieldinstance->is_empty());
+    }
+
+    /**
+     * Test whether to show field content
+     */
+    public function test_show_field_content(): void {
+        $this->resetAfterTest();
+
+        $this->getDataGenerator()->create_custom_profile_field([
+            'datatype' => 'checkbox',
+            'name' => 'My check',
+            'shortname' => 'check',
+            'visible' => PROFILE_VISIBLE_PRIVATE,
+        ]);
+
+        // User can view their own value.
+        $userwith = $this->getDataGenerator()->create_user(['profile_field_check' => 1]);
+        $this->setUser($userwith);
+
+        /** @var profile_field_checkbox[] $fields */
+        $fields = profile_get_user_fields_with_data($userwith->id);
+        $fieldinstance = reset($fields);
+
+        $this->assertTrue($fieldinstance->show_field_content());
+
+        // Another user cannot view the value.
+        $userview = $this->getDataGenerator()->create_user();
+        $this->setUser($userview);
+
+        $this->assertFalse($fieldinstance->show_field_content());
+
+        // Another user with appropriate access can view the value.
+        $this->setAdminUser();
+        $this->assertTrue($fieldinstance->show_field_content());
+
+    }
+}