[Doctrine2 icm CodeIgniter 2] Wrappers maken

[Matis]

Beste PRGers,

Momenteel ben ik bezig een webtool te schrijven, welke gebaseerd is op Doctrine2 (2.1.1) en CodeIgniter 2 (2.1.2) welke ik gecloned heb van https://github.com/wildly...Igniter-2-with-Doctrine-2
Nu heb ik een volgende entity gemaakt:

PHP:

namespace Entity;
/* application/models/Entity/User.php */
/**
 * User Model
 *
 * @Entity
 * @Table(name="users")
 */
class User
{
    /**
     * @Id
     * @Column(type="integer", nullable=false)
     * @GeneratedValue(strategy="AUTO")
     */
    protected $id;

    /**
     * @Column(type="string", length=32, unique=true, nullable=false)
     */
    protected $name;

    /**
     * @Column(type="string", length=128, nullable=false)
     */
    protected $hashed_password;

    /**
     * @Column(type="string", length=255, unique=true, nullable=false)
     */
    protected $email;

Middels

./application/doctrine orm:generate-entities models

wordt de entity op de volgende manier gevormd

PHP:

<?php
/* application/models/Entity/User.php */
namespace Entity;

/**
 * User Model
 *
 * @Entity
 * @Table(name="users")
 */
class User
{
    /**
     * @Id
     * @Column(type="integer", nullable=false)
     * @GeneratedValue(strategy="AUTO")
     */
    protected $id;

    /**
     * @Column(type="string", length=32, unique=true, nullable=false)
     */
    protected $name;

    /**
     * @Column(type="string", length=128, nullable=false)
     */
    protected $hashed_password;

    /**
     * @Column(type="string", length=255, unique=true, nullable=false)
     */
    protected $email;
  
    /**
     * Get id
     *
     * @return integer 
     */
    public function getId()
    {
        return $this->id;
    }

    /**
     * Set name
     *
     * @param string $name
     * @return User
     */
    public function setName($name)
    {
        $this->name = $name;
        return $this;
    }

    /**
     * Get name
     *
     * @return string 
     */
    public function getName()
    {
        return $this->name;
    }

    /**
     * Set hashed_password
     *
     * @param string $hashedPassword
     * @return User
     */
    public function setHashedPassword($hashedPassword)
    {
        $this->hashed_password = $hashedPassword;
        return $this;
    }

    /**
     * Get hashed_password
     *
     * @return string 
     */
    public function getHashedPassword()
    {
        return $this->hashed_password;
    }

    /**
     * Set email
     *
     * @param string $email
     * @return User
     */
    public function setEmail($email)
    {
        $this->email = $email;
        return $this;
    }

    /**
     * Get email
     *
     * @return string 
     */
    public function getEmail()
    {
        return $this->email;
    }
}

Nu wil ik graag een wrapper model / class / functie schrijven welke een plaintext password omzet naar een hashed variant en zodoende kan valideren.
Ik dacht het volgende te kunnen doen.

PHP:

/* application/models/Entity/UserWrapper.php */

namespace Entity;

class UserWrapper extends User
{
    public function __construct()
    {
        parent::__construct();
    }

    public function setPassword($plaintext)
    {
        $hashed_password = $this->hashPassword($plaintext);
        $this->setHashedPassword($hashed_password);
    }

    private function hashPassword($password)
    {
        $salt = bin2hex(mcrypt_create_iv(32, MCRYPT_DEV_URANDOM));
        $hash = hash('sha256', $salt . $password);

        return $salt . $hash;
    }
}

Maar als ik vanuit de controller de setPassword functie wil aanroepen op het UserWrapper object krijg ik de volgende foutmelding
( ! ) Fatal error: Uncaught exception 'Doctrine\ORM\Mapping\MappingException' with message 'Class Entity\UserWrapper is not a valid entity or mapped super class.' in /home/matis/wwwroot/ci_checklists/application/libraries/Doctrine/ORM/Mapping/MappingException.php on line 147
( ! ) Doctrine\ORM\Mapping\MappingException: Class Entity\UserWrapper is not a valid entity or mapped super class. in /home/matis/wwwroot/ci_checklists/application/libraries/Doctrine/ORM/Mapping/MappingException.php on line 147

Hoe kan ik dit realiseren? Wat ik liever niet wil is de functie binnen de User class aanmaken, omdat deze telkens geupdated wordt gedurende de ontwikkeling.

Matis

[Seikeau]

Ben niet zo thuis in PHP, maar moet je de User class juist niet de UserWrapper class laten extenden?

[Verwijderd]

Ik heb geen kaas gegeten van Doctrine, maar misschien is het hier beter om een compositie te gebruiken ipv overerving.

Wikipedia: Composition over inheritance

[Bee.nl]

Matis schreef op dinsdag 25 september 2012 @ 12:28:
Hoe kan ik dit realiseren?

De hint is al gegeven in de foutmelding Heb je al naar Inheritance Mapping gekeken in de Doctrine manual?

Bovenstaande code werkt namelijk alleen als je de UserWrapper aanmerkt als @entity en de User class als @MappedSuperclass. Let wel op dat de super class zelf geen entity is, maar alleen mapping informatie bevat.

Wat ik liever niet wil is de functie binnen de User class aanmaken, omdat deze telkens geupdated wordt gedurende de ontwikkeling.

Je kunt gewoon (custom) methods in je entity class opgegeven zonder dat Doctrine daar aan zit bij het genereren van je entities. Ik zie daarom ook niet in waarom je hier een wrapper voor zou willen gebruiken. Het is juist de bedoeling dat je dergelijke methods in de User class onderbrengt.

Generate entity classes and method stubs from your mapping information.

$ php doctrine orm:generate-entities
$ php doctrine orm:generate-entities --update-entities
$ php doctrine orm:generate-entities --regenerate-entities

This command is not suited for constant usage. It is a little helper and does not support all the mapping edge cases very well. You still have to put work in your entities after using this command.

It is possible to use the EntityGenerator on code that you have already written. It will not be lost. The EntityGenerator will only append new code to your file and will not delete the old code. However this approach may still be prone to error and we suggest you use code repositories such as GIT or SVN to make backups of your code.

[ Voor 12% gewijzigd door Bee.nl op 25-09-2012 14:46 ]

[Cartman!]

Gewoon omdraaien, je entity die wrapper laten extenden en dan werkt het prima, daar heb je geen extra mappings voor nodig. Overigens ben ik t met mijn voorganger eens dat je die methods gewoon op je entity zelf kunt zetten (tenzij het librarywerk wordt).

Zelf pas ik de entities aan en gebruik ik de Codegen-functie van NetBeans om de getters/setters te genereren, dan wordt er niks overgeschreven verder.