Hey,

I find an issue when I try to import official interface translations ( *.po file). If I have the 'file_upload_secure_validator' enable, I get this error :

Error while uploading file: MimeTypeGuesser guessed 'text/x-po' and fileinfo 'application/octet-stream'

So I have to disable this module, do all my import and enable it again.

Can we had a configuration to exclude admin page for the check ?

Thanks,
Nicolas

Issue fork file_upload_secure_validator-3081934

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

nambroise created an issue. See original summary.

stefanos.petrakis made their first commit to this issue’s fork.

stefanos.petrakis opened merge request !4

stefanos.petrakis’s picture

stefanos.petrakis
Primary language English
Location Biel, Switzerland
commented

First step: Capture the problem with a test case

stefanos.petrakis closed merge request !3

stefanos.petrakis’s picture

stefanos.petrakis
Primary language English
Location Biel, Switzerland
commented
Status: Active » Needs review

Commited the solution, once the tests come out green, this is ready for review.

jcnventura’s picture

jcnventura
Primary language English
commented

This needs a refactor now that #3183160: Refactor equivalence groups is in.

jcnventura’s picture

jcnventura
Primary language English
commented
Status: Needs review » Needs work
stefanos.petrakis’s picture

stefanos.petrakis
Primary language English
Location Biel, Switzerland
commented
Status: Needs work » Needs review

This one is now ready for review, I kept the new tests and moved the eq. group into the configuration.

jcnventura’s picture

jcnventura
Primary language English
commented

Don't forget you also need to add the new config to the hook_update_8001() function. I've added that to the MR. I can try to test these soon(ish), but I'm happy that the test passes it.

jcnventura’s picture

jcnventura
Primary language English
commented
Status: Needs review » Reviewed & tested by the community
jcnventura’s picture

jcnventura
Primary language English
commented
Status: Reviewed & tested by the community » Needs work

You can't merge this without rebase.. I've done that already for the SVG issue.. I'll do that here, and set back to RTBC once I commt.

jcnventura’s picture

jcnventura
Primary language English
commented
Status: Needs work » Reviewed & tested by the community

stefanos.petrakis’s picture

stefanos.petrakis
Primary language English
Location Biel, Switzerland
commented
Status: Reviewed & tested by the community » Fixed

Commited, thanks!

Status: Fixed » Closed (fixed)

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