Comments on: Easy PHP Photo Gallery

By: David

David — Fri, 02 May 2008 20:35:06 +0000

Thanks for the script! Very Handy!

By: Joe Dolson

Joe Dolson — Wed, 16 Apr 2008 14:32:00 +0000

Go ahead and do as you will — - I’m not really all that concerned about the licensing terms. I’d appreciate a link in some way; but I’m not going to insist on it.

By: Ian

Ian — Wed, 16 Apr 2008 09:04:26 +0000

Hi Joe

I am just learning PHP & mysql at the moment and came across your great script.

It does seem by far the most straightforward I have found.

I would like to use it to help me learn PHP and adapt it to use on my own site eventually. To this end, the finished script would therefore be used commercially by my small business - what arrangement do you have for this?

The designs I am working on would not allow an obvious acknowledgement to you as I am offering web design design services.

Thanks in advance for your response.

Best regards

Ian Lain

By: Joe Dolson

Joe Dolson — Fri, 28 Mar 2008 15:50:42 +0000

Thanks for your comments and compliments, Bob! I appreciate them. Removing the line break in the alt attribute would certainly be worthwhile - I’m not sure that’s the choice I’ll make for fixing it, but the comment is well and duly noted.

I’ll confess that I may not have put as much attention as I could into the formatting of the code output - but your comments are welcome!

By: Bob

Bob — Fri, 28 Mar 2008 11:07:36 +0000

Hi Joe
To a PHP novice (learning from David Power’s books), I find your wee script relatively easy to understand. I’ve broken it up, thrown it across different CSS layouts, played with the thumb lists (I was/am a Lego kid-says it all maybe) I’m having some fun with it. It’s a nice script Joe.

I have a couple of tips, if I may be so bold.

I noticed that in IE (7 at least), the thumb alts (all but the last one) adds a line break (t’was driving me nuts :). In the titles.txt file, if one adds a Pipe after each line, it goes.

And for the aligned-markup purists, on line 98 of gallery.php (the thumbs list), by adding \n immediately after the tag, the markup/source for the thumb list straightens up nicely too.

Small as these ‘tips’ may be, I hope it’s of help to thy fellow novices. Again, nice script.

Best regards

By: Joe Dolson

Joe Dolson — Thu, 13 Mar 2008 01:46:16 +0000

Thanks, ah… “WeaponX” — - I’ll get that fixed shortly.

By: WeaponX

WeaponX — Wed, 12 Mar 2008 17:41:37 +0000

I was getting the following Error:

Notice: Undefined offset: 5 in C:\Program Files\Apache Group\Apache2\htdocs\gallery\includes\gallery.php on line 95

I noticed on line 94, in the for loop you have
for($i = 0; $i <= $result; $i++)

to resolve this issue you either have to remove the =, or use $result - 1

if you have 5 pictures the loop is running 6 times, when we only want it to run 5… hope this helps

By: Joe Dolson

Joe Dolson — Mon, 04 Feb 2008 17:35:02 +0000

Thanks, Aaron - I’ve fixed the sample gallery, and will replace the downloadable files very soon.

I appreciate that — - never noticed that bug!

By: Aaron

Aaron — Mon, 04 Feb 2008 15:55:21 +0000

Not sure if you’re still interested in this script, but there appears to be a small mistake in the gallery include. When the last picture in the list is being viewed, the bug causes the “previous” link to link to the page being viewed (the last picture). It’s written as:
case $suma: $prev = ($suma); $next = 0; break;
It should be ($suma - 1). You can see the problem on your example page.

By: Joe Dolson

Joe Dolson — Mon, 03 Dec 2007 04:27:58 +0000

No problem!

Nice job implementing the script!