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

Refactor WhereBuilder (caching) and mapping #45

Merged
merged 2 commits into from
Feb 9, 2017
Merged

Refactor WhereBuilder (caching) and mapping #45

merged 2 commits into from
Feb 9, 2017

Conversation

sstok
Copy link
Member

@sstok sstok commented Feb 8, 2017

  • WhereBuilder caching now uses PSR-16 (SimpleCache) instead of Doctrine Cache
  • Support for configuring a Cache has been removed, the Cache key is now automatically configured
    based on the FieldSet (name), serialized root ValuesGroup and Field mappings (exception field options)
  • The CacheWhereBuilder can now be used directly for configuring (no more if statements)
  • When no Cache implementation is provided caching is disabled rather then an exception is thrown
  • Configuring Conversions on a WhereBuilder has been removed, the type options now require configuring the
    column/value conversion

Some minor bug fixes and corrections.

Note: Type conversion configuring was broken due to a wrong FieldType class-reference.

@sstok
Copy link
Member Author

sstok commented Feb 8, 2017

Closes rollerworks/search#149

* WhereBuilder caching now uses PSR-16 (SimpleCache) instead of Doctrine Cache
* Support for configuring a Cache has been removed, the Cache key is now automatically configured
  based on the FieldSet (name), serialized root ValuesGroup and Field mappings (exception field options)
* The CacheWhereBuilder can now be used directly for configuring (no more if statements)
* When no Cache implementation is provided caching is disabled rather then an exception is thrown
* Configuring Conversions on a WhereBuilder has been removed, the type options now require configuring the
  column/value conversion

Some minor bug fixes and corrections.

**Note:** Type conversion configuring was broken due to a wrong FieldType class-reference.
@sstok sstok merged commit a71f6c1 into rollerworks:master Feb 9, 2017
@sstok sstok deleted the refactor/2.0/where-builder branch February 9, 2017 10:11
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