Revert "Removing unneeded App::uses() call" as it causes ControllerTest to fail. #937

Merged
merged 1 commit into from Nov 3, 2012

3 participants

@bar
bar commented Nov 3, 2012

This reverts commit 0fb4d1d.

@markstory markstory merged commit ba8f278 into cakephp:master Nov 3, 2012
@ADmad
CakePHP member

The ErrorHandler does not have any direct reference to AppController so I don't see why it should have the App::uses() call for it. If there are still problems the App::uses() statement should be added in required file.

@bar
bar commented Nov 3, 2012

@ADmad I'll try to isolate the required file.

@ADmad ADmad added a commit that referenced this pull request Nov 3, 2012
@ADmad ADmad Revert "Merge pull request #937 from bar/master-controlelr-test"
This reverts commit ba8f278, reversing
changes made to 4c6c3b0.
1476ccb
@bar
bar commented Nov 3, 2012

For documentation, continued here cc6b699

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment