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

Report post controller and report helper require specific implementation of config class

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 3.2.0-RC1
    • Fix Version/s: 3.2.0-RC2
    • Component/s: None
    • Labels:
      None

      Description

      Not sure if there is a specific reason for this we're missing (entirely possible), but we've extended the phpbb/config/config class and it's causing problems with the Report Post functionality. Both phpbb\report\controller\report and phpbb\report\report_handler expect a configuration object of type phpbb/config/db, which means we can't override the config object with our own object extending from phpbb/config/config.

      Changing the constructors to accept phpbb/config/config instead seems to work fine for our implementation. I can provide a pull request with the changes, but first I wanted to ask if this change made sense or if there was a specific reason for being so specific in the constructor.

        Attachments

          Activity

            People

            Assignee:
            Marc Marc
            Reporter:
            rfdy rfdy [X] (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: