Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Add configurable ObjectId tester in JObjectParser #1277

merged 1 commit into from Jun 13, 2012


None yet
2 participants

eltimn commented Jun 8, 2012


Currently, all Strings are tested to see if they are valid ObjectIds. This can cause problems if that's not what's needed. Allow configuration via a FactoryMaker.

object MongoDB extends Factory {

  val jobjectParserObjectIdTester = new FactoryMaker[String => Boolean](s => ObjectId.isValid(s)) {}


@ghost ghost assigned eltimn Jun 7, 2012

dpp pushed a commit that referenced this pull request Jun 13, 2012

Merge pull request #1277 from lift/tcn_issue_1277
Add configurable ObjectId tester in JObjectParser

@dpp dpp merged commit 85b8293 into master Jun 13, 2012

@eltimn eltimn deleted the tcn_issue_1277 branch Nov 6, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment