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

Brewnotes don't work any more #417

Closed
mikfire opened this issue Aug 4, 2018 · 2 comments
Closed

Brewnotes don't work any more #417

mikfire opened this issue Aug 4, 2018 · 2 comments

Comments

@mikfire
Copy link
Contributor

mikfire commented Aug 4, 2018

Trying to create a new brewnote throws this error:

[08:20:54.537] ERROR : void Database::updateEntry(Brewtarget::DBTable, int, const char*, QVariant, QMetaProperty, BeerXMLElement*, bool, bool) Could not update brewnote.attenuation to 100: UPDATE brewnote set attenuation=:value where id=121 ERROR:  syntax error at or near "("
LINE 1: EXECUTE  (100)

This is caused by

const QString kAttenuation("attenuation");

in brewnote.cpp. The brewnote table in the database does not have a field called "attenuation". I believe this should be

const QString kAttenuation("projected_atten");
@mikfire
Copy link
Contributor Author

mikfire commented Aug 4, 2018

It would seem the event that should have added this to the table definition didn't fire? My table version is at 8 and has been at least as far back as 2017-12-27. This update marks database v7, so it looks like the code would have never fired.

This is likely my problem. Closing the ticket.

@mikfire mikfire closed this as completed Aug 4, 2018
@pricelessbrewing
Copy link
Contributor

Tested on latest build, notes and taste notes are both working for me.

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