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

SearchCondition: __toString() -> toString() #187

Merged
merged 2 commits into from
Sep 26, 2017

Conversation

Slamdunk
Copy link
Collaborator

@Slamdunk Slamdunk commented Sep 26, 2017

Reference: http://phpsadness.com/sad/14

  • Introduce Search\ConditionInterface
  • Typehint against Search\ConditionInterface
  • Move existing search conditions from __toString() to toString()
  • Typehint DateTimeInterface istead of DateTime

BC Breaks

  1. Signature of Ddeboer\Imap\Search\AbstractCondition changed.
  2. Introduced Ddeboer\Imap\Search\ConditionInterface to let users implement their own Conditions without inheritance.

@Slamdunk Slamdunk changed the title [WIP] Search: __toString() -> toString() [WIP] SearchCondition: __toString() -> toString() Sep 26, 2017
@Slamdunk Slamdunk changed the title [WIP] SearchCondition: __toString() -> toString() SearchCondition: __toString() -> toString() Sep 26, 2017
@Slamdunk Slamdunk merged commit c53fdf1 into ddeboer:develop Sep 26, 2017
@Slamdunk Slamdunk deleted the search_refactor branch September 26, 2017 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant