Find autoload

[veger]

My vendor/autoload.php is not in the root of my project (as the project contains other stuff besides the source code).

The improved script is able to find any vendor/autoload.php in the project, if it is not found in the root of the project.

Additionally, changed the error message when the autoloader has been found (but the class is not), making it less obnoxious.

[harikt]

Hi @veger ,

Thank you for the PR. I have a different idea.

How about the users create a vendor/autoload.php file which can return a loader having a findFile method.

So in your case it would be something like

<?php
return require '/path/to/autoloader.php';

I guess that will resolve some other issues like even if there is no composer support. And the code becoming light itself.

What do you think ?

[veger]

It is a nice idea to keep the code light!

But I am not allowed to add helper files, just to let tooling work, in my project root (company policy). So it is not a solution in my case. :(

[filaruina]

+1 on this one.
Searching is better than adding a helper file, since it may become a burden due to version control if you're not commiting it.
Also I think this is useful to make the package more "works out of the box". No need to do anything, just use it.

[harikt]

Sure. I will consider looking into this again and merge it. Currently sick
and on bed.

On 12-Nov-2016 17:03, "Filipe La Ruina" [email protected] wrote:

+1 on this one.
Searching is better than adding a helper file, since it may become a
burden due to version control if you're not commiting it.
Also I think this is useful to make the package more "works out of the
box". No need to do anything, just use it.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#7 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAHWhpXCIxby-qGH0It-R5wvsHKl0Rjkks5q9aP0gaJpZM4Jrtga
.

[harikt]

Hey,

I have a question where did this $autoloader is assigned ? Did you missed / accidentally removed https://github.com/harikt/php-hyperclick/pull/7/files#diff-5ef8e23f048e9259d4783d28d2407b5dL5

[veger]

Oops, you are right!

The default location should be used here, but was deleted by accident.

I have updated the PR

[harikt]

Thank you. And sorry for keeping this wait for long.

[harikt]

Released https://github.com/harikt/php-hyperclick/releases/tag/v1.2.0 .