Toolkit-for-ynab: Auto-distribute splits error in calculation

[szrudi]

Name of Defective Feature

• auto-distribute-splits

System Settings (Required)

• ToolKit Version: v 2.8.6
• Operating System (Mac/Windows/Etc): Windows 10
• Browser (Chrome/Firefox/Safari): Chrome
• Browser Version: 69.0.3497.32 beta (64 bit)
• Other Extensions:
  
  
  
  • AdBlock 3.32.1
  
  
  • Better History 3.9.20
  
  
  • Session Buddy 3.6.3
  
  
  • TabCloud 1.17
  
  

Export of Toolkit Settings (Bottom Left of Settings Page)

Settings Export

[{"key":"AccountsDisplayDensity","value":"0"},{"key":"AccountsEmphasizedOutflows","value":false},{"key":"AccountsStripedRows","value":false},{"key":"AdjustableColumnWidths","value":false},{"key":"AutoDistributeSplits","value":true},{"key":"BetterScrollbars","value":"0"},{"key":"BudgetCategoryFeatures","value":true},{"key":"BudgetProgressBars","value":"0"},{"key":"BudgetQuickSwitch","value":true},{"key":"CalendarFirstDay","value":"1"},{"key":"CategoryActivityCopy","value":false},{"key":"CategoryActivityPopupWidth","value":"0"},{"key":"ChangeEnterBehavior","value":false},{"key":"CheckCreditBalances","value":false},{"key":"CheckNumbers","value":false},{"key":"ClearSelection","value":false},{"key":"CollapseSideMenu","value":false},{"key":"ColourBlindMode","value":false},{"key":"CompactIncomeVsExpense","value":true},{"key":"CreditCardEmoji","value":false},{"key":"CurrentMonthIndicator","value":true},{"key":"CustomFlagNames","value":false},{"key":"DaysOfBuffering","value":false},{"key":"DaysOfBufferingHistoryLookup","value":"0"},{"key":"DisableToolkit","value":false},{"key":"DisplayTargetGoalAmount","value":"0"},{"key":"DisplayTotalMonthlyGoals","value":true},{"key":"DisplayUpcomingAmount","value":false},{"key":"EasyTransactionApproval","value":false},{"key":"EditAccountButton","value":"0"},{"key":"EnlargeCategoriesDropdown","value":true},{"key":"EnterToMove","value":true},{"key":"GoalIndicator","value":true},{"key":"GoalWarningColor","value":false},{"key":"GoogleFontsSelector","value":"0"},{"key":"HideAccountBalancesType","value":"0"},{"key":"HideAgeOfMoney","value":false},{"key":"HideHelp","value":true},{"key":"HideReferralBanner","value":true},{"key":"HighlightNegatives","value":false},{"key":"ImportNotification","value":"0"},{"key":"IncomeFromLastMonth","value":"0"},{"key":"IncomeVsExpenseHoverHighlight","value":true},{"key":"LargerClickableIcons","value":false},{"key":"MonthlyNotesPopupWidth","value":"0"},{"key":"NavDisplayDensity","value":"0"},{"key":"Pacing","value":"0"},{"key":"PrintingImprovements","value":true},{"key":"PrivacyMode","value":"0"},{"key":"QuickBudgetWarning","value":false},{"key":"ReconciledTextColor","value":"0"},{"key":"RemovePositiveHighlight","value":false},{"key":"RemoveZeroCategories","value":false},{"key":"ResizeInspector","value":true},{"key":"RightClickToEdit","value":true},{"key":"RowHeight","value":"0"},{"key":"RowsHeight","value":"0"},{"key":"RunningBalance","value":"2"},{"key":"SeamlessBudgetHeader","value":true},{"key":"ShowCategoryBalance","value":false},{"key":"ShowIntercom","value":true},{"key":"SpareChange","value":false},{"key":"SplitKeyboardShortcut","value":true},{"key":"SplitTransactionAutoAdjust","value":false},{"key":"SquareNegativeMode","value":false},{"key":"StealingFromFuture","value":false},{"key":"SwapClearedFlagged","value":false},{"key":"TargetBalanceWarning","value":false},{"key":"ToBeBudgetedWarning","value":false},{"key":"ToggleMasterCategories","value":false},{"key":"ToggleSplits","value":false},{"key":"ToggleTransactionFilters","value":"1"},{"key":"TransactionGridFeatures","value":true},{"key":"ViewZeroAsEmpty","value":true},{"key":"goalIndicator","value":true},{"key":"highlightNegativesNegative","value":false},{"key":"l10n","value":"0"},{"key":"pacing","value":"0"},{"key":"popupCalculator","value":false},{"key":"reports","value":false}]

Steps to Reproduce the Behavior

1. ​Start a split transaction
2. ​Set total amount (in my example 22560)
3. ​Set sub-category values (ex: 2241; 2670; 1992)
4. Hit Auto-distribute button

Expected Behavior

In my example, the correct auto distribute values would be: 7324; 8726; 6510

Actual Behavior

The sub category values are: 8.80; 8.80; 4.40
https://drive.google.com/open?id=1WQ-c07-ugRyKmKW_f6ELwHtc45PXCs3C

With a different thousand separator: 7.32; 8.73; 6.51
https://drive.google.com/open?id=12EN6yzEKLjSsGhLO61_SdIHof6l7tFSQ

[joshmadewell]

Thanks for the report! Looks like we're just doing some bad rounding number formatting. We'll try to get this looked into ASAP!

[CrowderSoup]

I'd like to tackle this to get started on working with the YNAB Toolkit if that's okay 😄

[joshmadewell]

@CrowderSoup absolutely! Go for it!