Ticket #520 (reopened defect)

Opened 7 months ago

Last modified 5 months ago

Improper use of "characterFormat" codes around a table causes run-away processing

Reported by: EricShulman Owned by: MartinBudden
Priority: major Milestone: 2.5
Component: core Version:
Severity: high Keywords:
Cc:

Description (last modified by MartinBudden) (diff)

Found by Harley Pebley and reported on TiddlyWikiDev? GoogleGroup?:


If you put bold marks around a table, browser locks up and then reports a message indicating a script isn't responding.

Pretty easy to do: 1. Create a new tiddler. 2. Paste this into it:

''|C1R1|C2R1|
|C1R2|C2R2|
|C1R3|C2R3|
|C1R4|C2R4|''

3. Click done. ==> Error.


Note that the above syntax is "improper", but easy to use by mistake.

Also applies to use of //, , , , or --, but NOT

Note: cancelling processing (after FireFox? message) shows that first row of table is rendered as regular text (not a table) with characterFormat applied, intermediate rows are being rendered as a table, final row is not rendered at all.

Change History

in reply to: ↑ description   Changed 7 months ago by EricShulman

Replying to EricShulman: re-entered example so it is readable as a code block

''|C1R1|C2R1|
|C1R2|C2R2|
|C1R3|C2R3|
|C1R4|C2R4|''

Also applies to use of other characterFormat syntax:

'', //, __, ^^, ~~, or --

  Changed 6 months ago by MartinBudden

  • priority changed from undefined to major
  • milestone set to 2.4

  Changed 6 months ago by MartinBudden

  • milestone changed from 2.4 to 2.5

  Changed 5 months ago by MartinBudden

  • owner changed from JeremyRuston to MartinBudden

  Changed 5 months ago by MartinBudden

  • description modified (diff)

  Changed 5 months ago by MartinBudden

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

Fixed in changeset:5154

  Changed 5 months ago by MartinBudden

  • status changed from closed to reopened
  • resolution deleted

  Changed 5 months ago by FND

previous fix reverted in changeset:5159

Note: See TracTickets for help on using tickets.