Generally when we have the require_once , it returns sites/all/modules/encrypt/,
In case, if we have the Docker/VM setup in the project environment, the Drupal codebase will reside inside the sub directory..
At this point of time..
If i run drush command with some operation on encrypt module, it would throw an warning message like Path not Found.
So, it better to have DRUPAL_ROOT included here..

-      require_once $info['file'];
+      require_once DRUPAL_ROOT . '/' . $info['file'];
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

heykarthikwithu created an issue. See original summary.

heykarthikwithu’s picture

Assigned: heykarthikwithu » Unassigned
Status: Active » Needs review
FileSize
574 bytes

added a patch for this

heykarthikwithu’s picture

was able to reproduce this warning is been thrown when the module folder is been placed in the sites/all/modules or sites/all/modules/contrib

rlhawk’s picture

Can you switch to using Encrypt 7.x-2.3? The 1.x branch has not been supported for quite some time.

heykarthikwithu’s picture

Status: Needs review » Closed (won't fix)

yup sure.. but their were some reasons, where we are still using 1.x..

any how, their is no reason to get this minor improvement in the unsupported branch..

we can ignore this :)