2cabc5d188
* Use a tree‐based approach for adv. text formatting Sanitizing HTML/Markdown means parsing the content into an HTML tree under‐the‐hood anyway, and it is more accurate to do mention/hashtag replacement on the text nodes in that tree than it is to try to hack it in with regexes et cetera. This undoes the overrides of `#entities` and `#rewrite` on `AdvancedTextFormatter` but also stops using them, instead keeping track of the parsed Nokogiri tree itself and using that in the `#to_s` method. Internally, this tree uses `<mastodon-entity>` nodes to keep track of hashtags, links, and mentions. Sanitization is moved to the beginning, so it should be known that these do not appear in the input. * Also disallow entities inside of `<code>` I think this is generally expected behaviour, and people are annoyed when their code gets turned into links/hashtags/mentions. * Minor cleanup to AdvancedTextFormatter * Change AdvancedTextFormatter to rewrite entities in one pass and sanitize at the end Also, minor refactoring to better match how other formatters are organized. * Add some tests Co-authored-by: Claire <claire.github-309c@sitedethib.com> |
||
---|---|---|
.. | ||
activitypub | ||
connection_pool | ||
ostatus | ||
settings | ||
vacuum | ||
advanced_text_formatter_spec.rb | ||
delivery_failure_tracker_spec.rb | ||
emoji_formatter_spec.rb | ||
entity_cache_spec.rb | ||
extractor_spec.rb | ||
fast_ip_map_spec.rb | ||
feed_manager_spec.rb | ||
hash_object_spec.rb | ||
hashtag_normalizer_spec.rb | ||
html_aware_formatter_spec.rb | ||
link_details_extractor_spec.rb | ||
permalink_redirector_spec.rb | ||
plain_text_formatter_spec.rb | ||
request_pool_spec.rb | ||
request_spec.rb | ||
sanitize_config_spec.rb | ||
scope_transformer_spec.rb | ||
status_cache_hydrator_spec.rb | ||
status_filter_spec.rb | ||
status_finder_spec.rb | ||
status_reach_finder_spec.rb | ||
suspicious_sign_in_detector_spec.rb | ||
tag_manager_spec.rb | ||
text_formatter_spec.rb | ||
user_settings_decorator_spec.rb | ||
webfinger_resource_spec.rb |