You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
in Service.php
require_once('Apache/Solr/Document.php');
require_once('Apache/Solr/Response.php');
It's much easier to use the library if it's something like:
require_once(dirname(__FILE__) . '/Document.php');
require_once(dirname(__FILE__) . '/Response.php');
Original issue reported on code.google.com by pwola...@gmail.com on 27 Jan 2009 at 7:02
The text was updated successfully, but these errors were encountered:
On the one hand, I'm against this because it's not in the preferred style and
standard of Zend Framework or
PEAR. Libraries like this are intended to be on the include path so that
through their Class naming
conventions and file structures they can take advantage of the __autoload magic
function for dependency
injection.
On the other hand, I AM already doing require_once's to save the 10-20ms or so
of overhead using
__autoload since my Service class will directly use both other classes in most
of its use cases. In addition I
have fielded a number of email complaining about my "broken" requires.
So, for the sake of convenience to the user I'm willing to break with the
preferred conventions. I'll commit a
change shortly
Original comment by donovan....@gmail.com on 28 Jan 2009 at 4:20
Original issue reported on code.google.com by
pwola...@gmail.com
on 27 Jan 2009 at 7:02The text was updated successfully, but these errors were encountered: