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.

Sign up for GitHub

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

Jump to bottom

Fix using a field name different from attribute name #45

Open
wants to merge 32 commits into
base: master
Choose a base branch
from
Open

Fix using a field name different from attribute name #45

wants to merge 32 commits into from

Conversation

tahajahangir
Copy link
Contributor

Currently overriding Field.name not works as expected:

class CollectionTest(Collection):
    new_field = StringField(name='string_attr')

I added a failing test in one commit, and next fixed it.

(This branch is based on tahajahangir/coroutine-py3, see network)

Stanislav Vilisov and others added 30 commits September 21, 2013 17:00
added 2to3 for setup.py
887ec37
fix for python3
f6372f3
fix EmailField bug with trying to validate default=None
c08bfa6
find and aind_on changes (@gen.engine -> @gen.coroutine)
2c13fe8
all method for Manager
161c2ae
small refactoring collection.py
7ceea34
count() over gen.coroutine
27c4c65
incr version
aa97ce2
@geerk
fix bug with __collection__ autofill
5e65753
@geerk
add documentation for __collection__ auto-generation
b884a4b
@geerk
add documentation for manager's all method
d3f0a96
@geerk
add python 2.6+ requirement, change package name back to mongotor
b270e3a
@geerk
fix bug with endless recursion
ed5cfc4 
when call hasattr(self, '_initialized') with not initialized database
@geerk
fix collection tests
47bdc13
@geerk
distinct over gen.coroutine
c071b59
@geerk
fix bug (mistype) with max_value validation in NumberField
1bdf4d4
@geerk
fix bug with NumberField
086047a 
when default is not specified but min or max value is specified
you got a validation error
@geerk
add remove method to collection manager
4d7cb60
@geerk
complete manager over gen.coroutine
c5199bb
@tahajahangir
Merge remote-tracking branch 'marcelnicolay/master' into coroutine-py3
f8f157e
@tahajahangir
Fix wrong tests / use gen.coroutine in tests
0ed9a05
@tahajahangir
Add python 2.6/3.2/3.3/3.4/pypy to travis
62404ec
@tahajahangir
Compatible with new version of sure
27d5ee2
@tahajahangir
Use new except syntax (Python 2.6+/3.x)
cb6fd20 
Conflicts:
	mongotor/node.py
@tahajahangir
Use Exceptions as errors base class (instead of StandardError)
85f9428
@tahajahangir
Use six to compatible with python 2,3
a58d831
@tahajahangir
Use string literals instead of unicode
9c0aecb
@tahajahangir
Use unittest instead of sure
dbdd89a
@tahajahangir
No need to run 2to3.py (Code is compatible using six)
95dbb41
@tahajahangir
stop testing on python 2.6/pypy for now
4d4d5ac
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tahajahangir @geerk