[pLog-svn] r4171 - plugins/branches/lifetype-1.1/gallery

[Oscar Renalias]

It probably is needed by most of the other plugins but you don't
notice because as soon as one of them loads the class, it becomes
automatically available to all other plugins.

But if you test plugins one by one (install, test, remove), then
you're probably going to find a few more cases like this. It probably
is just a matter of going through all of them, check if they require
the Menu class and if they do, add the include at the top.

On 10/24/06, Reto Hugi <plog at hugi.to> wrote:
> On 23.10.2006 12:25, oscar at devel.lifetype.net wrote:
> > Author: oscar
> > Date: 2006-10-23 10:25:02 +0000 (Mon, 23 Oct 2006)
> > New Revision: 4171
> >
> > Modified:
> >    plugins/branches/lifetype-1.1/gallery/plugingallery.class.php
> > Log:
> > Fixed. I will upload it later.
> >
> >
> > Modified: plugins/branches/lifetype-1.1/gallery/plugingallery.class.php
> =================================================================== ---
> plugins/branches/lifetype-1.1/gallery/plugingallery.class.php   2006-10-23
> 10:24:22 UTC (rev 4170)
> > +++
> plugins/branches/lifetype-1.1/gallery/plugingallery.class.php   2006-10-23
> 10:25:02 UTC (rev 4171)
> > @@ -1,6 +1,7 @@
> >  <?php
> >  include_once( PLOG_CLASS_PATH."class/plugin/pluginbase.class.php" );
> include_once(
> PLOG_CLASS_PATH."plugins/gallery/class/gallery/galleryhelper.class.php" );
> > +include_once( PLOG_CLASS_PATH."class/template/menu/menu.class.php" );
> >
>
> Hi Oscar,
>
> I'm upgrading several sites, including plugins (all ported ones like
> authimage etc.). While most upgrades where no problem, I got fatal errors
> with one installation because the menu.class.php wasn't included. I
> removed everything and started the whole update process (including
> removing the old files and copying 1.1.1 again). This time everything
> worked. I couldn't reproduce the problem.
>
> But now, as you're fixing the gallery plugin by including menu.class.php
> - can you elaborate on this? Why is it needed for the gallery plugin, but
> not for most of the other plugins?
>
> thanks!
> reto
>
>
> _______________________________________________
> pLog-svn mailing list
> pLog-svn at devel.lifetype.net
> http://devel.lifetype.net/mailman/listinfo/plog-svn
>