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.

Sign up for GitHub

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

Jump to bottom

Split memos disappear #701

Closed
pnemonic78 opened this issue Jun 26, 2017 · 2 comments
Closed

Split memos disappear #701

pnemonic78 opened this issue Jun 26, 2017 · 2 comments
Labels
bug
Milestone
v2.2.1

Comments

@pnemonic78
Copy link
Contributor

Steps to reproduce the behaviour

  1. Click on transaction
  2. Click on edit FAB button
  3. Click on split icon
  4. Add memo to 1 or more splits
  5. Click "save"
  6. Click "save" again
  7. Click on edit FAB button
  8. Click on split icon

Expected behaviour

Splits have their memos.

Actual behaviour

Split do not have their memos anymore.

Software specifications

  • GnuCash Android version: 2.2.0
  • System Android version: 6.0
  • Device type: Any
@pnemonic78
Copy link
Contributor Author

pnemonic78 commented Jun 26, 2017 •
edited
Loading

caused by commit 96a2045 in TransactionFormFragment#extractSplitsFromView

@rivaldi8 rivaldi8 added the bug label Jun 26, 2017
@pnemonic78 pnemonic78 mentioned this issue Jun 26, 2017
Closed
rivaldi8 pushed a commit that referenced this issue Jul 3, 2017
@pnemonic78 @rivaldi8
For transactions that have only 2 splits, re-creating the splits lose…
957ae86 
…s all other attributes.

Fixes #701
@rivaldi8
Copy link
Collaborator

rivaldi8 commented Jul 3, 2017

Merged into hotfix/patches (957ae86). Thanks!

@rivaldi8 rivaldi8 closed this as completed Jul 3, 2017
@codinguser codinguser modified the milestones: v2.3.0, v2.2.1 Jul 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
v2.2.1
Development

No branches or pull requests
3 participants
@codinguser @rivaldi8 @pnemonic78