tommcfarlin / simple-autoloader-for-wordpress Goto Github PK

Getting Started states:

Add include_once 'lib/autoload.php' to your main plugin file.

but Adding The Autoloader states

Then, at the top of your plugin file add the following: require_once 'lib/autoload.php';

So is include_once or require_once recommended?

https://packagist.org/packages/szepeviktor/wordpress-autoloaded

@tommcfarlin This is my first try on commenting out require-s in core and autoloading classes by Composer. Still implemented in a shell.

What do you think?

Maybe I need to do some more homework but I was under the impression that once you register an autoloader all class requests will attempt to be processed. Yet, currently, there are no tests for (more of less) "is this a class we know about or not?"

For example, in the autoloader mentioned in the other issue (by @carlalexander) on line 26 he checks for:

if (0 !== strpos($class, 'CommentIQ')) {
return
}

https://github.com/postmatic/elevated-comments/blob/master/src/CommentIQ/Autoloader.php

Once you have more than one autoloader is something like this necessary?

On the same note, prior to doing the require wouldn't it be wise to check that the file exists? Just to be safe?

Good stuff either way. Thanks for sharing.

I'm a little struggling with namespaces and autoloaders, is it possible to get a demo plugin in the docs?

We've implemented a slightly modified version of this autoloader in a theme and needed interface support.

Incoming PR leverages a naming convention (I know, ick) that supports both prefixing and suffixing depending on developer preference, Interface_Foo or Foo_Interface.

Posting here by your request:

I'm thinking through a plugin namespace/directory layout using your new autoloader.

Typically I'd have my bootstrap file load the main plugin class, and the admin plugin class (in the admin context).

Any thoughts on where you'd put the main plugin class?

It's obvious for things like Models and Controllers, and maybe this is indicative that I'm putting too much on a God-class but I'd love your ideas.

Thanks in advance!

What is the min PHP requiement?

https://tommcfarlin.com/namespaces-and-autoloading-2017/ actually says,

Note that this code is not checking to see if the file exists. Though I recommend it, it’s been left out for the sake of length and because, in our example, we know the file does exist.

:)

I didn't submit a PR because your article says there are a few options for what to do if it doesn't exist and I didn't want to waste time writing one if you preferred another.

Not a complaint, just trying to learn from you. What's the value here instead of a PSR autoloader ( https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-4-autoloader-examples.md ) ?