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

Problem adding a second account #10

Open
cjfloss opened this issue Jan 18, 2018 · 3 comments
Open

Problem adding a second account #10

cjfloss opened this issue Jan 18, 2018 · 3 comments
Labels
bug Something isn't working

Comments

@cjfloss
Copy link
Owner

cjfloss commented Jan 18, 2018

Second account is added to the database, but not to the source list
after added the app crashes

[DEBUG 16:31:28.855111] Sidebar.vala:912: row activated
[INFO 16:31:28.870783] [Gtk] GtkDialog mapped without a transient parent. This is discouraged.
[DEBUG 16:31:45.898611] Database.vala:964: executing query: INSERT INTO `accounts` (`number`, `description`, `balance`, `type`) VALUES ('Caixa', 'Federal', 50.0, 0);
**
ERROR:treemap.c:7246:gee_tree_map_value_collection_real_add: code should not be reached
Aborted (core dumped)
@cjfloss
Copy link
Owner Author

cjfloss commented Jan 18, 2018

After deleting the database and restarting the app the problem seems not to occur anymore.

@cjfloss cjfloss added bug Something isn't working needs confirmation labels Jan 18, 2018
@cjfloss cjfloss changed the title Regression: Adding a second account Problem adding a second account Jan 18, 2018
@cjfloss
Copy link
Owner Author

cjfloss commented Jan 18, 2018

Steps to reproduce

  • Start app for the first time
  • Create an account
  • Create another account
  • Record a transaction in any account
  • Restart the app
  • Create an account
    --
    App Crashes

@cjfloss cjfloss added this to the AppcenterRelease milestone Jan 20, 2018
cjfloss pushed a commit that referenced this issue Feb 9, 2018
@cjfloss
Copy link
Owner Author

cjfloss commented Feb 9, 2018

The problem is in the account's caching.
Disabling it for now.

@cjfloss cjfloss moved this from To Do to In progress in Fully Functional Release Feb 14, 2018
@cjfloss cjfloss moved this from In progress to Need Improvement in Fully Functional Release May 24, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Fully Functional Release
  
Need Improvement
Development

No branches or pull requests

1 participant