Bug 230645 - plasmapkg doesn't work for languages in which package types have capital letters
Summary: plasmapkg doesn't work for languages in which package types have capital letters
Status: RESOLVED FIXED
Alias: None
Product: plasma4
Classification: Unmaintained
Component: general (show other bugs)
Version: unspecified
Platform: Debian testing Unspecified
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords: junior-jobs
Depends on:
Blocks:
 
Reported: 2010-03-14 03:03 UTC by Eckhart Wörner
Modified: 2010-06-17 10:34 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Eckhart Wörner 2010-03-14 03:03:21 UTC 
Version:            (using KDE 4.4.1)
Installed from:    Debian testing/unstable Packages

If a package type translation contains a capital letter, like "Plasmoid", plasmapkg fails to interpret it correctly.

main.cpp:

const QString type = args->getOption("type").toLower();
[...]
if (type == i18n("plasmoid") || type == "plasmoid") {

The error is in the assumption that i18n("plasmoid") is lowercase (same applies to all other types as well).
Comment 1 Matteo Agostinelli 2010-06-17 10:34:06 UTC 
Fixed in r1139060