Project

General

Profile

Bug #620

Swedish letters ÅÄÖ is case sensitive in search EPG

Added by Frans Nilsson over 13 years ago. Updated over 12 years ago.

Status:
Fixed
Priority:
Normal
Assignee:
Adam Sutton
Category:
EPG
Target version:
-
Start date:
2011-08-02
Due date:
% Done:

100%

Estimated time:
Found in version:
2.12
Affected Versions:

Description

If I'm searching in the EPG, åäö is case sensitive. The letters a-z are not.

Files

fix_case_sensitive_search_non-english.patch (1.29 KB) fix_case_sensitive_search_non-english.patch Patch Frans Nilsson, 2012-04-21 17:08

History

#1

Updated by Frans Nilsson over 13 years ago

I have found a solution that works for me. I put the line "setlocale (LC_ALL,"");" in the function extjs_load. (and included <locale.h>;)

#2

Updated by Frans Nilsson over 12 years ago

I have noticed that this problem also exists in the simple.html mobile interface. I guess the problem applies not only to the swedish åäö, but to all non-english letters. I applied the same solution here in the file "src/webui/simpleui.c".
#include <locale.h>
//in the function page_simple I add:
setlocale (LC_ALL,"");

I have written a patch to fix these bugs. It`s my first ever open source software patch so please point me in the right direction if I have done something wrong :)

#3

Updated by Adam Sutton over 12 years ago

  • Assignee changed from Andreas Smas to Adam Sutton
  • Target version set to 3.0

I'll take a look at this when I get a chance.

#4

Updated by Adam Sutton over 12 years ago

  • Status changed from New to Accepted
  • Target version deleted (3.0)
#5

Updated by Adam Sutton over 12 years ago

OK,

I can repeat this, the only thing I'd suggest is that we probably make this setting global. I need to double check, but I'll try and put a fix in git master shortly.

Adam

#6

Updated by Adam Sutton over 12 years ago

  • Status changed from Accepted to Fixed
  • % Done changed from 0 to 100

Applied in changeset commit:b904d735f1b14405493522ea4ff6670d3d202e37.

Also available in: Atom PDF