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

updateInsertStudent Mistake #213

Closed
ghost opened this issue Mar 25, 2017 · 2 comments
Closed

updateInsertStudent Mistake #213

ghost opened this issue Mar 25, 2017 · 2 comments

Comments

@ghost
Copy link

ghost commented Mar 25, 2017

In the updateInsertStudent function is a Mistake. You do not write the passed id in the database.
I guess something like
FieldByName('id').AsLargeInt := student.getId;
is missing.

@ghost ghost added bug database labels Mar 25, 2017
@ghost ghost added this to the Version 1.0 milestone Mar 25, 2017
@ghost ghost assigned cnmicha Mar 25, 2017
@cnmicha
Copy link
Member

cnmicha commented Mar 25, 2017

No, this way we would assign the standard value (-1) of new created rentals as id. Instead, we want SQL to auto-generate the value for us. I guess you mean #204 :)

@cnmicha cnmicha closed this as completed Mar 25, 2017
@ghost
Copy link
Author

ghost commented Mar 27, 2017

No, I do not, we would not and we have to do!
The managemen create for every student his id. This ids have to fit some conditions and if you let it auto-generate they would not fit.
By adding this line in line 538 it will be fixed.
Best regards for your exam.

@ghost ghost reopened this Mar 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant