bug #6872 fix: Do not override HTML attribute title if already set for Action… (jbonnier)

This PR was merged into the 4.x branch.

Discussion
----------

fix: Do not override HTML attribute `title` if already set for Action…

If an action has no label but has the HTML attribute already set, it should not be replaced by the action name.

Use case example:

```php
$moveDown = Action::new('moveDown', false, 'fa fa-angle-down')
    ->setHtmlAttributes(['title' => $this->trans('Move Down')])
    ->linkToCrudAction('moveDown')
    ->displayIf(static fn(SortableEntityInterface $entity): bool => $entity->getPosition() < $entityCount - 1)
;
```

Commits
-------

4dd8bae fix: Do not override HTML attribute `title` if already set for Actions without a label

Author: javiereguiluz

src/Factory/ActionFactory.php

@@ -157,9 +157,10 @@ private function processAction(string $pageName, ActionDto $actionDto, ?EntityDt
     private function processActionLabel(ActionDto $actionDto, ?EntityDto $entityDto, string $translationDomain, array $defaultTranslationParameters): void
     {
         $label = $actionDto->getLabel();
+        $title = $actionDto->getHtmlAttributes()['title'] ?? '';
 
         // FALSE means that action doesn't show a visible label in the interface
-        if (false === $label) {
+        if (false === $label && '' === trim($title)) {
             $actionDto->setHtmlAttribute('title', $actionDto->getName());
 
             return;