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

Move Join to Model\Join #680

Merged
merged 1 commit into from
Jul 30, 2020
Merged

Move Join to Model\Join #680

merged 1 commit into from
Jul 30, 2020

Conversation

georgehristov
Copy link
Collaborator

@georgehristov georgehristov commented Jul 28, 2020

BC break (minor due to internal use of the classes only):

  • instanceof \atk4\data\Join::class should be replaced with instanceof \atk4\data\Model\Join::class
  • instanceof \atk4\data\Join\Array_::class should be replaced with instanceof \atk4\data\Persistence\Array_\Join::class
  • instanceof \atk4\data\Join\Sql::class should be replaced with instanceof \atk4\data\Persistence\Sql\Join::class

@mvorisek
Copy link
Member

Why Join needs to be Join instead of Model and what si the difference between Union Model?

@georgehristov
Copy link
Collaborator Author

Why Join needs to be Join instead of Model

Because of separation of Model and Persistence. Each persistence handles the JOIN in a child class

what si the difference between Join and Union Model?

Same like in SQL:
https://stackoverflow.com/questions/905379/what-is-the-difference-between-join-and-union

@mvorisek
Copy link
Member

mvorisek commented Jul 28, 2020

answered in #682

if this is rename only and @DarkSide666 thinks it should be moved lets move it, I can not comment on something that should be not presented at all ;-)

@DarkSide666
Copy link
Member

@georgehristov Is this just moving namespaces and moving functionality out from Model class to trait or is there something more to pay attention on?
If it's just moving NS, then it looks fine with me to move it under respective persistence.

@georgehristov
Copy link
Collaborator Author

@DarkSide666 nothing more than reordering and separating into Trait

@DarkSide666 DarkSide666 merged commit 8369ca2 into develop Jul 30, 2020
@DarkSide666 DarkSide666 deleted the refactor/join-namespace branch July 30, 2020 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants