PHPcomRapadura / site-principal

Site da Comunidade PHP do Ceará - PHP Com Rapadura
http://phpcomrapadura.org
GNU General Public License v3.0
14 stars 4 forks source link

Feature/issue 32 #38

Closed ghost closed 6 years ago

ghost commented 6 years ago

Feita a refatoração das propriedades de Usuário.

dersonsena commented 6 years ago

É isso ai @jholdy ! Valeu pela contribuição. Só um pequeno detalhe, na próxima task que tu pegar, põe o nome para: feature/issue-32 (com hífen) ;)

dersonsena commented 6 years ago

Vi o projeto aqui com mais calma e tenho mais algumas observações:

1 - Vi que tu alterou a migration que já tinha para criação da tabela de usuários. Depois que tu fez a refatoração da Entity User, bastava dá um doctrine:migrations:diff que ele iria gerar uma nova migration somente alterando os nomes;

2 - Faltou tu mapear o nome da tabela da Entity User para users, para isso, bastava adicionar:

/**
 * @ORM\Entity(repositoryClass="App\Repository\UserRepository")
 * @ORM\Table(name="users")
 */
class User implements UserInterface
{
...
}

Acho que caberia um Bugfix para essas modificações... o que acha @jholdy @Luitame ? O seria melhor reverter essa PR ?