Galleria Not Valid

Hi Will,

There are no stable versions of Galleria - the code is in beta, which means it's still liable to change. The most recent warning you are getting is due to new code that I have added to check whether a node is a valid galleria.

Part of this new check is to ensure that all attached files have an image extension. I would imagine this is where your node is failing....

'jpg', 'png', 'jpeg', 'gif'

are the extensions it's looking for.

Does that help?

Hi Mark!

I still have the same issue in spite of your recommandation for the file format (I have 2 .jepg files for testing).

I enabled the Attachments and Gallleria into Workflow settings into my content type. I attached 2 .jepg with the File Attachment section and when I display my node I get the error message.

Any idea?

Steph

Hi Mark,

I found our issue into function galleria_is_valid(): .jpeg is actually not support because we check the file extension by keeping the 3 last characters only...

Stephane

Thanks, I will look into it next year... I will start by changing the error message to be something more specific, so we can see where the problem is.

I also forgot: the test is also case-sensitive...maybe it is a bit too restrictive?

Yeah... that's probably it ... :P

Apologies, will sort it out in the New Year!

Yeah I ran into this as well, due to .JPG images. File rename to .jpg fixed things.

also need to check that the extension is not 4 characters.

Fixed in Beta 2

I've added JPG, JPEG, PNG and GIF to line 30 of galleria.module (allowed extensions). Hope nobody minds. Just be sure you delete .txt from the file name when you upload.

Sorry. Here's an up-to-date (beta 2) galleria.module with line 28 amended to include uppercase extensions. Of course, I don't know enough about code to know whether the uppercase versions are necessary. Just trying to be helpful.

Remember to delete the .txt file extension when you upload.

Actually the following line of code in Galleria ensures that you don't need to include uppercase and lowercase values in the allowed extensions list:

$ext = strtolower(substr($file->filename, -4));

It means "take the last four characters of the filename, convert them to lowercase". So the comparison will always be between two lowercase extensions.

But thanks anyway ;)

You're welcome. I reckoned you might have worked some sort of magic like that.