Add ListNotifications on DataModelProvider

Author: tersal

src/app/WriteHandler.cpp

@@ -258,17 +258,18 @@ CHIP_ERROR WriteHandler::SendWriteResponse(System::PacketBufferTLVWriter && aMes
 
 void WriteHandler::DeliverListWriteBegin(const ConcreteAttributePath & aPath)
 {
-    if (auto * attrOverride = AttributeAccessInterfaceRegistry::Instance().Get(aPath.mEndpointId, aPath.mClusterId))
+    if(mDataModelProvider != nullptr)
     {
-        attrOverride->OnListWriteBegin(aPath);
+        mDataModelProvider->ListAttributeWriteNotification(aPath, DataModel::ListWriteOperation::kListWriteBegin);
     }
 }
 
 void WriteHandler::DeliverListWriteEnd(const ConcreteAttributePath & aPath, bool writeWasSuccessful)
 {
-    if (auto * attrOverride = AttributeAccessInterfaceRegistry::Instance().Get(aPath.mEndpointId, aPath.mClusterId))
+    if(mDataModelProvider != nullptr)
     {
-        attrOverride->OnListWriteEnd(aPath, writeWasSuccessful);
+        mDataModelProvider->ListAttributeWriteNotification(aPath, writeWasSuccessful ? 
+        DataModel::ListWriteOperation::kListWriteEndFinal : DataModel::ListWriteOperation::kListWriteEnd);
     }
 }
 

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

@@ -73,6 +73,13 @@ enum class ReadFlags : uint32_t
     kFabricFiltered = 0x0001, // reading is performed fabric-filtered
 };
 
+enum class ListWriteOperation : uint8_t
+{
+    kListWriteBegin = 0,
+    kListWriteEnd,
+    kListWriteEndFinal
+};
+
 struct ReadAttributeRequest : OperationRequest
 {
     ConcreteAttributePath path;

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

@@ -79,6 +79,14 @@ class Provider : public ProviderMetadataTree
     ///    - validation of ACL/timed interaction flags/writability, if those checks are desired.
     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.
+    ///
+    ///   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.
+    virtual void ListAttributeWriteNotification(const ConcreteAttributePath & aPath, BitFlags<ListWriteOperation> opType) = 0;
+
     /// `handler` is used to send back the reply.
     ///    - returning `std::nullopt` means that return value was placed in handler directly.
     ///      This includes cases where command handling and value return will be done asynchronously.

src/data-model-providers/codegen/CodegenDataModelProvider.h

@@ -63,6 +63,7 @@ class CodegenDataModelProvider : public DataModel::Provider
                                                 AttributeValueEncoder & encoder) override;
     DataModel::ActionReturnStatus WriteAttribute(const DataModel::WriteAttributeRequest & request,
                                                  AttributeValueDecoder & decoder) override;
+    void ListAttributeWriteNotification(const ConcreteAttributePath & aPath, BitFlags<DataModel::ListWriteOperation> opType) override;
     std::optional<DataModel::ActionReturnStatus> Invoke(const DataModel::InvokeRequest & request, TLV::TLVReader & input_arguments,
                                                         CommandHandler * handler) override;
 

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

@@ -198,6 +198,28 @@ DataModel::ActionReturnStatus CodegenDataModelProvider::WriteAttribute(const Dat
     return CHIP_NO_ERROR;
 }
 
+void CodegenDataModelProvider::ListAttributeWriteNotification(const ConcreteAttributePath & aPath, BitFlags<DataModel::ListWriteOperation> opType) {
+    AttributeAccessInterface * aai =
+        AttributeAccessInterfaceRegistry::Instance().Get(aPath.mEndpointId, aPath.mClusterId);
+
+    if(aai != nullptr)
+    {
+        switch(opType)
+        {
+            case DataModel::ListWriteOperation::kListWriteBegin:
+                aai->OnListWriteBegin(aPath);
+                break;
+            case DataModel::ListWriteOperation::kListWriteEnd:
+                aai->OnListWriteEnd(aPath, false);
+                break;
+            case DataModel::ListWriteOperation::kListWriteEndFinal:
+                aai->OnListWriteEnd(aPath, true);
+                break;
+        }
+    }
+
+}
+
 void CodegenDataModelProvider::Temporary_ReportAttributeChanged(const AttributePathParams & path)
 {
     ContextAttributesChangeListener change_listener(CurrentContext());