Skip to content

PSR-4 compliance & class autoloading #15

@jameswilson

Description

@jameswilson
require_once __DIR__ . '/lib/EBSCODocument.php';

In order to follow modern PHP best practices, the Drupal 8 ebsco.module file and the various classes in the lib folder should be refactored to use PSR-4 autoloading.

A more complete solution however would be to consider extracting EBSCODocument and all the other classes into a separate repository, so they can be exposed to Packagist, and installed via a composer dependency in the Drupal module's composer.json this would mean that any PHP project (eg Symfony or Laravel) could benefit from using these classes.

See Drupal 8 Coding Standards for PSR-4 Namespaces and Autoloading

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions