Ticket #725 (closed defect: fixed)

Opened 5 years ago

Last modified 5 years ago

[TiddlyWeb] tiddlers with / in title cannot be imported/used to/with tiddlyweb text store

Reported by: cdent Owned by: ChrisDent
Priority: major Milestone:
Component: tiddlyweb Version:
Severity: high Keywords:
Cc:

Description

/ is not escaped in tiddler titles in the default tiddlyweb text store, so when the system tries to create the directory to store the new tiddler, an IOError exception is thrown (this may be filesystem dependent).

This is a known bug that was left intentionally unfixed while we try to gain some consensus on what is legit in tiddler.title, bag.name, recipe.name.

Change History

Changed 5 years ago by FND

  • summary changed from tiddlers with / in title cannot be imported/used to/with tiddlyweb text store to [TiddlyWeb] tiddlers with / in title cannot be imported/used to/with tiddlyweb text store

Changed 5 years ago by cdent

  • status changed from new to closed
  • resolution set to fixed

This has been resolved by URI escaping tiddler, bag and recipe names in the text store system. First the name is encoded to UTF-8 and then escaped by urllib.quote. The reverse is done when reading names off the disk.

In addition to dealing with / in names, it also makes it possible to write unicode names to disk on filesystems that previously weren't happy about it.

Note: See TracTickets for help on using tickets.