When editing an existing config, the file type(s) check boxes are no longer selected, despite the config overview on the "Global" tab correctly showing it(them).

CommentFileSizeAuthor
#4 file_type_config-1794166-4.patch749 bytestinker
#3 edit.jpg17.62 KBpaulbeaney
#3 config.jpg32.46 KBpaulbeaney

Comments

szantog’s picture

szantog commented
Status: Active » Postponed (maintainer needs more info)

As I know, this is fixed in latest dev, can't reproduce now. Did you run update.php?

paulbeaney’s picture

paulbeaney commented

Strange this: in config.jpg you can see the config I have, with Image type clearly selected. If I click edit, I get what you see in edit.jpg - nothing is selected. If I save the config without touching any of the values, the Image type gets retained (!?) However, if I change the content type to, say, 'Audio', the change DOES get saved.

I did run update.php and my system table is showing this module as being at update 7002.

Any ideas??

paulbeaney’s picture

paulbeaney commented
StatusFileSize
new config.jpg32.46 KB
new edit.jpg17.62 KB

Forget the images in #2...

tinker’s picture

tinker commented
Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new file_type_config-1794166-4.patch749 bytes

I have checked and this is a bug in alpha2 and dev. I have attached a patch made on dev but should work for both as its a one line change and on the same line.

szantog’s picture

szantog commented
Status: Needs review » Fixed

Hmm.. This was fixed, but became wrong again.. Added test to check it.
@tinker: this path is inverz, the correct code on this line is: 

$file_types_defaults = isset($config->data['file_entity']) ? $config->data['file_entity'] : array();

Commited: http://drupalcode.org/project/fe_paths.git/commit/d7164af32a7c17a93dda5c...

tinker’s picture

tinker commented

Hey thx for the fast commit. Yep got my head and working branches reversed. Sorry about that.

Status: Fixed » Closed (fixed)

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