I have tested this item ✅ successfully on 0059c80

Hmm, I'm not sure if this PR is right. Currently without the PR, the trim is applied on the alias before the check if it is empty:

$this->alias = trim($this->alias);

if (empty($this->alias)) {
    $this->alias = $this->title;
}

So if someone enters spaces as alias in the form, that will not be used, and the title will be used.

Now with the PR you move trim to after the check, so if someone enters spaces as alias in the form, it will be used and not the title because the empty check for a string with spaces will return false:

if (empty($this->alias)) {
    $this->alias = $this->title;
}

$this->alias = trim($this->alias);

Maybe the following code would be better?

$this->alias = trim($this->alias ?? '');

if (empty($this->alias)) {
    $this->alias = $this->title;
}

I.e. leave order of processing like it is without this PR and use the null coalescing operator inside the trim call to not let it operate on a null value.

Agree with Richard. @alikon Could you please implement the change suggested by Richard?

done

I have tested this item ✅ successfully on 3ac3c7c

RTC

Thanks @alikon !