autoGallery

A little bit of logic could fix the naming but this started as a trivial bit of code written in 40mins to solve a quick problem Its on my expanding list of things to do, although Im not quite sure what a complete way round it would be, since just checking that the problem doesn’t exist wouldn’t preclude the possibility of someone adding a file later that DID clash.

You are right about the modx db prefix, that came up for me last week and is top of the imaginary list of things to add in yet another release. Been a bit busy though as my daughter was born this morning after 15 hours of labour, so I haven’t actually slept for two days...


Ramsay

Congrats on your new family addition!

In the meantime I’ve made a lot of structural and functional changes in your original AutoGallery code. It still needs some clean-up here and there, but I think it’s working quite well.

I’ve added:
- support for gif format
- now it respects modx table prefix and db charsets
- now it adds entries into db

Of course it needs testing. All comments and suggestions are welcome

Thanks for your efforts, fixedmachine!

Looks like you’ve tidied up all the dodgy bits I had been meaning to do, and fixed all the bits where I needed to learn a few php tricks

If its ok with you I’ll stick it up on our site as AG2.3 and try to work out how to use the MODx CMS ReleaseMe thingy again to put it on here.


Ramsay

No problem You can of course put it on your site! It’s on GPL isn’t it?

I thinking about some improvements though.

I hope that my work will be useful for somebody

I cannot for the life of me get AutoGallery to work properly. I’m using version 2.3, acquired today from http://www.zombietuesday.com

I CAN get AutoGallery to create a single thumbnail, called "tn_.jpg" which it thinks is the name of all the thumbnails (obviously, that’s not right).

My autogallerycall chunk looks like this:



and my agItemChunk looks like so:



and the output of the debug portion of the call looks like this:



Obviously, I’m very much trying to trouble shoot this most promising of snippets, but am not getting anywhere.

This seems so much like it’s close to working, but that there is some sort of problem with the section of code that outputs the name of the thumbnail.

Any ideas?

I love responding to my own posts. Seriously, I hope this helps someone.

I am certainly no php guru, but I think I found the problem with version 2.3’s thumbnail generation code. Specifically, on line 162 where the thumbnail filename is created by concatenating several elements, I think there’s a bug.

Currently, in version 2.3 of AutoGallery.inc.php, the code is like so:

The problem is, $path_parts was declared only a few lines before in line 157 as:

While $path_parts does contain something, $path_parts[’filename’] does not (at least on my system).

The fix therefore is to change line 162 to read:

With this change made, the code does indeed create multiple thumbnail images AND returns a different filename for each image (rather important).

Hope I’m on the right track here.

Hi Wyleyrabbit,

Thats very strange! What OS and version of PHP are you using? I confess I have only tried this on our Linux server with PHP 5.2.9.

The workaround you suggests seems sensible, though. I’ll put it up later today.



Ramsay

Hi Ramsay,

I’m on a Centos 4.8 server (same as Redhat Enterprise Linux), running PHP 5.1.6. The fix seemed to work great on this side, so I hope it helps someone.

Now, the next part I need to figure out is the best way to get the image descriptions into the mix. While I would be fine working with phpMyAdmin or similar, unfortunately that’s not a realistic solution for my client who will be creating the content.

So I was thinking that while there are likely many ways to get the data in there, the easiest solution would likely be a text file that is stored in the image gallery directory that would contain the name of the large image (ie. "testfile0003.jpg"), then a delimiter, then the description. Perhaps like so:



Also, I was thinking that the file name could be specified as an argument passed along to the snippet.

Thoughts Ramsay? Or anyone else?

Cheers,
Wyleyrabbit

Hi,

I have a set of pages and a snippet for uploading files including descriptions, and for modifying existing ones to add descriptions.

I’m packaging it up to port it to a new site of ours, so I can put it up once its working, if that would help?


Ramsay