Field type: Block issue

[betterapp]

We have such field in the Book brick:

We have dataport that does not touch this field at all.

BUT

We see in log something like this:
[INFO] Reason for saving: Field ReviewBlock in brick Books changed

but there is no information about new value for it like:
Value for field ReviewBlock: xxxx

Logs show only information abut that this field was changed

Dataport has been started
Starting raw data processing for dataport "API SDBooks - Products - Create & Update" (#258)
Will only import raw items with ids 41280544233565940567791131869683127087
[INFO] --- Processing next raw data item
[INFO] Value for key field "EAN": 5035393051129
[INFO] Importing object /Products/Draft/SDB_NotGrey/SDB/Really Tiny Book Light - Petal Pink - 5035393051129
[INFO] Value for field EAN: 5035393051129
[INFO] Value for field key: Really Tiny Book Light - Petal Pink - 5035393051129
[INFO] Value for field published: true
[INFO] Value for field CountryOfOrigin: 
[INFO] Value for field TechnicalSource: SDB
[INFO] Field "Currency" is populated and configured to be write-protected -> won't change value
[INFO] Value for field DistributorCode: SD
[INFO] Field "Height" is populated and configured to be write-protected -> won't change value
[INFO] Field "HeightUnit" is populated and configured to be write-protected -> won't change value
[INFO] Value for field ImportedName: Really Tiny Book Light - Petal Pink
[INFO] Field "Length" is populated and configured to be write-protected -> won't change value
[INFO] Field "LengthUnit" is populated and configured to be write-protected -> won't change value
[INFO] Value for field LifecycleStatus: green
[INFO] Value for field MinPurchQtyND: 1
[INFO] Value for field MinPurchQtyStore: 1
[INFO] Value for field NumerFpakInDpak: 1
[INFO] Field "OutgoingVATCode" is populated and configured to be write-protected -> won't change value
[INFO] Value for field ProductOwner: /Vareeiere/SD Books
[INFO] Value for field ProductSubtitle: NULL
[INFO] Value for field __virtual_PRODUCT_TYPE_NEW: NONBOOK
[INFO] Value for field ProductType: NONBOOK
[INFO] Value for field ReleaseDate: NULL
[INFO] Value for field SalesPrice: 168
[INFO] Value for field StatusCodeSD: IP
[INFO] Field "Suppliers" is populated and configured to be write-protected -> won't change value
[INFO] Value for field SupplierSKU: 5035393051129
[INFO] Value for field Template: Pliktige varer
[INFO] Field "Weight" is populated and configured to be write-protected -> won't change value
[INFO] Field "WeightUnit" is populated and configured to be write-protected -> won't change value
[INFO] Field "Width" is populated and configured to be write-protected -> won't change value
[INFO] Field "WidthUnit" is populated and configured to be write-protected -> won't change value
[INFO] Current [TechnicalSource] = SDB
[INFO] Current [ProductType] = NONBOOK
[INFO] Value for field Books/BookBinding: nn
[INFO] Value for field Books/BookGroup: [
    "/Litteraturattributter/Bokgrupper/8 - Diverse andre varer/8.3 - Annet/8.3.0 - Diverse [[]]"
]
[INFO] Value for field Books/BookLanguage: NULL
[INFO] Value for field Books/BookLanguageOriginal: NULL
[INFO] Value for field Books/BookSeries: []
[INFO] Value for field Books/Contributor: []
[INFO] Current [TechnicalSource] = SDB
[INFO] Current [ProductType] = NONBOOK
[INFO] Value for field Books/NumberOfPages: NULL
[INFO] Value for field Books/ProductNameOriginalLanguage: NULL
[INFO] Value for field Books/Publisher: NULL
[INFO] Value for field Books/PublishingYear: NULL
[INFO] Value for field Books/StatusBokbasenDescription: Grønn
[INFO] Reason for saving: Field ReviewBlock in brick Books changed
[INFO] object /Products/Draft/SDB_NotGrey/SDB/Really Tiny Book Light - Petal Pink - 5035393051129 queued for saving
[INFO] Successfully saved object #515270 /Products/Draft/SDB_NotGrey/SDB/Really Tiny Book Light - Petal Pink - 5035393051129 (version #79883926)
Duration: 1.52 seconds, Peak memory: 56.84 MB

DD: 3.9.19

@BlackbitDevs

[betterapp]

Also what I see: DataDirector always compare all fields values from class/brick definition.
But why ?

If we want to change only few fields in dataport ?
I think it should check only mapped fields.

[betterapp]

lib/Pim/Item/Importer.php

} elseif ($fieldDefinition instanceof Data\Block || $fieldDefinition instanceof Data\Fieldcollections) {
    return serialize($value1) === serialize($value2);
}

should be like this:

} elseif ($fieldDefinition instanceof Data\Block) {
            $block1Values = $block2Values = [];
            foreach ($value1 ?? [] as $rowIndex => $row) {
                foreach ($row as $fieldName => $blockElement) {
                    $block1Values[$rowIndex][$fieldName] = $blockElement->getData();
                }
            }
            foreach ($value2 ?? [] as $rowIndex => $row) {
                foreach ($row as $fieldName => $blockElement) {
                    $block2Values[$rowIndex][$fieldName] = $blockElement->getData();
                }
            }

            return serialize($block1Values) === serialize($block2Values);
        } elseif ($fieldDefinition instanceof Data\Fieldcollections) {
            return serialize($value1) === serialize($value2);
        } elseif ($fieldDefinition instanceof Data\AdvancedManyToManyObjectRelation || $fieldDefinition instanceof Data\AdvancedManyToManyRelation) {