Good question.. I was thinking that since I didn't assign the fopen to
anything, it would take care of that immediately.  I've seen other
people do this, but your guess is as good as mine as to if it closes
it when nothing references the filehandle, or only at script end.
--Bob

On 9/4/05, David Zülke <[EMAIL PROTECTED]> wrote:
> S houldn't we fclose() the file after opening it?
> 
> - David
> 
> 
> > -----Original Message-----
> > From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] On
> > Behalf Of Bob Zoller
> > Sent: Sunday, September 04, 2005 11:08 PM
> > To: Agavi Development
> > Subject: Re: [agavi-dev] autoload.ini and include_path
> >
> > FYI, just committed this to trunk.  From the commit message:
> >
> > *BREAKAGE WARNING*
> > this means is that you must have a full path specified in
> > autoload.ini.  "lib/blah.class.php" (to reference
> > "/path/to/mywebapp/lib/blah.class.php") would no longer work - unless
> > /path/to/mywebapp is in your include_path of course :)  This should
> > only be a problem for user-provided autoload.ini entries, as the
> > "stock" ones all reference the %AG_APP_DIR% constant to provide a full
> > path.
> >
> > --Bob
> >
> > On 9/3/05, Bob Zoller <[EMAIL PROTECTED]> wrote:
> > > I was seeing the same thing yesterday - Since
> > > ConfigHandler::replacePath() will prepend the AG_WEBAPP_DIR if it sees
> > > a relative path specified, it will destroy autoload.ini entries like
> > > the one below.  I had some ideas on how to fix this, in short:
> > >
> > >  * remove the call to replacePath()... all this means is that you have
> > > to have %AG_WEBAPP_DIR% in your path value in autoload.ini.
> > > "lib/blah.class.php" (to reference
> > > "/path/to/mywebapp/lib/blah.class.php" would no longer work - unless
> > > /path/to/mywebapp is in your include_path of course :)
> > >
> > >  * replace the call to is_readable() (which is the spot that actually
> > > throws that Exception below) to a @fopen($file, 'r', true) - of which
> > > the 3rd param will search the include_path.
> > >
> > > If there are no objections and it passes the sniff test, I can go
> > > ahead and make those changes.
> > > --Bob
> > >
> > > On 9/1/05, David Zülke <[EMAIL PROTECTED]> wrote:
> > > > config.php is loaded before the framework itself is loaded and
> > started.
> > > > Check your include path setup again.
> > > >
> > > > - David
> > > >
> > > >
> > > > > -----Original Message-----
> > > > > From: [EMAIL PROTECTED] [mailto:agavi-dev-
> > [EMAIL PROTECTED] On
> > > > > Behalf Of J. Erik Heinz
> > > > > Sent: Thursday, September 01, 2005 3:26 PM
> > > > > To: agavi-dev mailinglist
> > > > > Subject: [agavi-dev] autoload.ini and include_path
> > > > >
> > > > > Hi,
> > > > >
> > > > > I have some problems with autolaod.ini and config.php
> > > > > at the moment i try to get propelintegration to work.
> > > > >
> > > > > i spezified my include_path in the config.php. When i run my project
> > > > > from the browser, i get the Warning:
> > > > > Warning: main(propel/util/BasePeer.php) [function.main]: failed to
> > > > > open stream: No such file or directory in
> > > > >
> > /usr/local/workspace/projects/sandbox/webapp/lib/sandbox/om/BaseUsersPeer.
> > > > > php
> > > > > on line 3])
> > > > >
> > > > > this 'propel/util/BasePeer.php' should be in the include_path. So i
> > > > > think the config.php will be loaded after the autoload.ini has been
> > > > > loaded.
> > > > >
> > > > > Where du I have to put the include_path information, that they are
> > > > > available as well at the autoload.ini?
> > > > >
> > > > > Regards Erik
> > > > >
> > > > >
> > > > > --
> > > > > J. Erik Heinz
> > > > > Keyboard-samuraing in process
> > > > > _______________________________________________
> > > > > agavi-dev mailing list
> > > > > [email protected]
> > > > > http://labworkz.com/cgi-bin/mailman/listinfo/agavi-dev
> > > >
> > > >
> > > >
> > > > _______________________________________________
> > > > agavi-dev mailing list
> > > > [email protected]
> > > > http://labworkz.com/cgi-bin/mailman/listinfo/agavi-dev
> > > >
> > >
> > _______________________________________________
> > agavi-dev mailing list
> > [email protected]
> > http://labworkz.com/cgi-bin/mailman/listinfo/agavi-dev
> 
> 
> 
> _______________________________________________
> agavi-dev mailing list
> [email protected]
> http://labworkz.com/cgi-bin/mailman/listinfo/agavi-dev
>
_______________________________________________
agavi-dev mailing list
[email protected]
http://labworkz.com/cgi-bin/mailman/listinfo/agavi-dev

Reply via email to