Bug 396869 - Sensitive word is misspelled in the config and source code
Summary: Sensitive word is misspelled in the config and source code
Status: RESOLVED FIXED
Alias: None
Product: krusader
Classification: Applications
Component: general (show other bugs)
Version: 2.7.0
Platform: Other Linux
: LO minor
Target Milestone: ---
Assignee: Krusader Bugs Distribution List
URL:
Keywords: triaged
Depends on:
Blocks:
 
Reported: 2018-07-26 09:49 UTC by Toni Asensi Esteve
Modified: 2024-12-05 13:26 UTC (History)
4 users (show)

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


Attachments
Several places where the word "sensative" is used (216.19 KB, image/png)
2018-07-26 09:54 UTC, Toni Asensi Esteve
Details
The option "case sensitive sorting" that the user can check but that later is written incorrectly in the "krusaderrc" file (75.70 KB, image/png)
2018-07-26 09:57 UTC, Toni Asensi Esteve
Details
An option that is badly written in the "krusaderrc" file (153.04 KB, image/png)
2018-07-26 10:01 UTC, Toni Asensi Esteve
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Toni Asensi Esteve 2018-07-26 09:49:59 UTC
In the source code the word "sensative" is seen in several places, it's better if it's replaced with the word "sensitive" because that word is a correct one.

I'll attach some screenshots to give additional information.

EXTRA:
In the case of the users (they aren't many) that are already using that preference:
    [Look&Feel]
    Case Sensative Sort=true
then some additional actions might be done (although those users would be able to enable that preference again and then their problem would be solved).
Comment 1 Toni Asensi Esteve 2018-07-26 09:54:47 UTC
Created attachment 114136 [details]
Several places where the word "sensative" is used
Comment 2 Toni Asensi Esteve 2018-07-26 09:57:35 UTC
Created attachment 114137 [details]
The option "case sensitive sorting" that the user can check but that later is written incorrectly in the "krusaderrc" file
Comment 3 Toni Asensi Esteve 2018-07-26 10:01:27 UTC
Created attachment 114138 [details]
An option that is badly written in the "krusaderrc" file
Comment 4 dufferzafar 2018-07-26 14:55:07 UTC
Is the fix as simple as just "s/sensative/sensitive" across the entire codebase?

Will this correctly read already existing "krusaderrc" config files that have "Case Sensative Sorting" written into them?
Comment 5 Toni Asensi Esteve 2018-07-26 15:22:03 UTC
> Is the fix as simple as just "s/sensative/sensitive" across the entire codebase?

It seems that the most important part will be solved this way :-), although someone should try it :-)

> Will this correctly read already existing "krusaderrc" config files that have 
> "Case Sensative Sorting" written into them?

No :-) , and there comes the "EXTRA" section in the bug report, maybe someone will have an idea to solve that, and if he doesn't, it's not a big problem (because "users would be able to enable that preference again").
Comment 6 dufferzafar 2018-07-27 03:18:21 UTC
Replaced all occurrences of "sensative": https://phabricator.kde.org/D14416
Comment 7 Bug Janitor Service 2024-12-05 13:21:55 UTC
A possibly relevant merge request was started @ https://invent.kde.org/utilities/krusader/-/merge_requests/155
Comment 8 Alex Bikadorov 2024-12-05 13:26:53 UTC
I recreated the code changes as a GitLab merge request.

However, the wrong spelling of the UI label in the settings has already been fixed. Having this bug report open for only issues in the source code and configuration file is not necessary, so I close it.