Including files for their class definition is so Drupal 7. Let's use the PSR-0 class autoloader.

CommentFileSizeAuthor
#3 1661104-3.patch4.85 KBamateescu
psr0.patch10.31 KBRobLoach
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

msonnabaum’s picture

msonnabaum CreditAttribution: msonnabaum commented

I appreciate the work you've done here, but I don't really want to change much in this branch aside from bug fixes.

We need to start a new branch that brings in https://github.com/msonnabaum/xhproflib as a dependency via composer. I'd be very appreciative if you were into working on that :)

RobLoach’s picture

RobLoach
he/him
CreditAttribution: RobLoach commented

Oh! Much nicer than what's in the module!!! :-)

amateescu’s picture

amateescu CreditAttribution: amateescu commented
Category: task » bug
FileSize
1661104-3.patch4.85 KB

Since the registry is now gone from D8, I think this is actually a bug for the current 8.x-1.x branch. Here's an updated patch :)

amateescu’s picture

amateescu CreditAttribution: amateescu commented
Status: Needs review » Fixed

Committed to 8.x-1.x.

http://drupalcode.org/project/XHProf.git/commit/3d1df25

Status: Fixed » Closed (fixed)

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