Merge lp://qastaging/~felihommen/unity-scope-calculator/fix into lp://qastaging/unity-scope-calculator
Proposed by
Felipe Hommen
| Status: | Needs review |
|---|---|
| Proposed branch: | lp://qastaging/~felihommen/unity-scope-calculator/fix |
| Merge into: | lp://qastaging/unity-scope-calculator |
| Diff against target: |
45 lines (+9/-10) 1 file modified
src/unity_calculator_daemon.py (+9/-10) |
| To merge this branch: | bzr merge lp://qastaging/~felihommen/unity-scope-calculator/fix |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Submarine | Pending | ||
|
Review via email:
|
|||
Commit message
Removed string translation from ',' to '.'; Fixes #1333795
Description of the change
Removed string translation from ',' to '.'
Calculator expects decimal separator in localized form, so changing ',' to '.' breaks decimal point calculations in languages like spanish
To post a comment you must log in.
Unmerged revisions
- 45. By Felipe Hommen
-
Removed <search = search.
replace( ',','.' )> which breaks decimal point operations where ',' is the decimal separator
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
