Uploaded image for project: 'phpBB'
  1. phpBB
  2. PHPBB-13805

Make generate_text_for_storage() match the signature and feature set of message_parser::parse()

XMLWordPrintable

    • Icon: Improvement Improvement
    • Resolution: Fixed
    • Icon: Major Major
    • 3.2.0-a1
    • 3.2.0-dev
    • Posting
    • None

      Current signatures:

      generate_text_for_storage(&$text, &$uid, &$bitfield, &$flags, $allow_bbcode = false, $allow_urls = false, $allow_smilies = false)
      parse($allow_bbcode, $allow_magic_url, $allow_smilies, $allow_img_bbcode = true, $allow_flash_bbcode = true, $allow_quote_bbcode = true, $allow_url_bbcode = true, $update_this_message = true, $mode = 'post')
      

      generate_text_for_storage() supports the following toggles: allow_bbcode, allow_urls, allow_smilies. All set to false.
      message_parser::parse() supports: allow_bbcode, allow_magic_url, allow_smilies (all three required parameters) plus allow_img_bbcode, allow_flash_bbcode, allow_quote_bbcode, allow_url_bbcode, which default to true.

      I propose to add those four extra parameters with a default value of true to match current behaviour.

            nicofuma nicofuma
            JoshyPHP JoshyPHP
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: