At the top of relation.module we do a module_exists outside of a function body. This is referenced here and in my own experience this can cause a lot of issues with priming caches too early in the bootstrap and causing unrelated issues.

CommentFileSizeAuthor
#2 2549837-api-function-outside-of-body.patch373 bytessam152

Comments

Sam152 created an issue. See original summary.

sam152’s picture

sam152 commented
Status: Active » Needs review
StatusFileSize
new 2549837-api-function-outside-of-body.patch373 bytes

I can't see a problem with including the ctools file even if ctools is disabled.

benjy’s picture

benjy commented
Status: Needs review » Reviewed & tested by the community

Looks good.

  • mikran committed 54d8a12 on 7.x-1.x authored by Sam152 
    Issue #2549837 by Sam152: Code should not call Drupal API functions...
mikran’s picture

mikran commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.