<table><tr><td style="">Joefish created this revision.<br />Herald added a subscriber: kde-utils-devel.<br />Joefish requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D22282">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">The integer_regex in KNumber(const QString &s) does not recognize
numbers with group separators. Therefore, they are removed from the
string representation of the number before being passed.
Also, sanitizing the decimal separator is not needed as it is
already done in the KNumber constructor.</pre></div></div></div><br /><div><strong>TEST PLAN</strong><div><div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">(Copied from @Dan that reported the bug)
1. In Settings select Science Mode
2. In Settings turn on Constants Buttons
3. With a number greater than one thousand in the display field
   (with comma separators) select the Shift button.
   The C1-C6 buttons will all change to Store
4. Select one of the Store buttons
5. Clear the display with AC button
6. Hover over the button the number was stored in (C1-C6).
   The stored number should be displayed as a tool tip
7. Select the button the number was stored in</pre></div>

<p>BUG: 408752</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R353 KCalc</div></div></div><br /><div><strong>BRANCH</strong><div><div>bug_408752 (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D22282">https://phabricator.kde.org/D22282</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>kcalc.cpp</div></div></div><br /><div><strong>To: </strong>Joefish<br /><strong>Cc: </strong>kde-utils-devel<br /></div>