Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implemented: Single, Global eztemplateautoload.php Include #4

Conversation

@brookinsconsulting
Copy link
Contributor

@brookinsconsulting brookinsconsulting commented Sep 6, 2014

With the acceptance of our previous pull request #2 it no longer makes sense to force developers and users make static instances of eztemplateautoload.php operator autoload files since now the extension's provided autoload/eztemplateautoload.php file is now both dynamic and ini settings based.

Once merged these changes allow for a single global ezetemplateautoload.php file (already in place in the extension's autoload directory.

To create a custom operator you no longer need a custom class file + eztemplateautoload.php file + ini settings override; you only need your custom class file + ini settings override.

This is a big improvement to the current state of affairs. We hope you'll agree and accept this pull request.

These changes make using the OWSimpleOperator extension to create a custom operator class stored in a separate extension's class directory and activated via global ini settings override both painless and eZ to do!

Please let us know what you think!

Cheers,
Brookins Consulting

@Dryobs
Copy link
Contributor

@Dryobs Dryobs commented Sep 8, 2014

Thanks, it's a great improvement.
We will merge it.

Cheers,
Simon Boyer from OpenWide

Dryobs pushed a commit that referenced this pull request Sep 8, 2014
…mplateautoload

Implemented: Single, Global eztemplateautoload.php Include
@Dryobs Dryobs merged commit ecd3aad into Open-Wide:master Sep 8, 2014
@brookinsconsulting brookinsconsulting deleted the brookinsconsulting:single_instance_of_eztemplateautoload branch Sep 11, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants