Comment 7 for bug 951739

Revision history for this message
SegundoBob (bhossley) wrote :

There are problems with your latest change: 1) For me it does not do what it was intended to do. 2) For me, it (or some other change in this area), changes an http: URL into a file: URL.

The file-scheme URL with blanks still works. For example for me:

file:///media/datw1/BobH/0/2002 Dodge Neon SXT/Dodge Dealers.txt

I ran the above through urllib.quote() to get:

file:///media/datw1/BobH/0/2002%20Dodge%20Neon%20SXT/Dodge%20Dealers.txt

But the "quoted" URL still fails with this message in the Log pane:

File '/media/datw1/BobH/0/2002%20Dodge%20Neon%20SXT/Dodge%20Dealers.txt' does not exist

What is worse, is that now double-click-icon-box with a URL :

https://bugs.launchpad.net/leo-editor/+bug/951739

Now calls the default browser to open URL:

file:///leo-editor/+bug/951739

urllib.unquote() does NOT change https://bugs.launchpad.net/leo-editor/+bug/951739 so something else is changing it.

Test System:
Ubuntu 11.10 with Fluxbox window manager
Leo Log Window
Leo 4.9.1 devel, build 5128, 2012-03-12 08:55:37
Python 2.7.2, qt version 4.7.3
linux2
setting leoID from os.getenv('USER'): 'bob'
load dir: /home/bob/bzr/LeoLatest/leo/core
global config dir: /home/bob/bzr/LeoLatest/leo/config
home dir: /home/bob
reading settings in /home/bob/bzr/LeoLatest/leo/config/leoSettings.leo
reading settings in /home/bob/.leo/myLeoSettings.leo
reading settings in /home/bob/.leo/workbook.leo
reading: /home/bob/.leo/workbook.leo