Hi @rdeutz I have changed getClientId() with isAdmin() as per request.

@test success

(int) $this->params->get($side, 0) is used twice. Can be refactored as $index.

The if can be more readable if changed to

if (isset($skindirs[$index]))

or whole thing as a ternary, maybe!

$side  = $app->isAdmin() ? 'skin_admin' : 'skin';
$index = (int) $this->params->get($side, 0);
$skin  = isset($skindirs[$index]) ? basename($skindirs[$index]) : 'lightgray';
$skin  = 'skin : "' . $skin . '",';

Yours is good too. I'm just suggesting. 👍

What is the status?
Can we test this, if so can we have a Test scenario?

IMO, This is most of a code review.

However the current code (without this patch) would break in the case where $app is neither site nor administrator. Try loading TinyMCE editor in the installation application with error display and error reporting on.

Or you can create a third application similar to administrator with client_id = 3 also. But that is too much of work for this small test.

I have tested this item ✅ successfully on 3ed4624

Opened TinyMCE normally, worked w/o a problem. Added a tinymce instance in installation app manually, worked as well. Code review seems fine, too. @icampus

I have tested this item ✅ successfully on 3ed4624

tested @icampus:
simplification does what it should

@demis-palma Could you make the suggested changes please?