Zend_Db_Table_Abstract::_primary returns array?

Question

So I have defined a model like this:

class Model extends Zend_Db_Table_Abstract
{
    $_primary = 'modelID';


    /**
     *
     * @param mixed $primaryKey
     * @return int 
     */
    public function delete($primaryKey)
    {
        $where = $this->getAdapter()->quoteInto($this->_primary.' = ?', $primaryKey);
        return parent::delete($where);
    }
}

When calling the delete method, I get a warning telling me $this->_primary is an array. Why? I have assigned a string value to $_primary property.

From logs:

2012-02-05T17:41:03+00:00 INFO (6): Array
(
    [1] => modelID
)

I get a warning telling me $this->primary Is that a typo? You've used _primary, not primary. — Chris Laplante
– Chris Laplante, Commented Feb 5, 2012 at 17:13
Hmm. I've never seen this problem before. Can you post a var_dump of the array right before the first line of your delete method? — Chris Laplante
– Chris Laplante, Commented Feb 5, 2012 at 17:29

vascowhite · Accepted Answer · 2012-02-05 19:05:50Z

5

Zend_Db_Table stores primary keys as an array in case a compound key is used, so strictly speaking, it is best (not compulsory) to declare them like this:-

class Model extends Zend_Db_Table_Abstract
{
    public function __construct(array $config = null)
    {
        $this->_primary[1] = 'modelId';
        parent::__construct($config);
        //.............

From the docblock in Zend_Db_Table_Abstract:-

/**
 * The primary key column or columns.
 * A compound key should be declared as an array.
 * You may declare a single-column primary key
 * as a string.
 *
 * @var mixed
 */
protected $_primary = null;

And from the dockblock for $_identity:-

/**
 * If your primary key is a compound key, and one of the columns uses
 * an auto-increment or sequence-generated value, set _identity
 * to the ordinal index in the $_primary array for that column.
 * Note this index is the position of the column in the primary key,
 * not the position of the column in the table.  The primary key
 * array is 1-based.
 *
 * @var integer
 */
protected $_identity = 1;

So you could probably use that instead.
If you have only one column in your primary key then it will be at $_primary[1].

I think this would work for you:-

public function delete($primaryKey)
{

    $where = $this->getAdapter()->quoteInto($this->_primary[1] .' = ?', $primaryKey);
    return parent::delete($where);
}

edited Feb 5, 2012 at 19:05

answered Feb 5, 2012 at 18:02

vascowhite

18.5k9 gold badges64 silver badges79 bronze badges

Sign up to request clarification or add additional context in comments.

7 Comments

Chris Laplante Over a year ago

+1 @vascowhite: I've deleted my answer since yours explains this much better.

Richard Knop Over a year ago

@vascowhite. I think this might be a bug in Zend. Because in some of my unit tests $_protected is array, in others it is a string...

vascowhite Over a year ago

$_protected? do u mean $_primary? If you have extended Zend_Db_Table_Abstract then $_primary is an array. Check the actual code and you will see.

vascowhite Over a year ago

You are obviously getting a mix of behaviours, you could always declare your primary keys as arrays to keep things consistent. I don't think what you are seeing is a bug as it is clearly described in the docs.

vascowhite Over a year ago

Extending Zend_Db_Table like this is not good for unit testing. You should be injecting an instance of it as a dependancy.

|

Collectives™ on Stack Overflow

Zend_Db_Table_Abstract::_primary returns array?

1 Answer 1

7 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

7 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related