DDST-166: Hardening

modules/dgi_actions_handle/dgi_actions_handle.module

@@ -94,15 +94,15 @@ function dgi_actions_handle_node_presave(NodeInterface $node): void {
     $suffix_field = _dgi_actions_handle_get_suffix_field();
 
     // Check if the node has the suffix field.
-    if ($suffix_field && $node->hasField($suffix_field)) {
+    if (!$node->isNew() && $suffix_field && $node->hasField($suffix_field)) {
       // Load the original node from the database.
       $original_node = Drupal::entityTypeManager()->getStorage('node')->loadUnchanged($node->id());
 
       // Check if the field value has been changed.
       if ($node->{$suffix_field}->value != $original_node->{$suffix_field}->value) {
         // If the field value has been changed, revert it to the original value.
         $node->{$suffix_field}->value = $original_node->{$suffix_field}->value;
-        Drupal::messenger()->addMessage('The suffix field cannot be changed.');
+        Drupal::messenger()->addWarning('The suffix field cannot be changed.');
       }
     }
   }

modules/dgi_actions_handle/src/Utility/HandleTrait.php

@@ -62,10 +62,10 @@ public function getPrefix(): string {
    * Gets the suffix for the entity.
    */
   public function getSuffix(): ?string {
-    $suffix_field = $this->getIdentifier()->getServiceData()->getData()['prefix'];
+    $suffix_field = $this->getIdentifier()->getServiceData()->getData()['suffix_field'];
 
     // If a field is configured, use that.
-    if ($this->getEntity()->hasField($suffix_field)) {
+    if ($suffix_field && $this->getEntity()->hasField($suffix_field)) {
       return $this->getEntity()->get($suffix_field)->value;
     }