Bug #620

Swedish letters ÅÄÖ is case sensitive in search EPG

Added by Frans Nilsson almost 2 years ago. Updated 10 months ago.

Status:FixedStart date:08/02/2011
Priority:NormalDue date:
Assignee:Adam Sutton% Done:

100%

Category:EPG
Target version:-
Found in version:2.12 Affected Versions:

Description

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

fix_case_sensitive_search_non-english.patch Magnifier - Patch (1.29 KB) Frans Nilsson, 04/21/2012 05:08 pm

Associated revisions

Revision b904d735
Added by Adam Sutton 10 months ago

Set locale at program start to ensure its properly used throughout. Fixes #620.

History

#1 Updated by Frans Nilsson almost 2 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 about 1 year 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 11 months ago

  • Assignee changed from Andreas Öman 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 11 months ago

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

#5 Updated by Adam Sutton 10 months 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 10 months ago

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

Also available in: Atom PDF