V8-archive: Can't upload logo in Global Settings

[jameswill]

Hi,

I was trying to upload logo in Global Settings in Directus 7.0.8 version and it gives error stating "Attempting to write an array as the value for field "value" in "directus_settings""

On checking error log file, I found [2018-11-20 12:23:45] api[_].ERROR: Directus\Database\Exception\SuppliedArrayAsColumnValue: Attempting to write an array as the value for field "value" in "directus_settings" in {path}/{To}/{directus708}/src/core/Directus/Database/TableGateway/BaseTableGateway.php:1737

Also while updating color from the global settings, it gives "scope: This value should not be blank." error.

[2018-11-20 12:26:16] api[_].ERROR: Directus\Validator\Exception\InvalidRequestException: value: This value should not be blank. in {path}/{To}/{directus708}/src/core/Directus/Services/AbstractService.php:188

What is causing this ?

[benhaynes]

Hmm, that's odd. We just implemented this a few days ago so perhaps there are a few issues... maybe DB migration problems?

Any ideas @rijkvanzanten ?

@jameswill — how did you install 7.0.8? Clone, zip, Docker? Was it an upgrade from an older version or fresh?

[shealavington]

I had a brief look and had the same errors on directus.app, haven't done any in-depth testing as to why though.

[rijkvanzanten]

Ughh yeah I know why.. The file interface will send the value as a file object (eg _array_ in php) while the directus_settings logo field expects just a sting. This is most likely a misconfiguration in the database where the logo field in directus_settings is set up to use the string data type instead of the file datatype. I have to confirm this though

[jameswill]

@jameswill — how did you install 7.0.8? Clone, zip, Docker? Was it an upgrade from an older version or fresh?

I have installed through zip archive downloaded from https://github.com/directus/directus

It is a fresh install

[stickgrinder]

I don't know if this may be related but I am able to upload a logo (it seems) but saving the global settings always results in the following error:

scope: This value should not be blank.

displayed in a flash message at the rightmost bottom of the screen... any suggestion?

[andreekeberg]

I have the exact same issue. First hit with the Attempting to write an array as the value for field "value" in "directus_settings" and then when trying again with the same logo but also changing the color got the scope: This value should not be blank.

I looked in the database (in directus_fields) and logo in the directus_settings collection already has the type "file".

Is there anything I can do to fix this?

[rijkvanzanten]

@WellingGuzman The API should use the directus_fields settings for directus_settings when updating settings. I think the API is using the datatype for the value column in directus_settings (string) instead of the datatype for the logo row in directus_fields (file) which causes the API to error out (because the file is an object instead of a string and doesn't "fit" the field).

[wellingguzman]

I will work on making each setting record match their directus_fields type.

@stickgrinder you need to upgrade your database. it should removes the scope field from the directus_settings. You can read how to upgrade using the /update endpoint or the db:upgrade cli command.

[stickgrinder]

@WellingGuzman many thanks for this information. I'll try ASAP.

[rijkvanzanten]

@WellingGuzman @stickgrinder We've also recently added a update DB button in the settings menu if you prefer doing it from the admin app 😄

[stickgrinder]

@WellingGuzman @stickgrinder We've also recently added a update DB button in the settings menu if you prefer doing it from the admin app smile

Wow! :D I should update the docker image I guess ;)

[wellingguzman]

@stickgrinder Let us know if that worked for you. You also need to get the most recent version of your Directus in order to get the latest migration files used to upgrade the database.

[stickgrinder]

OK, I just updated to the last docker images: I'm now running API v2.0.10 and application 7.0.10.

I upgraded the database both via the button in the app admin section that running bin/directus db:upgrade inside the container.

Still when I try to upload a file via backend, a flash message says Attempting to write an array as the value for field "value" in "directus_settings".

Did I miss something?

[rijkvanzanten]

@stickgrinder nah that's the issue @wellingguzman is solving:

I will work on making each setting record match their directus_fields type.

The one solved by updating the DB was the missing column scope 🙃

[stickgrinder]

It was a bit confusing actually... I didn't get the reason to update the DB then :sweat_smile: sorry!
Anyway, great to see that the docker images are working like a charm! :smile:

[jameswill]

Hi,

Issue still persists in Directus 7.0.11 version as well.

[kjj77]

Hi guys,

I've installed the Directus suite, I've updated to the latest git commit (b0c93e7189a0a7a5292d8e9d265fc1b69d09150c) and I'm having too the same error while trying to save the logo :

Attempting to write an array as the value for field "value" in "directus_settings"

Tried to update DB via the admin app, nothing has changed either.
This is being tested on an AWS EC2 instance with an ordinary bitnami-lampstack-7.1.22-1-r40-linux-ubuntu-16.04-x86

[stickgrinder]

Did you guys run a db:upgrade?

[kjj77]

Just run it and re-tested it, same error though

[benhaynes]

I've just elevated today to the high priority board. We'll look into what's going on here first thing tomorrow (@WellingGuzman is off today).

[wellingguzman]

This still is a problem that we need to fix. we will try to fix it as soon as possible. I will keep you all posted.

[devninjajeremy]

Has this been fixed? I'm getting the same issue when trying to upload a logo. I have a fresh master branch installed. I'm nervous to switch to the development branch because this is running in production currently for a client.

[benhaynes]

Still working on this one @devninjajeremy — though it's high on our list.

[alrick]

Is there a workaround to set the logo in the meantime ?

[carolin-skiply]

Hi guys,
I am having the same issue with the logo as well. Any idea on a temporary fix ?

[wellingguzman]

At the moment there's not temporary fix.

[gamito]

Hi, guys.
Is there is any progress on this?
Thank you.

[rijkvanzanten]

Hey @gamito! This was actually fixed just now in https://github.com/directus/api/pull/821 😄

[justrealmilk]

Experiencing this error on 7.1 with Thumbnail Quality Tags and Thumbnail Actions

[RVaniuta]

Have same issue with Thumbnail Quality Tags and Thumbnail Actions too

[victoriris]

Still having the problem with the current latest version. Now, the logo gets 'saved' but the interface never shows it

[theharshin]

The logo is being saved successfully. Just need to change some variables on APP side to reflect it. This PR should fix it. https://github.com/directus/app/pull/1510