Comodojo\Extender\Traits\TaskErrorHandlerTrait

Complexity

Total Complexity

3

Size/Duplication

Total Lines	21
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	installErrorHandler()	0	3	1
A	restoreErrorHandler()	0	3	1
A	customErrorHandler()	0	7	1

<?php namespace Comodojo\Extender\Traits;

use \Comodojo\Foundation\Utils\ErrorLevelConverter;

/**
 * @package     Comodojo Extender
 * @author      Marco Giovinazzi <marco.giovinazzi@comodojo.org>
 * @license     MIT
 *
 * LICENSE:
 *
 * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
 * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
 * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
 * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
 * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
 * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
 * THE SOFTWARE.
 */

trait TaskErrorHandlerTrait {

    public function installErrorHandler() {

        set_error_handler([$this, 'customErrorHandler']);

    }

    public function restoreErrorHandler() {

        restore_error_handler();

    }

    public function customErrorHandler($errno, $errstr, $errfile, $errline) {

        $error = ErrorLevelConverter::convert($errno);

        $this->getLogger()->error("Unhandled $error ($errno): $errstr [in $errfile line $errline]");

It seems like getLogger() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        return true;

    }

}