Drupal has a nice autoloader (aka. registry), let's take advantage of it.

CommentFileSizeAuthor
#2 1272998_maestro_autoloader-2.patch18.22 KBAron Novak
#1 1272998_maestro_autoloader.patch18.23 KBAron Novak
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Aron Novak’s picture

Aron Novak
Primary language Hungarian
Location Hungary, Budapest
CreditAttribution: Aron Novak commented
Status: Active » Needs review
FileSize
1272998_maestro_autoloader.patch18.23 KB

Makes the code a bit smaller & simplier.

Aron Novak’s picture

Aron Novak
Primary language Hungarian
Location Hungary, Budapest
CreditAttribution: Aron Novak commented
FileSize
1272998_maestro_autoloader-2.patch18.22 KB

A message contained a not defined variable.

_randy’s picture

_randy CreditAttribution: _randy commented
Status: Needs review » Needs work

I'm not 100% sure on committing this yet as there are dependencies on the maestro engine version. By removing the explicit include statement for the engine means that we're unable to fire up the right engine version dynamically.

You can see this in the includes for the engine where {version} is used.