Skip to content
This repository has been archived by the owner on Nov 26, 2017. It is now read-only.

Commit

Permalink
Setting JError to default to ignore since it is deprecated and the
Browse files Browse the repository at this point in the history
platform has no use for it.  We will need to have the CMS set things for
backward compatability.
  • Loading branch information
LouisLandry committed Oct 15, 2011
1 parent f992660 commit e82b513
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 13 deletions.
7 changes: 4 additions & 3 deletions libraries/joomla/error/error.php
Expand Up @@ -55,9 +55,10 @@ abstract class JError
protected static $levels = array(E_NOTICE => 'Notice', E_WARNING => 'Warning', E_ERROR => 'Error');

protected static $handlers = array(
E_NOTICE => array('mode' => 'message'),
E_WARNING => array('mode' => 'message'),
E_ERROR => array('mode' => 'callback', 'options' => array('JError', 'customErrorPage')));
E_NOTICE => array('mode' => 'ignore'),
E_WARNING => array('mode' => 'ignore'),
E_ERROR => array('mode' => 'ignore')
);

protected static $stack = array();

Expand Down
15 changes: 5 additions & 10 deletions tests/bootstrap.php
Expand Up @@ -77,20 +77,15 @@
require_once JPATH_TESTS . '/config.dist.php';
}

// Include helper class
require_once(dirname(__FILE__).'/includes/JUnitHelper.php');
// Import the platform.
require_once JPATH_PLATFORM . '/import.php';

// Include helper class.
require_once JPATH_TESTS . '/includes/JUnitHelper.php';

// Include the base test cases.
require_once JPATH_TESTS . '/includes/JoomlaTestCase.php';
require_once JPATH_TESTS . '/includes/JoomlaDatabaseTestCase.php';

// Import the platform.
require_once JPATH_PLATFORM . '/import.php';

// Exclude all of the tests and platform files from code coverage reports
PHP_CodeCoverage_Filter::getInstance()->addDirectoryToBlacklist(JPATH_TESTS);

// Set error handling.
JError::setErrorHandling(E_NOTICE, 'ignore');
JError::setErrorHandling(E_WARNING, 'ignore');
JError::setErrorHandling(E_ERROR, 'ignore');

0 comments on commit e82b513

Please sign in to comment.