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

Added an "asString" function in mongoUtil that can set a value to be str... #38

Merged
merged 2 commits into from Apr 18, 2013

Conversation

Projects
None yet
2 participants
@prismic

prismic commented Mar 27, 2013

Added an "asString" function in mongoUtil that can set a value to be string in mongodb

This is in reference to the issue talked about in https://groups.google.com/forum/#!topic/cfmongodb/Eeia92jt7Uo

Instead of using StringBuffer, I created a separate class for it to avoid breaking any existing implementations.

@marcesher

This comment has been minimized.

Show comment
Hide comment
@marcesher

marcesher Apr 6, 2013

Owner

It's interesting to see how the different solutions to the problem have evolved... this is considerably simpler than the initial approach. I like this, and I want to accept it, but I have one more request:

Please put an example usage into the "examples/gettingstarted.cfm" file. The examples are the primary form of documentation for the project and it'd be good to have your work discoverable by others.

Once we have an example which shows how to save and fetch numbers/booleans as strings, then I'll merge.

Thanks for all your work on this!

Owner

marcesher commented Apr 6, 2013

It's interesting to see how the different solutions to the problem have evolved... this is considerably simpler than the initial approach. I like this, and I want to accept it, but I have one more request:

Please put an example usage into the "examples/gettingstarted.cfm" file. The examples are the primary form of documentation for the project and it'd be good to have your work discoverable by others.

Once we have an example which shows how to save and fetch numbers/booleans as strings, then I'll merge.

Thanks for all your work on this!

@prismic

This comment has been minimized.

Show comment
Hide comment
@prismic

prismic Apr 16, 2013

Thanks. I updated it with an example, and fixed a bug when the method is used in railo (it just bypasses and doesn't do any kind of conversion).

prismic commented Apr 16, 2013

Thanks. I updated it with an example, and fixed a bug when the method is used in railo (it just bypasses and doesn't do any kind of conversion).

marcesher added a commit that referenced this pull request Apr 18, 2013

Merge pull request #38 from prismic/develop
Added an "asString" function in mongoUtil that can set a value to be str...

@marcesher marcesher merged commit 1db88d1 into marcesher:develop Apr 18, 2013

@marcesher

This comment has been minimized.

Show comment
Hide comment
@marcesher

marcesher Apr 18, 2013

Owner

Thanks!

Owner

marcesher commented Apr 18, 2013

Thanks!

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