eXpansion\Framework\PlayersBundle\Repository\PlayerRepository

Complexity

Total Complexity

4

Size/Duplication

Total Lines	33
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Test Coverage

Coverage

0%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	findByLogin()	0	5	1
A	save()	0	14	3

<?php

namespace eXpansion\Framework\PlayersBundle\Repository;

use eXpansion\Framework\PlayersBundle\Entity\Player;

/**
 * PlayerRepository
 *
 * This class was generated by the Doctrine ORM. Add your own custom
 * repository methods below.
 */
class PlayerRepository extends \Doctrine\ORM\EntityRepository
{
    /**
     * @param string $login Login of the player to laod.
     *
     * @return Player|null
     */
    public function findByLogin($login)
    {
        $player = $this->findOneBy(['login' => $login]);
        return $player;
    }

    /**
     * Save player data in database.
     *
     * @param Player[] $players
     */
    public function save(array $players)
    {
        if (empty($players)) {
            return;
        }

        foreach ($players as $player) {
            var_dump($player->getId());

var_dump($player->getId()); looks like debug code. Are you sure you do not want to remove it? This might expose sensitive data.

            $player = $this->getEntityManager()->merge($player);
            $this->getEntityManager()->persist($player);
        }

        $this->getEntityManager()->flush();
    }
}