Homestyx hydra
Diffusion hydra 176ee9a889df

Fix {Fnnn} rule in Remarkup

Description

Fix {Fnnn} rule in Remarkup

Summary:
Remarkup rule callbacks now get SafeHTML matches instead of string matches. If they call:

$some_lisk_dao->load($matches[1]);

..as is the case with the {F123} rule, we reject the SafeHTML as an invalid ID and return null.

Allow load() to string convert any object (which will either succeed or fatal in an obviously-broken way).

(Long ago we threw instead of returning null here, but it meant we had to do a lot of redundant checks.)

Test Plan: {F123} shows an image again. {C1} embeds a countdown.

Reviewers: vrana, chad

Reviewed By: vrana

CC: aran

Differential Revision: https://secure.phabricator.com/D4961

Details

Provenance
epriestleyAuthored on Feb 14 2013, 4:09 PM
sirocylPushed on Oct 16 2024, 5:49 AM
Parents
R1:fb32a64d8437: Convert Projects to use ObjectItemList
Branches
Unknown
Tags
Unknown

Event Timeline