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

Circular dependencies when cron task depends on controller.helper

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • 3.3.2-RC1
    • 3.3.1
    • Cron
    • None

      This PR added the cron.manager dependency to the controller.helper service. This can lead to cyclic dependencies when a cron task depends on the controller.helper service. I've attached a minimal extension to reproduce the problem. The issue is not limited to extensions though.

      Expected error message:

      Circular reference detected for service "kasimi.circrefin331.cron.task.sample", path: "kasimi.circrefin331.cron.task.sample -> controller.helper -> cron.manager -> kasimi.circrefin331.cron.task.sample".

      Real world example: while having both mChat and ABBC3 extensions enabled, mChat's cron task can't be executed and throws an error because of circular references:

      dmzx.mchat.cron.task.mchat_prune
      -> text_formatter.s9e.parser
      -> controller.helper
      -> cron.manager
      -> cron.task_collection
      -> dmzx.mchat.cron.task.mchat_prune

       

            Marc Marc
            kasimi kasimi
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: