matter: samples: Remove unused code

Remove old code handling clearing users/credentials from fabric as it's
unused and not spec compilant.

Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no>

Author: adigie

samples/matter/lock/Kconfig

@@ -71,13 +71,6 @@ config SHELL_CMD_BUFF_SIZE
 
 endif
 
-config LOCK_LEAVE_FABRIC_CLEAR_CREDENTIAL
-	bool "Clear all credentials assigned to given fabric index"
-	default y
-	help
-	  This feature allows to remove all credentials which were created or modified
-	  within the fabric which is being removed from the door lock.
-
 config STATE_LEDS
 	bool "Use LEDs to indicate the device state"
 	default y

samples/matter/lock/src/access/access_manager.h

@@ -264,15 +264,6 @@ template <DoorLockData::CredentialsBits CRED_BIT_MASK> class AccessManager {
 	 */
 	bool GetRequirePIN() { return mRequirePINForRemoteOperation; };
 
-#ifdef CONFIG_LOCK_LEAVE_FABRIC_CLEAR_CREDENTIAL
-	/**
-	 * @brief Clear all credentials from the fabric which is currently being removed
-	 *
-	 * @return true on success, false otherwise
-	 */
-	bool ClearAllCredentialsFromFabric();
-#endif
-
 #ifdef CONFIG_LOCK_ENABLE_DEBUG
 	/* DEBUG API allowing to retrieve internally stored credentials and user data */
 	void PrintCredential(CredentialTypeEnum type, uint16_t index);
@@ -336,10 +327,6 @@ template <DoorLockData::CredentialsBits CRED_BIT_MASK> class AccessManager {
 				    DlCredentialStatus credentialStatus, CredentialTypeEnum credentialType,
 				    const chip::ByteSpan &secret);
 
-#ifdef CONFIG_LOCK_LEAVE_FABRIC_CLEAR_CREDENTIAL
-	static bool ClearCredential(DoorLockData::Credential &credential, uint8_t credIdx);
-#endif
-
 	SetCredentialCallback mSetCredentialCallback{ nullptr };
 	ClearCredentialCallback mClearCredentialCallback{ nullptr };
 	ValidateCredentialCallback mValidateCredentialCallback{ nullptr };

samples/matter/lock/src/access/access_manager_credentials.cpp

@@ -145,32 +145,6 @@ CHIP_ERROR AccessManager<CRED_BIT_MASK>::GetCredentialUserId(uint16_t credential
 	return CHIP_ERROR_NOT_FOUND;
 }
 
-#ifdef CONFIG_LOCK_LEAVE_FABRIC_CLEAR_CREDENTIAL
-template <CredentialsBits CRED_BIT_MASK> bool AccessManager<CRED_BIT_MASK>::ClearAllCredentialsFromFabric()
-{
-	return mCredentials.ForEach([](DoorLockData::Credential &credential, uint8_t credIdx) {
-		/* At this point the door-lock-server already invalidated both mCreatedBy and mLastModifiedBy
-		    of all credentials assigned to the fabric which is currently being removed */
-		if (credential.mInfo.mFields.mCreatedBy == kUndefinedFabricIndex &&
-		    credential.mInfo.mFields.mLastModifiedBy == kUndefinedFabricIndex &&
-		    credential.mSecret.mDataLength != 0) {
-			return Instance().ClearCredential(credential, credIdx);
-		}
-		return true;
-	});
-}
-
-template <CredentialsBits CRED_BIT_MASK>
-bool AccessManager<CRED_BIT_MASK>::ClearCredential(DoorLockData::Credential &credential, uint8_t credIdx)
-{
-	credIdx++; /* DoSetCredential expects indexes starting from 1 */
-	return DoSetCredential(credential, credIdx, kUndefinedFabricIndex, kUndefinedFabricIndex,
-			       DlCredentialStatus::kAvailable,
-			       static_cast<CredentialTypeEnum>(credential.mInfo.mFields.mCredentialType),
-			       chip::ByteSpan());
-}
-#endif
-
 template <CredentialsBits CRED_BIT_MASK> void AccessManager<CRED_BIT_MASK>::LoadCredentialsFromPersistentStorage()
 {
 	uint8_t credentialData[DoorLockData::Credential::RequiredBufferSize()] = { 0 };
@@ -186,8 +160,8 @@ template <CredentialsBits CRED_BIT_MASK> void AccessManager<CRED_BIT_MASK>::Load
 		}
 		outSize = 0;
 		if (!AccessStorage::Instance().Load(AccessStorage::Type::CredentialsIndexes,
-							 credentialIndexesSerialized,
-							 sizeof(credentialIndexesSerialized), outSize, type)) {
+						    credentialIndexesSerialized, sizeof(credentialIndexesSerialized),
+						    outSize, type)) {
 			LOG_INF("No stored indexes for credential of type: %d", type);
 			continue;
 		}
@@ -203,8 +177,7 @@ template <CredentialsBits CRED_BIT_MASK> void AccessManager<CRED_BIT_MASK>::Load
 			uint16_t credentialIndex = credIndexes.mList.mIndexes[idx];
 			outSize = 0;
 			if (!AccessStorage::Instance().Load(AccessStorage::Type::Credential, credentialData,
-								 sizeof(credentialData), outSize, type,
-								 credentialIndex)) {
+							    sizeof(credentialData), outSize, type, credentialIndex)) {
 				LOG_ERR("Cannot load credentials of type %d for index: %d", static_cast<uint8_t>(type),
 					credentialIndex);
 			}
@@ -217,7 +190,7 @@ template <CredentialsBits CRED_BIT_MASK> void AccessManager<CRED_BIT_MASK>::Load
 	}
 	outSize = 0;
 	if (!AccessStorage::Instance().Load(AccessStorage::Type::RequirePIN, &mRequirePINForRemoteOperation,
-						 sizeof(mRequirePINForRemoteOperation), outSize) ||
+					    sizeof(mRequirePINForRemoteOperation), outSize) ||
 	    outSize != sizeof(mRequirePINForRemoteOperation)) {
 		LOG_DBG("Cannot load RequirePINforRemoteOperation");
 	}