Rename enum values and add more information in comments.

src/app/WriteHandler.cpp

@@ -269,8 +269,8 @@ void WriteHandler::DeliverListWriteEnd(const ConcreteAttributePath & aPath, bool
     if (mDataModelProvider != nullptr)
     {
         mDataModelProvider->ListAttributeWriteNotification(aPath,
-                                                           writeWasSuccessful ? DataModel::ListWriteOperation::kListWriteEndFinal
-                                                                              : DataModel::ListWriteOperation::kListWriteEnd);
+                                                           writeWasSuccessful ? DataModel::ListWriteOperation::kListWriteSuccess
+                                                                              : DataModel::ListWriteOperation::kListWriteFailure);
     }
 }
 

src/app/data-model-provider/OperationTypes.h

@@ -76,8 +76,8 @@ enum class ReadFlags : uint32_t
 enum class ListWriteOperation : uint8_t
 {
     kListWriteBegin = 0,
-    kListWriteEnd,
-    kListWriteEndFinal
+    kListWriteSuccess,
+    kListWriteFailure
 };
 
 struct ReadAttributeRequest : OperationRequest

src/app/data-model-provider/Provider.h

@@ -80,11 +80,12 @@ class Provider : public ProviderMetadataTree
     virtual ActionReturnStatus WriteAttribute(const WriteAttributeRequest & request, AttributeValueDecoder & decoder) = 0;
 
     ///   Indicates the start/end of a series of list operations. This function will be called either before the first
-    ///   Write operation or after the last one of a series of consequence attribute data of the same attribute.
+    ///   Write operation or after the last one of a series of consecutive attribute data of the same attribute.
     ///
     ///   1) This function will be called if the client tries to set a nullable list attribute to null.
-    ///   2) This function will only be called once for a series of consequent attribute data (regardless the kind of list
-    ///   operation) of the same attribute.
+    ///   2) This function will only be called for a series of consecutive attribute data (regardless the kind of list
+    ///   operation) of the same attribute at the beggining or end of the Write operations.
+    ///   3) The opType argument indicates the type of notification (Start, Failure, Success).
     virtual void ListAttributeWriteNotification(const ConcreteAttributePath & aPath, BitFlags<ListWriteOperation> opType) = 0;
 
     /// `handler` is used to send back the reply.

src/data-model-providers/codegen/CodegenDataModelProvider_Write.cpp

@@ -210,10 +210,10 @@ void CodegenDataModelProvider::ListAttributeWriteNotification(const ConcreteAttr
         case DataModel::ListWriteOperation::kListWriteBegin:
             aai->OnListWriteBegin(aPath);
             break;
-        case DataModel::ListWriteOperation::kListWriteEnd:
+        case DataModel::ListWriteOperation::kListWriteFailure:
             aai->OnListWriteEnd(aPath, false);
             break;
-        case DataModel::ListWriteOperation::kListWriteEndFinal:
+        case DataModel::ListWriteOperation::kListWriteSuccess:
             aai->OnListWriteEnd(aPath, true);
             break;
         }