After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 534565 - HTML bookmarks containing %s are not exported correctly
HTML bookmarks containing %s are not exported correctly
Status: RESOLVED FIXED
Product: epiphany
Classification: Core
Component: Bookmarks
2.22.x
Other Linux
: Normal normal
: ---
Assigned To: Epiphany Maintainers
Epiphany Maintainers
: 685116 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2008-05-23 23:28 UTC by Sebastian Keller
Modified: 2012-10-02 09:06 UTC
See Also:
GNOME target: ---
GNOME version: 2.21/2.22


Attachments
Prioritizes smart link URIs over regular ones (2.91 KB, patch)
2012-04-03 14:01 UTC, William Ting
committed Details | Review

Description Sebastian Keller 2008-05-23 23:28:34 UTC
When you export a bookmark containing %s to HTML, remove the bookmark and import it again, the link changes.

http://www.google.com/search?q=%s
becomes
http://www.google.com

This might be usefull when opening the exported HTML-file in a browser, because you have links which actually work, when you click them. On the other hand it makes a usefull backup using HTML-export impossible.

Either it should be changed to use the correct url, since backup is more commonly used than viewing the exported file in a browser.
Or the HTML-file could contain the correct link as a <!-- --> comment, while the anchor-tag still uses the changed link, but that would require changes in the importer, too.
Comment 1 William Ting 2012-04-03 14:01:56 UTC
Created attachment 211223 [details] [review]
Prioritizes smart link URIs over regular ones
Comment 2 Xan Lopez 2012-04-04 08:43:25 UTC
Review of attachment 211223 [details] [review]:

This looks good to me, thank you for the patch!
Comment 3 Xan Lopez 2012-04-20 14:17:22 UTC
Hey William,

I asked chpe about this and he suggested the reason this was done in the past was because Firefox would not accept bookmarks exported with the '%s' in it. Could you verify this still works with this patch?
Comment 4 William Ting 2012-05-01 07:48:30 UTC
Sorry about the late response, I thought I was added to the bug report's CC list but apparently not.

I just tested an export of a smart bookmark with a bookmarks.html file containing the entry:

    <dt><a href="http://www.google.com/search?q=%s">Google Bug Test %s</a></dt>

I then imported it into a new profile under Firefox 12. The bookmark imported successfully into Firefox and worked properly. However, it doesn't resolve into a smart link, but functions as a normal link instead searching for the term "%s" via Google.
Comment 5 Xan Lopez 2012-05-01 20:57:10 UTC
Comment on attachment 211223 [details] [review]
Prioritizes smart link URIs over regular ones

Pushed to master, thank you William!
Comment 6 Xan Lopez 2012-05-01 20:57:25 UTC
Closing.
Comment 7 Xan Lopez 2012-10-02 09:06:12 UTC
*** Bug 685116 has been marked as a duplicate of this bug. ***