Skip to content

Create interface RegistryAware#760

Merged
mblaney merged 2 commits intosimplepie:masterfrom
Art4:create-registryaware
Dec 2, 2022
Merged

Create interface RegistryAware#760
mblaney merged 2 commits intosimplepie:masterfrom
Art4:create-registryaware

Conversation

@Art4
Copy link
Contributor

@Art4 Art4 commented Oct 31, 2022

Classes created by the Registry class could have a method set_registry(). The Registry instance will then set itself into the created instance by calling ->set_registry($this).

This PR creates a new interface RegistryAware that defines the method in a type safe way.

Copy link
Member

@jtojnar jtojnar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@Art4 Art4 mentioned this pull request Nov 3, 2022
48 tasks
@Art4
Copy link
Contributor Author

Art4 commented Nov 21, 2022

Ping @mblaney.

@mblaney
Copy link
Member

mblaney commented Dec 2, 2022

looks good, sorry it took so long!

@mblaney mblaney merged commit 7474018 into simplepie:master Dec 2, 2022
@Art4 Art4 deleted the create-registryaware branch December 2, 2022 08:03
Art4 added a commit to Art4/simplepie that referenced this pull request Dec 2, 2022
mblaney pushed a commit that referenced this pull request Jan 20, 2023
* bump version to 1.8.0

* Update CHANGELOG.md

* Fix version tags in deprecated messages

* fix version in old deprecation messages

* Fix typo

see comment from @jtojnar in #752

* Add comment for DataCache interface

see comment from @jtojnar in #752

* Update CHANGELOG.md for #760, #764 and #765

* Update CHANGELOG.md for #762, #767 and #763

* Update CHANGELOG.md for #768 and #770

* Update release date

* Update CHANGELOG.md for #769 and #771

* Update CHANGELOG.md for #766
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants