search

goodmind / sunflower-fm

Automatically exported from code.google.com/p/sunflower-fm
GNU General Public License v3.0
1 stars 0 forks source link

Error showing if leaving a directory with cjk charactor in name #354

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
1.run sunflower and go into a directory with cjk charactor in name (here to 
/var/build/测试)
2.go to the parent directory (here to /var/build)

What is the expected output?
only see the files and diectories in /var/build

What do you see instead?
only see the files and diectories in /var/build/测试 with a broken icon and 
stderr:

Exception in thread Thread-9:
Traceback (most recent call last):
  File "/usr/lib/python2.7/threading.py", line 810, in __bootstrap_inner
    self.run()
  File "/usr/lib/python2.7/threading.py", line 763, in run
    self.__target(*self.__args, **self.__kwargs)
  File "/usr/share/sunflower/application/plugins/file_list/plugin.py", line 1842, in thread_method
    self._load_emblems(parent, parent_path)
  File "/usr/share/sunflower/application/plugins/file_list/plugin.py", line 1867, in _load_emblems
    emblems = emblem_manager.get_emblems_for_path(path)
  File "/usr/share/sunflower/application/emblems.py", line 245, in get_emblems_for_path
    cursor.execute('SELECT id, name FROM items WHERE path=?', (path,))
ProgrammingError: You must not use 8-bit bytestrings unless you use a 
text_factory that can interpret 8-bit bytestrings (like text_factory = str). It 
is highly recommended that you instead just switch your application to Unicode 
strings.

also cpu goes to 100%.

What version of the product are you using? On what operating system?
reversion 7e9c6e0d97d7 (of cource with the change icon path patch)
on Linux 3.14-1-amd64 #1 SMP Debian 3.14.4-1 (2014-05-13) x86_64 GNU/Linux

patch would help, at least on my system, but maybe(must be?) not enough, or 
make some other things worse.
Sorry, I don't know about sql operate.

Original issue reported on code.google.com by infinity.blick.winkel@gmail.com on 11 Jun 2014 at 2:56

Attachments:

GoogleCodeExporter commented 9 years ago 
Ohh...
reupload patch

Original comment by infinity.blick.winkel@gmail.com on 11 Jun 2014 at 2:57

Attachments:

GoogleCodeExporter commented 9 years ago 
I thought I fixed this issue long time ago. Thanks!
A small note, we do have GitHub repo, if you find using Git easier.

Original comment by MeanEYE.rcf on 11 Jun 2014 at 3:04

GoogleCodeExporter commented 9 years ago

Original comment by MeanEYE.rcf on 18 Jun 2014 at 11:22