From 19cf29b9a46ef71e67dea5baf3c6cf7498b4e404 Mon Sep 17 00:00:00 2001
From: Ryuji AMANO <ryuji@ryus.co.jp>
Date: Sun, 22 Jan 2023 11:04:02 +0900
Subject: [PATCH] =?UTF-8?q?fix:=20=E3=83=AD=E3=82=B0=E3=82=A4=E3=83=B3ID(u?=
 =?UTF-8?q?sername)=E5=A4=89=E6=9B=B4=E6=99=82=E3=81=AB=E3=83=AD=E3=82=B0?=
 =?UTF-8?q?=E3=82=A4=E3=83=B3ID=E3=81=AE=E3=83=90=E3=83=AA=E3=83=87?=
 =?UTF-8?q?=E3=83=BC=E3=82=B7=E3=83=A7=E3=83=B3=E3=81=8C=E5=8B=95=E4=BD=9C?=
 =?UTF-8?q?=E3=81=97=E3=81=AA=E3=81=84=E4=B8=8D=E5=85=B7=E5=90=88=E3=82=92?=
 =?UTF-8?q?=E4=BF=AE=E6=AD=A3?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 Model/User.php | 30 +++++++++++++++++++++++++++++-
 1 file changed, 29 insertions(+), 1 deletion(-)

diff --git a/Model/User.php b/Model/User.php
index 37b2089..3d50c80 100644
--- a/Model/User.php
+++ b/Model/User.php
@@ -303,7 +303,7 @@ public function beforeValidate($options = array()) {
  */
 	protected function _setUsernameValidate() {
 		//ログインID
-		if (! Hash::get($this->data, 'User.id')) {
+		if ($this->__isValidateUsernameRequired()) {
 			$this->validate = ValidateMerge::merge($this->validate, array(
 				'username' => array(
 					'notBlank' => array(
@@ -334,6 +334,34 @@ protected function _setUsernameValidate() {
 		}
 	}
 
+/**
+ * ログインID(username)のバリデーションを行うかどうか
+ *
+ * @return bool
+ */
+	private function __isValidateUsernameRequired() {
+		if (!Hash::get($this->data, 'User.id')) {
+			// 新規登録ならば、必ずバリデーションを行う
+			return true;
+		}
+		// 更新時
+		if (!isset($this->data['User']['username'])) {
+			// ログインID(username)がPOSTされてなければバリデーション不要
+			return false;
+		}
+		$beforeUser = $this->find('first',
+			[
+				'conditions' => [
+					'id' => $this->data['User']['id']
+				],
+				'recursive' => -1,
+				'fields' => ['username']
+			]
+		);
+		// ログインID(username)が変更されてなければバリデーション不要。変更されてたらバリデーション必要
+		return $beforeUser['User']['username'] !== $this->data['User']['username'];
+	}
+
 /**
  * バリデーションのセット(パスワード)
  *