Hello,

thank you for this great module that I loved to use under Drupal 7.
I am sorry, I am not able to understand the problem but I thought my repport would help improve the D8 port.
I get the following error on a D8.5 install :

InvalidArgumentException : The configuration property filters.filter_image_resize.settings.image_locations.local doesn't exist. in Drupal\Core\Config\Schema\ArrayElement->get() (line 76 of core/lib/Drupal/Core/Config/Schema/ArrayElement.php). Drupal\Core\Config\StorableConfigBase->castValue('filters.filter_image_resize.settings.image_locations.local', 0) (Line: 211)

The filters aren't saved and module is unusable.

Thank you.

CommentFileSizeAuthor
#5 2955937-filter-schema-5.patch588 bytesgnuget

Comments

Clauce created an issue. See original summary.

thomas.frobieter’s picture

thomas.frobieter
Primary language German
Location EU
commented

Same here - let me know if i can provide more information to help you fixing it.

anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented

Okay I had a look into it and the problem seems to be the nested array created by "checkboxes" FAPI element.

Have a look at:
https://drupal.stackexchange.com/questions/155382/filter-module-wont-let...

That seems to be related. So I'd guess to create an own schema in the module (and as update).

Furthermore there should be a little improvement to the default value (check if the value exists):

'#default_value' => isset($this->settings['image_locations']) ? $this->settings['image_locations'] : [],
thomas.frobieter’s picture

thomas.frobieter
Primary language German
Location EU
commented
Priority: Major » Critical

I set this issue to critical, because you can't save textformats anymore while this module is active.

gnuget’s picture

gnuget
Primary language Spanish
commented
Status: Active » Needs review
StatusFileSize
new 2955937-filter-schema-5.patch588 bytes

Hi!

The schema file is missing in the module. On this patch, I just added it.

(The module hasn't even an alpha release yet, so no upgrade path provided at this time, you might need to reinstall the module to load the schema)

mlncn’s picture

mlncn commented
Status: Needs review » Reviewed & tested by the community

This works!

  • ram4nd committed 3557d71 on 8.x-1.x authored by gnuget 
    Issue #2955937 by gnuget, mlncn: InvalidArgumentException on settings
ram4nd’s picture

ram4nd commented
Status: Reviewed & tested by the community » Fixed
anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented

Thank you so much!! :)

anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented
Status: Fixed » Needs work

We still get
Auf der Website ist ein unvorhergesehener Fehler aufgetreten. Bitte versuchen Sie es später nochmal.InvalidArgumentException: The configuration property filters.filter_image_resize.settings.image_locations.local doesn't exist. in Drupal\Core\Config\Schema\ArrayElement->get() (line 76 of core/lib/Drupal/Core/Config/Schema/ArrayElement.php). 

Drupal\Core\Config\StorableConfigBase->castValue('filters.filter_image_resize.settings.image_locations.local', 'local') (Line: 211)

when saving the filter settings form after update to the latest dev version. Sorry.
gnuget’s picture

gnuget
Primary language Spanish
commented

Did you uninstalled and re-installed the module? (check my comment that there is not a upgrade path because we don't even have an alpha release)

gnuget’s picture

gnuget
Primary language Spanish
commented
Status: Needs work » Fixed

I'm going to mark it as fixed again, if you still have issues even after reinstalling the module, please feel free to re-opening it so we can look closer why is not working for you.

David.

anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented

Sorry. I overread that. Works perfectly after re-install. Time for an alpha release? :)

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.