Uploaded image for project: 'phpBB3'
  1. phpBB3
  2. PHPBB3-10600

When creating a report for a post, copy the post itself

    Details

    • Type: New Feature
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.0.10, 3.1.0-dev
    • Fix Version/s: 3.1.0-a1
    • Component/s: Other
    • Labels:
      None
    • Environment:
      PHP 5.3.8, MySQL 5.5.16,Apache/2.2.21,
      firefox: 3.6, 9.0
      chrome: 16.0
      SO: WINNT 6.1 (windows 7)

      Description

      Tracker for the feature as described in http://area51.phpbb.com/phpBB/viewtopic.php?f=108&t=42459

        Issue Links

          Activity

          Hide
          brunoais brunoais added a comment - - edited
          • Changing the database update script to add the new table column -> OK
          • Changing the report file so that the original post is saved in the database -> OK

          Continuing...

          Show
          brunoais brunoais added a comment - - edited Changing the database update script to add the new table column -> OK Changing the report file so that the original post is saved in the database -> OK Continuing...
          Hide
          brunoais brunoais added a comment - - edited
          • Changing the database update script to add the new table column -> OK
          • Changing the report file so that the original post is saved in the database -> OK
          • Changing the mcp_reports so that the reported version of the post is shown in the preview instead of the current post -> OK
          • Changing posting_topic_review.html so that the reported post is marked in the Topic review section of Report details -> OK
          • Testing posting_topic_review.html applied to other pages (while posting, for example) to verify that the previous changes do not affect other forum features where this feature is not used -> PASSED.

          Continuing...

          Show
          brunoais brunoais added a comment - - edited Changing the database update script to add the new table column -> OK Changing the report file so that the original post is saved in the database -> OK Changing the mcp_reports so that the reported version of the post is shown in the preview instead of the current post -> OK Changing posting_topic_review.html so that the reported post is marked in the Topic review section of Report details -> OK Testing posting_topic_review.html applied to other pages (while posting, for example) to verify that the previous changes do not affect other forum features where this feature is not used -> PASSED. Continuing...
          Hide
          nickvergessen Joas Schilling added a comment -

          I think this should also include the bbcode stuff, so we can parse the post in the mcp

          Show
          nickvergessen Joas Schilling added a comment - I think this should also include the bbcode stuff, so we can parse the post in the mcp
          Hide
          Oleg Oleg [X] (Inactive) added a comment -

          bbcode was not included, only nl2br is done.

          Show
          Oleg Oleg [X] (Inactive) added a comment - bbcode was not included, only nl2br is done.
          Hide
          brunoais brunoais added a comment -

          Should BBCode parse have been included?
          I thought (according to my read) bbcode_nl2br parses bbcode. Is that wrong?

          Show
          brunoais brunoais added a comment - Should BBCode parse have been included? I thought (according to my read) bbcode_nl2br parses bbcode. Is that wrong?
          Hide
          brunoais brunoais added a comment -

          Except for the situation as described in:
          PHPBB3-10845
          All use cases discussed and I could think of were tested and no problems were found

          Show
          brunoais brunoais added a comment - Except for the situation as described in: PHPBB3-10845 All use cases discussed and I could think of were tested and no problems were found

            People

            • Assignee:
              Oleg Oleg [X] (Inactive)
              Reporter:
              brunoais brunoais
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development