Merge lp:~jimtremblay-s/placesfiles/placesfiles into lp:placesfiles

Proposed by Jim Tremblay
Status: Approved
Approved by: Jacob Vlijm
Approved revision: 2
Proposed branch: lp:~jimtremblay-s/placesfiles/placesfiles
Merge into: lp:placesfiles
Diff against target: 24 lines (+3/-3)
1 file modified
code/placesfiles_indicator (+3/-3)
To merge this branch: bzr merge lp:~jimtremblay-s/placesfiles/placesfiles
Reviewer Review Type Date Requested Status
Jacob Vlijm Pending
Review via email: mp+328267@code.launchpad.net

Description of the change

To be able to open folder with accentuated characters, all %xx must be converted to there equivalent single character.

Feel free to contact me about changes.

Thanks,
Jim

To post a comment you must log in.
Revision history for this message
Jacob Vlijm (vlijm) wrote :

Hi Jim,

Thanks a lot for the improvement!

Cheers,
Jacob

Revision history for this message
Jim Tremblay (jimtremblay-s) wrote :

Hi Jacob,

I'm not aware of how it works on launchpad, is it me that needs to do the merge?

Thanks,
Jim

Unmerged revisions

2. By Jim Tremblay

Unquote %xx characters in path

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'code/placesfiles_indicator'
--- code/placesfiles_indicator 2017-02-28 09:19:53 +0000
+++ code/placesfiles_indicator 2017-07-29 13:12:48 +0000
@@ -9,6 +9,7 @@
9import os9import os
10import subprocess10import subprocess
11import pftools11import pftools
12import urllib.parse
1213
13"""14"""
14Places & Files indicator15Places & Files indicator
@@ -224,10 +225,9 @@
224 set(loc_bookmarks+defaults+netw_bookmarks))])225 set(loc_bookmarks+defaults+netw_bookmarks))])
225226
226 def replace_sc(self, path):227 def replace_sc(self, path):
227 for c in [("%23", "#"), ("%5D", "]"), ("%5E", "^"),228 for c in [("file://", ""), ("'", "'")]:
228 ("file://", ""), ("%20", " "), ("'", "'")]:
229 path = path.replace(c[0], c[1])229 path = path.replace(c[0], c[1])
230 return path230 return urllib.parse.unquote(path)
231231
232 def execute(self, command):232 def execute(self, command):
233 subprocess.Popen(command)233 subprocess.Popen(command)

Subscribers

People subscribed via source and target branches

to all changes: