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

Enums serialize using .name() #304

Closed
wants to merge 2 commits into from
Closed

Enums serialize using .name() #304

wants to merge 2 commits into from

Conversation

benohalloran
Copy link
Contributor

Resolves issue #300

@whoshuu
Copy link
Collaborator

whoshuu commented Apr 10, 2015

Hey @benohalloran, thanks for the PR, I really appreciate it. If possible, can you write a test case that validates the behavior you're claiming here? That would speed up the process of getting this merged by many many factors.

@whoshuu whoshuu added this to the 1.4.0 milestone Apr 10, 2015
@benohalloran
Copy link
Contributor Author

@whoshuu I'll write some tests this afternoon and add it to the PR

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.03%) to 65.16% when pulling 39b33f2 on benohalloran:master into 95eb367 on satyan:master.

Tests for enums with overriden and default toString methods
@coveralls
Copy link

Coverage Status

Coverage increased (+0.54%) to 65.72% when pulling 122deb5 on benohalloran:master into 95eb367 on satyan:master.

whoshuu pushed a commit that referenced this pull request Apr 11, 2015
References #304
References #300
References #292
@whoshuu whoshuu closed this in ec8573e Apr 11, 2015
@dhinx
Copy link

dhinx commented Mar 15, 2016

i have a problem.

when i query database with criteria enum and i does not work and shown zero in list

when i check table schema it shown no data type at it
TABLE STOCK_OPNAME ( ID INTEGER PRIMARY KEY AUTOINCREMENT , BUILDING , DESCRIPTION TEXT, EMPLOYEE INTEGER, STOCK_OPNAME_STATUS , UPDATE_DATE INTEGER NULL, ACTIVE_FLAG , CREATE_BY TEXT, CREATE_DATE INTEGER NULL, REF_ID TEXT, UPDATE_BY TEXT, TYPE INTEGER, ACTIVE INTEGER );

here is the code
List<StockOpname> stockOpnames = Select.from(StockOpname.class) .where(Condition.prop("STOCK_OPNAME_STATUS").eq(StockOpnameStatus.INCOMPLETE)) .orderBy("CREATE_DATE DESC").list();

thank you

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.

4 participants