Solution: top 10 albums and last updates including sub-album

June 11 2004 -
Edit: fixed the URL where you can find my gallery help pages, as I moved from PHP-Nuke to Geeklog this weekend...

=========================================

March 1 2004-
I've still been getting quite a few requests for help with these scripts. I realize that this is not the easiest thing to do if you are not familiar with PHP. So I've decided to put together some >> help pages << on these two features (last X updates and top X albums).

I have also updated the code and improved the scripts some, as well as made it easier to customize. For example, it is now easy to set the number of albums to be displayed. You can also display the top X albums with a rank (like on >> my web site <<).

Unless you already have it working and you are happy with it, please download the new versions.

I will leave the info below so that this thread still makes sense. But please check the more detailed instructions on my >> help pages <<.

=========================================

Feb 23, 2004 - I updated this post to make it a little easier to read. Since the boards update the code sections was just too messy looking

I know some of you said you were working on this, so I didn't want to keep bugging you. But since I just reorganized my gallery into only 3 top level albums (for now), it was urgent for me to try to fix the top 10 / last updated scripts so that they include sub-albums.

Check out >> my web site <<to see the new top 10 script in action.

I haven't updated the last-updated one yet, but here's what you need to do to make top10albums.php get a list of all albums, not only the top-level albums:

1.) Download the script you want >> here <<.

Credit still goes to pixelchimp, all I did was change some function calls to call new modified functions (see next step) and take out the double isRoot() check.

March 1, 2004 - By now I've made more changes than that...

2.) Add 3 custom functions to AlbumDB.php. They are modified copies of functions that already exist and that are used by top10.

You can add these anywhere within AlbumDB.php in the functions section (except in the middle of another function, of course). One way is to add all custom functions at the very bottom, just before the last

}

?>

Add these 3 functions:

// --------- BEGIN CUSTOMIZATIONS --------- //

        /* 
          20040218: Added numAllAlbums, getAllAlbum, and getAllVisibleAlbums to also scan sub-albums (not just root albums)
          to be used by top 10, most recently updated, etc mods 
         */
         
        function numAllAlbums($user) {
                return sizeof($this->getAllVisibleAlbums($user));
        }

        function getAllAlbum($user, $index) {
                global $gallery;
                $list = $this->getAllVisibleAlbums($user);
                if (!$list[$index-1]->transient->photosloaded) {
                        $list[$index-1]->loadPhotos($gallery->app->albumDir . "/" . $list[$index-1]->fields["name"]);
                }
                return $list[$index-1];
        }

        function getAllVisibleAlbums($user) {
                global $gallery;
                $list = array();
                foreach ($this->albumList as $album) {
                        if ($user->canReadAlbum($album)) {
                                array_push($list, $album);
                        }
                }
                return $list;
        }

// --------- END CUSTOMIZATIONS --------- //

That's it! Have fun!

~Eva

This is great! Thank you for taking the time!

With that said..I'm having one problem ...

When I add it to my page I get an error "Object Expected" on line 29.

The page works fine without the include. ==> http://www.Picolio.com/

The include works fine without the page ==> http://www.Picolio.com/test.php

But when I put the include into the page I get the error on page ==> http://www.Picolio.com/indexTest.php
(you can see the little error message at the bottom of IE5.5 especially if you mouse over the links in the 'last updated' area.

When viewing the code itself line 29 falls in the IFRAMEs for the random images...

<IFRAME id="blockrandom"
onLoad="iFrameHeight();"
src="http://www.picolio.com/Gallery/block-random.php?&amp;target=_top"
width="180"
height="130" align=center valign=middle scrolling=no frameborder=0>
[This feature requires IFRAME.]
</IFRAME>

Any ideas?

Thanks again!

Deb

Hmmm, I'm not versed enough with IFRAMEs to understand it but I removed the following line from both of the IFRAMEs on top of the page for the random images

onLoad="iFrameHeight();"

Now it all works :razz: http://www.Picolio.com/ <<== shows it

Thanks very much for taking the time to provide this enhanced feature!

Deb

I was just looking at your pictures (Nikki and MacDonalds is a crack up!), and noticed you actually seem to have that error on every page. The random image iFrames in the header still have this line in them:

onLoad="iFrameHeight();"

But when I did a view source I couldn't find the function defined in your <head> tags.

LOL - no problem, just thought I'd mention it, as I was worried how the last-updated block could have caused that error. :grin:

Nebular -

I get that same problem when I'm logged into the gallery as admin. I followed your link and it looks fine for me, so I think this is the same problem. Try logging off and look at it again.

This is some problem with the original code, I never tried to figure it out once I realized it looked fine to everyone else. I don't have any users with login IDs on my gallery, so I'm not sure if it also affects non-admin users.

Also, as a tip, it also breaks if you have a "/" anywhere in your album title, because the cache file is /-delimited.

First of all, thanks for your coding, ppmnt, it's a good improvement for a lot of galleries, including mine :smile:

I see some people want it to work with an "include" instead of an "iframe", though. I've had to do this in my gallery. It's as simple as commenting the following lines:

<!-- BBCode Start --><TABLE BORDER=0 ALIGN=CENTER WIDTH=85%><TR><TD><font class="pn-sub">Code:</font><HR></TD></TR><TR><TD><FONT class="pn-sub"><PRE>require($GALLERY_BASEDIR . "init.php");
readfile("header.html");</TD></TR></TABLE><!-- BBCode End -->

Also, it can be useful editing the last lines of the PHP page, so it integrates better in the gallery web page. Something like this:

<!-- BBCode Start --><TABLE BORDER=0 ALIGN=CENTER WIDTH=85%><TR><TD><font class="pn-sub">Code:</font><HR></TD></TR><TR><TD><FONT class="pn-sub"><PRE>[...]
$numAllAlbums = $albumDB->numAllAlbums($gallery->user);
$numPhotos = $albumDB->numPhotos($gallery->user);

?>

<span class="title">Last updates</span><br>

<?

// for loop reads the data from the arrays and renders it on screen

for ($i = 0; $i <= 4; $i++) {
$albumURL = makeGalleryUrl($cacheName[$i]);
if (strcmp($cacheRealName[$i],"")) {
echo ("&amp;nbsp&amp;nbsp<a href=" . $albumURL . ">" . $cacheRealName[$i] . "</a> (" . $cacheDate[$i] . ")<br>");
}
}

}

echo ("<br>");

?>
</table></TD></TR></TABLE><!-- BBCode End -->

You can see it working (in Spanish :roll: ) in <!-- BBCode Start --><A HREF="http://ganso.org" TARGET="_blank">my web page</A><!-- BBCode End -->.

You're getting this error because your Gallery doesn't have mod_rewrite enabled. This script assumes you have mod_rewrite enabled (not that good). You could solve it by rewriting the part of the script where URLs are created to use the non-mod_rewrite URLS, e.g. inserting "view_album.php?set_albumName=" in the code.

Thanks, TheGoose, I'll try that out. Might look a little better loading over a slow connection.

Oops !! I tried to keep it just in the main gallery page, and I though view_album.php was the right file. I'll try to correct it later at home.

I don't use the top10 albums feature, as I have less than 10 albums :wink:
Again, I'll try to look at this afterwards. Anyway, as the PHP code is included inside the original code, which already inclused init.php, it should work, at last when included only in the main page (as I tried to do :roll: )

thanks ppmnt for fixing this issue for me, i was trying to find time to get around to making these changes to the scripts, as i don't use sub-albums on my site it wasn't a priority for me unfortunatly.

good work though, pat on back well deserved!

- pixelchimp
[<!-- BBCode Start --><A HREF="http://www.chaoslibrary.com" TARGET="_blank">www.chaoslibrary.com</A><!-- BBCode End -->]

Hi -- This is really great! I am pretty new to this but I was having a little bit of trouble making the links work within my phpnuke-powered site. Anyway, after a little tweaking, I finally got it to work and I also added a photograph (although that had to be manually selected.). If anyone is interested, I will make the revised code available. If you want to see it, look at http://www.kalorama.net. Anyway, thanks ppmnt and pixelchimp for a very useful script.

discobug is back up now, try again: <!-- BBCode Start --><A HREF="http://www.discobug.com/last-updates.phps" TARGET="_blank">>>last-updates.php <<</A><!-- BBCode End --> and <!-- BBCode Start --><A HREF="http://www.discobug.com/top10albums.phps" TARGET="_blank"> >> top10albums.php <<</A><!-- BBCode End -->

Well, I've been with this host for 2 years, this is the first time Apache has gone down. That's a pretty good record, I guess. :smile:

~Eva

Works perfectly in tests. I don't know if anyone else does this, but I use / in my dates on the album names like:

NYC - 5/5/99 and save/read cache functions use / as the separator for each item so the data getting pulled from the cache cut off..

I deleted my cache, changed the lines in the code to use | (since I never use pipe ever) for the save cache function to be (make sure you get all of them):

fwrite($fd, "$nameArray[$currentKey]|$albumName[$currentKey]|$clickArray[$currentKey]n");

and in your read cache function, look for the explode and change it to whatever character you're using, like:

explode("|", $line);

....and no, don't tell me to write my title dates differently, it's easier to do this, then rename all my albums :wink:

hello all!

i'm having a problem with the top 10 and last updates addin.... when i try to use the php i get this error

Warning: readfile(header.html) [function.readfile]: failed to create stream: No such file or directory in /home/drazin/public_html/gallery/last-updates.php on line 87
1917 photos in 47 albums:

and

Warning: readfile(header.html) [function.readfile]: failed to create stream: No such file or directory in /home/drazin/public_html/gallery/top10albums.php on line 72

any ideas?

edit: i took out the line of code that refers to header.html and now it sorta works but it donsent have any info like how many clicks....or when it was updated.....plz help

nice hacks, cheers

where about exactly does that code go?

I also had the admin seing an odd album name with a javascript edit when I used the last updated mod.

I added the following when getting the album name...

$albumName[$i] = strip_tags(editField($gallery->album, "title", $edit));
$albumName[$i] = str_replace("[edit title]", "", $albumName[$i]);

Which just cleans it up in case the admin happens to be the person who renews the cache.

This thread should help you:

<!-- BBCode Start --><A HREF="http://gallery.menalto.com/modules.php?op=modload&amp;name=phpBB_14&amp;file=index&amp;action=viewtopic&amp;topic=2317&amp;start=0" TARGET="_blank">http://gallery.menalto.com/modules.php?op=modload&amp;name=phpBB_14&amp;file=index&amp;action=viewtopic&amp;topic=2317&amp;start=0</A><!-- BBCode End -->

I have been using this script for the past week and just took it out, I was looking at my httpd processes (1.3.27) and noticed that they were all chewing up a ton of memory.

I did an 'apachectl graceful' and they go back to normal - but as soon as I refresh my top10.php they jump up to about 60mb/process and stay there.

I can refresh my main page without the top10.php include() and the processes stay at the normal 15mb/process.

So - I suspect there is something wrong with the code - especially since it causes the mem jump even when its reading from the cache file (ie not scanning the albums).

http://jadz.com/last10.phps is the code I'm using (with the added functions at the beginning of this thread).

Sorry it took so long for you to get a response, I didn't see your question right away.

Just comment out the line where it calls header.html:

// write common header elements - text colour etc
readfile("header.html");

It's line 90 for me. Put "//" in front of the second line.

You seem to have another problem as well:

Warning: fopen("/home/virtual/site147/fst/var/www/html/gallery/albums/last-updated.cache", "w") - Permission denied in /home/virtual/site147/fst/var/www/html/gallery/platform/fs_unix.php on line 53

Make sure the cache files are writable by the ID that runs your web server (usually www or nobody).

hey everybody, i have 2 quick questions that i haven't been able to find a definitive answer for:

1. http://www.anothersaab.com/gallery/top10albums.php

- what is causing just "Visits" to appear. what am I missing?

2. http://www.anothersaab.com/gallery/last-updates.php

- why are my dates showing up incorrectly. the order is correct, but 3 of the 5show up right and 2 show just a number and 2 have long titles and they are getting chopped.

thoughts?

yes, i did get the last-updates corrected. the problem with that was simple. 2 of the albums contained dates with slashes (01/01/2001) and the slashes caused the album name to be truncated and the last update date to read just a number.

here is my top10albums.php: http://www.anothersaab.com/gallery/top10albums.phps

thanks for the dog compliment. as a matter of fact, i just got home from walking them to the coffee shop and i saw your post. thanks for the compliment *and* helping me out.

Tim

cyberlok -

Interesting layout you have at that link. Looks familiar somehow (especially when I do a view source!)

Anyway. To trouble-shoot your problem, go to last-updates.php on your site in a browser (> here <). Then do a "view source" on it. You'll see that it's a bit messed up:

<link rel="stylesheet" type="text/css" href="http://www.cyberlok.org/cyberlok/gallery/css/standalone_style.css.default">

927</b> photos in <b>68</b> albums:
<tr><td><a href=view_album.php?set_albumName=album60 target='_blank'>Minnesota Friends</a><br>  last updated: May 13, 2003
</td></tr><tr><td><a href=view_album.php?set_albumName=album61 target='_blank'>The Kids</a><br>  last updated: May 13, 2003
</td></tr><tr><td><a href=view_album.php?set_albumName=album38 target='_blank'>Photographs</a><br>  last updated: May 11, 2003
</td></tr><tr><td><a href=view_album.php?set_albumName=Family target='_blank'>Family</a><br>  last updated: May 11, 2003
</td></tr><tr><td><a href=view_album.php?set_albumName=album58 target='_blank'>Pictures of Sally</a><br>  last updated: May 11, 2003
</td></tr></table> 

Your stylesheet reference is not inside a <head> tag. It starts table rows but there is no <table> tag. However, at the end there is a close table tag.

When I do the same thing at my site, a view source shows this:

<head>
<link rel="stylesheet" type="text/css" href="/css/gray-purple.css">
</head>
     <b>1997</b> photos in <b>112</b> albums:
     <p>

<table>
<tr><td><a href=http://www.discobug.com/gallery/bday_seaplane_05-02 target='_top'>Sea plane flight over the Bay Area, 5-02</a>
<br><font size=1>&&updated: May 17, 2003
</font size></td></tr>
<tr><td><a href=http://www.discobug.com/gallery/mama_visit_12-02 target='_top'>My mom visits, 12-02</a>
<br><font size=1>&&updated: May 17, 2003
</font size></td></tr>
<tr><td><a href=http://www.discobug.com/gallery/muir_woods_03-02 target='_top'>Muir Woods, 3-02</a>
<br><font size=1>&&updated: May 17, 2003
</font size></td></tr>
<tr><td><a href=http://www.discobug.com/gallery/mike_bday_2002 target='_top'>Mike's birthday, 2-02</a>
<br><font size=1>&&updated: May 17, 2003
</font size></td></tr>
<tr><td><a href=http://www.discobug.com/gallery/2002 target='_top'>2002</a>
<br><font size=1>&&updated: May 17, 2003
</font size></td></tr>
 -- etc --
</table>

It looks like maybe at some point you tried to change the way it is displayed and some tags got screwed up.

This is how the for loop at the end of last-updates.php looks for me:

<table>
<?
   
// for loop reads the data from the arrays and renders it on screen

    for ($i = 0; $i <= 9; $i++) {

         $albumURL = makeGalleryUrl($cacheName[$i]);
         echo ("<tr><td><a href=" . $albumURL . " target='_top'>" . $cacheRealName[$i] . "</a>\n");
         echo ("<br><font size=1>&&updated: " . $cacheDate[$i] .  "</font size></td></tr>\n");
    }

}

?>
</table>

Looks like you need to tweak yours some to make sure you open and close all the tags you need...

Tip: when something goes wrong, a view source usually helps trouble-shoot.

~Eva

Just wanted to say I updated to your version (with some minor modifications so it fits my site) and its super, thanks for asking. I was hoping something like this would appear and it did. Glad you could do the coding, cause asside from minor cosmetic tweaks I am lost in gallery code

If anyone wants to see how it was applied, http://fryfrog.com/ on the left side

Kyliedog,

I was moving and didn't see your post until now. I apologize. It should be as simple as changing this:

// Check the cache file to see if it's up to date
$rebuild = 1;
if (fs_file_exists(CACHE_FILE)) {
    $stat = fs_stat(CACHE_FILE);
    $mtime = $stat[9];
    if (time() - $mtime < CACHE_EXPIRED) {
        $rebuild = 0;
    }
}

to this:

// Check the cache file to see if it's up to date
// and if the viewer is logged in as a Gallery user
$rebuild = 1;
if (fs_file_exists(CACHE_FILE)) {
    $stat = fs_stat(CACHE_FILE);
    $mtime = $stat[9];
    if ((time() - $mtime < CACHE_EXPIRED) && ( $gallery->user->isLoggedIn() == True )) {
        $rebuild = 0;
    }
}

This would make it so that the cache is only rebuild if the user viewing the page is logged in. I haven't tested this yet, but it should work fine. Is that what you wanted?

~Eva