Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implementar polimorfismo do método AbstractBaseModel::join #1

Open
LeonanCarvalho opened this issue Aug 9, 2018 · 0 comments
Open
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@LeonanCarvalho
Copy link
Member

Tornar flexível a criação de joins:

Ao receber uma instância filha de "AbstractBaseModel" o método join deve interepretar como string tornando possível operações como esta possíveis:

$CharactersModel = \L2jBrasil\L2JPHP\ModelFactory::build('Players/Characters'); 
$CharactersModel->select(['character.id','account.name'])
    ->join(\L2jBrasil\L2JPHP\ModelFactory::build('Players/Account'));

Desafios:

  • Como tornar padronizado os "alias" das tabelas, "t1, t2,tn" pode ser uma forma dinâmica fácil de implementar, mas foge das premissas da lib que é tornar o trabalho mais fácil aos desenvolvedores. ?

R: Talvez já definir um alias padrão para cada model através de atributo protected assim como é o nome da tabela.

  • Obrigar que a tabela seja um model?
    R: Não, o desenvolvedor pode ter criado tabelas de apoio que talvez nescessessárias para join.
@LeonanCarvalho LeonanCarvalho added enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers labels Aug 9, 2018
@LeonanCarvalho LeonanCarvalho self-assigned this Aug 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant