davidedmundson created this revision.
davidedmundson added a reviewer: Plasma.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  We would check if the string (unit1) matched any known units (s) using a
  case insensitve compare.
  
  If it did, we would treat that as a valid unit and pass it on to
  KUnitConversion, but unfortunately that is case senstive, and then fails
  to find anything.
  
  This fixes the case of the user typed unit to be correct.

TEST PLAN
  Ran krunner with "640 gbp in eur" it now returns the same as "640 GBP 
  in EUR" . 
  debug in kunitconversion shows it now has a valid unit ID for the first 
  query.
  (the result is still utterly wrong, but it's one problem less)

REPOSITORY
  rKDEPLASMAADDONS Plasma Addons

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D1338

AFFECTED FILES
  runners/converter/converterrunner.cpp

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: davidedmundson, Plasma
Cc: plasma-devel, sebas
_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to