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

IdTable<int> Can't use not autoincrement id field #57

Closed
sergeybannikov opened this issue Oct 19, 2016 · 5 comments
Closed

IdTable<int> Can't use not autoincrement id field #57

sergeybannikov opened this issue Oct 19, 2016 · 5 comments

Comments

@sergeybannikov
Copy link

sergeybannikov commented Oct 19, 2016

issue is only in 0.7.0, 0.6.8 is fine with it
Entity.kt 430

                for ((entry, genValues) in toFlush.zip(ids)) {
// genvalues are empty, null pointer Exception
                    val id = genValues[table.id]!!

// rewrites always by generateg value
                    entry.id._value = (table.id.columnType as EntityIDColumnType<*>).idColumn.columnType.valueFromDB(when (id) {
                        is EntityID<*> -> id._value!!
                        else -> id
                    })

Thank you!

@Tapac
Copy link
Contributor

Tapac commented Oct 19, 2016

@Aileron1410 , can you provide simple snippet to reproduce that case and please ensure that it doesn't work on latest version ( 0.7.1 ).

@Tapac
Copy link
Contributor

Tapac commented Oct 19, 2016

I can reproduce it by myself, so will fix it ASAP (and add code as test).
Thank you for report

@sergeybannikov
Copy link
Author

Thank you very much!
We will waight 0.7.2)

19 Окт 2016 г. 21:16 пользователь "Andrey.Tarashevskiy" <
notifications@github.com> написал:

Fixed. Will be available a bit later in 0.7.2


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#57 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEQHtAmgzGtJBmOUdreepnNe3Sn11hWFks5q1l6KgaJpZM4KaquE
.

@Tapac
Copy link
Contributor

Tapac commented Oct 19, 2016

Fixed in 0.7.2 version. Please check it at bintray.

@Tapac Tapac closed this as completed Oct 19, 2016
@sergeybannikov
Copy link
Author

Thank you very much, it works fine now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants