[3.1] White labelling error when using multiple logo's and env variables

[robdekort]

Bug Description

When using multiple custom logo URL's this doesn't work:

'custom_logo_url' => [
    'nav' => env('STATAMIC_CUSTOM_LOGO_NAV_URL', null),
    'outside' => env('STATAMIC_CUSTOM_LOGO_OUTSIDE_URL', null)
]

Gives an error:

ErrorException
htmlspecialchars() expects parameter 1 to be string, array given (View: /Users/robdekort/Sites/anne-meithe/vendor/statamic/cms/resources/views/partials/global-header.blade.php)

It wil work when you set the logo's to empty strings, but then the CP doesn't fallback to the default logo's.

How to Reproduce

Try to use .env variables for setting the nav and outside logo in config/statamic/cp.php like above.

Extra Detail

Environment

Statamic 3.1.0-beta.1 Pro
Laravel 8.33.1
PHP 7.4.13
No addons installed

Install method (choose one):

• Starter kit

[duncanmcclean]

If you run php artisan tinker and enter config('statamic.cp.custom_logo_url'), what's returned?

[robdekort]

This is what's returned when it's set to null. Then I get the above error.

When I set the env to nothing I don't get the error, but here's no logo in the CP nav bar. Then it returns this.

So with the current options it seems you can't keep the default Statamic logo in the nav bar.

[robdekort]

I just noticed this on another site which might be relevant:

[jasonvarga]

On that site, have you published the views? Do you have a login.blade.php?

[robdekort]

Yep, that was it indeed. I just wanted to remove my comment. Hahaha. And it seems you fixed the initial issue already. Awesome!