Moodle
  1. Moodle
  2. MDL-31167 PHP strict META
  3. MDL-34204

Strict standards: Non-static method grade_report_grader::process_action()

    Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.3
    • Fix Version/s: 2.3.4
    • Component/s: Gradebook
    • Labels:
    • Testing Instructions:
      Hide

      1. Go to the gradebook (grader report) in any course.

      2. Click the [-] icon next to any category (in the column headings).

      3. Verify that there are no Strict Syntax warnings.

      Show
      1. Go to the gradebook (grader report) in any course. 2. Click the [-] icon next to any category (in the column headings). 3. Verify that there are no Strict Syntax warnings.
    • Affected Branches:
      MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE
    • Pull from Repository:
    • Pull Master Branch:
    • Rank:
      42543

      Description

      Strict standards: Non-static method grade_report_grader::process_action() should not be called statically in 
      /home/fred/www/repositories/stable_master/moodle/grade/report/grader/index.php on line 110 
      Call Stack: 0.0007 728384 
      1. {main}() /home/fred/www/repositories/stable_master/moodle/grade/report/grader/index.php:0 
      
      1. Go to the gradebook
      2. Click on the little icon next to the course name (Aggregates only)
      3. Notice appears on next page}

        Activity

        Hide
        Dan Poltawski added a comment -

        Just saw this one myself

        Show
        Dan Poltawski added a comment - Just saw this one myself
        Hide
        Tim Hunt added a comment -

        Note that we cannot just change process_action() to be a static method. That method is defined in the base class as an abstract method, so we cannot change the API in a subclass.

        Show
        Tim Hunt added a comment - Note that we cannot just change process_action() to be a static method. That method is defined in the base class as an abstract method, so we cannot change the API in a subclass.
        Hide
        Dan Poltawski added a comment -

        Integrated, thanks.

        This is a bit ugly, but such is the life of STABLE bugfixes, I guess.

        Show
        Dan Poltawski added a comment - Integrated, thanks. This is a bit ugly, but such is the life of STABLE bugfixes, I guess.
        Hide
        Rossiani Wijaya added a comment -

        This works as expected.

        Tested for 2.3 and master

        Test passed.

        Show
        Rossiani Wijaya added a comment - This works as expected. Tested for 2.3 and master Test passed.
        Hide
        Dan Poltawski added a comment -

        Congratulations! Another bug solved.. only another 7330 to go, thanks for contributing to contributing to 0.8% of all bugs being fixed this week!

        ciao
        Dan

        Show
        Dan Poltawski added a comment - Congratulations! Another bug solved.. only another 7330 to go, thanks for contributing to contributing to 0.8% of all bugs being fixed this week! ciao Dan

          People

          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: