XMLWordPrintable

    • Icon: Sub-task Sub-task
    • Resolution: Fixed
    • Icon: Minor Minor
    • 3.7.4, 3.8.1
    • 3.6.6, 3.7.2, 3.8, 3.9
    • Forum

      This was detected while progressing with MDL-67114, so I'm creating it as subtask there because it's leading to php74 failures (not happening in older php versions).

      In any case, the code looks just "wrong", or I've not been able to find an explanation to it, so it requires review and, if possible, amendment.

      Code: https://github.com/moodle/moodle/blob/master/mod/forum/classes/task/send_user_digests.php#L202-L204

      Apparently we are replacing the whole array by the (boolean) result of a capability check. So any future use of the original array becomes borked.

      For your consideration, ciao

            dobedobedoh Andrew Lyons
            stronk7 Eloy Lafuente (stronk7)
            Eloy Lafuente (stronk7) Eloy Lafuente (stronk7)
            Sara Arjona (@sarjona) Sara Arjona (@sarjona)
            CiBoT CiBoT
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved:

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0 minutes
                0m
                Logged:
                Time Spent - 3 hours
                3h

                  Error rendering 'clockify-timesheets-time-tracking-reports:timer-sidebar'. Please contact your Jira administrators.