Page MenuHomePhabricator

Fix an error with Remarkup when a header-indicator row has more cells than the one above it
ClosedPublic

Authored by epriestley on Oct 31 2017, 8:22 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 25, 12:03 AM
Unknown Object (File)
Sat, Apr 13, 5:43 PM
Unknown Object (File)
Thu, Apr 11, 8:20 AM
Unknown Object (File)
Fri, Mar 29, 2:31 AM
Unknown Object (File)
Mar 4 2024, 6:39 PM
Unknown Object (File)
Feb 14 2024, 12:35 PM
Unknown Object (File)
Feb 8 2024, 12:32 AM
Unknown Object (File)
Feb 5 2024, 2:43 AM
Subscribers
None

Details

Summary

From fishing through error logs in PHI184. When row 1 has less cells than row 2, and row 2 has a |--| section (indicating that the cell above should be a table header), we try to convert an invalid cell into a table header. PHP obliges and creates this cell, but it doesn't have 'content', which produces this error:

ERROR 8: Undefined index: content at [/core/lib/libphutil/src/markup/engine/remarkup/blockrule/PhutilRemarkupBlockRule.php:160]

Test Plan

Added a failing (well, complaining) test; made it pass.

Diff Detail

Repository
rPHU libphutil
Lint
Lint Not Applicable
Unit
Tests Not Applicable