Skip to content

fixed getRelated already fetched relations#16449

Open
rudiservo wants to merge 1 commit intophalcon:5.0.xfrom
rudiservo:i16409
Open

fixed getRelated already fetched relations#16449
rudiservo wants to merge 1 commit intophalcon:5.0.xfrom
rudiservo:i16409

Conversation

@rudiservo
Copy link
Contributor

Hello!

In raising this pull request, I confirm the following:

  • I have read and understood the Contributing Guidelines
  • I have checked that another pull request for this purpose does not exist
  • I wrote some tests for this PR
  • I have updated the relevant CHANGELOG
  • I have created a PR for the documentation about this change

Small description of change:
GetRelated should reuse already fetched relation inside the model, avoiding more hits to the database.
If devs need fresh data they can use Model::refresh and with the PR also ResultSet::refresh.

Thanks

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.

1 participant