Custom Behaviors

Question

Custom Behaviors

odiel Feb '13

Created Feb '13	Last Reply Dec '15	Replies 3	Views 1853	Votes 2

odiel 12.1k

Feb '13

The example here:

https://docs.phalcon.io/en/latest/reference/models.html#creating-your-own-behaviors

Says that the $models should be defined as an instance of ModelInterface, I think this is not valid any more for Phalcon version 0.9.1, could you guys check this?

I'm getting this error all the time:

Fatal error: Declaration of Models\Behaviors\Timestampable::notify() must be compatible with that of Phalcon\Mvc\Model\BehaviorInterface::notify() in ..../app/models/Behaviors/Timestampable.php on line 10

Here is the code:

<?php

namespace Models\Behaviors;

use \Phalcon\Mvc\ModelInterface, \Phalcon\Mvc\Model\Behavior, \Phalcon\Mvc\Model\BehaviorInterface;

class Timestampable extends Behavior implements BehaviorInterface {

public function notify($eventType, ModelInterface $model)
{

    var_dump($eventType);die;

    if ($eventType == 'beforeCreate') {
        var_dump($model);die;

        $model->created = date('Y-m-d');
    }
}

}

as soon as I remove ModelInterface type, the error disappear.

2

arantir · Answer 1 · 2013-02-25T15:29:50-07:00

It's a typo. Look into the https://docs.phalcon.io/en/latest/api/Phalcon_Mvc_Model_BehaviorInterface.html The second argument should be an instance of Phalcon\Mvc\Model\Behavior, not Phalcon\Mvc\ModelInterface.

odiel · Answer 2 · 2013-02-25T15:51:11-07:00

I think you are wrong, the second argument is the model, and \Phalcon\Mvc\Model (https://docs.phalcon.io/en/latest/api/Phalcon_Mvc_Model.html) do not inherit or implement Phalcon\Mvc\Model\Behavior, maybe the second argument should not have an specific data type, since in my code I'm putting all the models under namespaces.

arantir · Answer 3 · 2013-02-25T16:14:26-07:00

If you implement interface, you must create functions with the same signatures. In your example you implement Phalcon\Mvc\ModelInterface. As you can see, the signature of notify() in Phalcon\Mvc\ModelInterface is:

public notify (string $type, Phalcon\Mvc\Model\Behavior $model)

but yours is

public notify($type, Phalcon\Mvc\ModelInterface $model)

which is not equivalent to needed. And the error says you exactly same as I do.

Docs are written by hand and couldn't be foolproof because of human factor, but API Indice displays actual right structure, because of automatically formed from it. So you should guided by API Indice at first.

odiel · Answer 4 · 2013-02-26T11:32:37-07:00

I get what you say, and I'm agree with you, but that does not work for me.

Here is the code again, and please correct me if the code is still wrong.

<?php

namespace Models\Behaviors;

use \Phalcon\Mvc\Model\Behavior,
    \Phalcon\Mvc\Model\BehaviorInterface;

class Timestampable extends Behavior implements BehaviorInterface
{

    public function notify($eventType, \Phalcon\Mvc\Model\Behavior $model)
    {
        if ($eventType == 'beforeValidation') {
            $model->created = date('Y-m-d');
        }
    }

}

and I still getting the signature error:

Fatal error: Declaration of Models\Behaviors\Timestampable::notify() must be compatible with that of Phalcon\Mvc\Model\BehaviorInterface::notify() in ......./app/models/Behaviors/Timestampable.php on line 9

as soon as I remove the type of $model, everything works.

gollomer · Answer 5 · 2015-12-24T06:40:46-07:00

This is working for me in Phalcon 2.0.8

<?php
use Phalcon\Mvc\ModelInterface;
use Phalcon\Mvc\Model\BehaviorInterface;
use Phalcon\Mvc\Model\Behavior;

class aclBehavior extends Behavior implements BehaviorInterface
{
    /**
     * 
     * @param string $eventType
     * @param Phalcon\Mvc\ModelInterface $model
     * @return mixed
     */
    public function notify($eventType, Phalcon\Mvc\ModelInterface $model)
    {
        if (method_exists($this, $eventType))
        {
            $this->$eventType($model);
        }
    }
}