Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-30735

cron.php maintenancerun detection

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2, 2.3
    • Fix Version/s: 2.2.1
    • Component/s: Administration
    • Labels:
    • Environment:
      LINUX WITH PHP 5.3.8
    • Database:
      Any
    • Testing Instructions:
      Hide

      1/ install moodle 2.2
      2/ run cron
      3/ verify cron warning disappeared from the notification page

      Show
      1/ install moodle 2.2 2/ run cron 3/ verify cron warning disappeared from the notification page
    • Workaround:
      Hide

      I have already identified the problem... It is in line 195 of the file moodle/admin/renderer.php... The parameter of "cron_overdue_warning" is incorrect... It would to be "$cronoverdue" instead the current "$errorsdisplayed". This change solves the problem.

      Show
      I have already identified the problem... It is in line 195 of the file moodle/admin/renderer.php... The parameter of "cron_overdue_warning" is incorrect... It would to be "$cronoverdue" instead the current "$errorsdisplayed". This change solves the problem.
    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_22_STABLE, MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_22_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      w51_MDL-30735_m23_cron

      Description

      The Admin Notification page does not seem to know about the last run of cron.php. The warning "The cron.php maintenance script has not been run for at least 24 hours" always appears. The script is running successfully...

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              salvetore Michael de Raadt added a comment -

              Thanks for reporting that and providing a solution.

              Show
              salvetore Michael de Raadt added a comment - Thanks for reporting that and providing a solution.
              Hide
              skodak Petr Skoda added a comment -

              Thanks a lot for the report and proposed solution!

              Show
              skodak Petr Skoda added a comment - Thanks a lot for the report and proposed solution!
              Hide
              nebgor Aparup Banerjee added a comment -

              Thanks, this has integrated into 2.2.x and master.

              This has been tested too, so passing.

              Show
              nebgor Aparup Banerjee added a comment - Thanks, this has integrated into 2.2.x and master. This has been tested too, so passing.
              Hide
              nebgor Aparup Banerjee added a comment -

              works for me

              Show
              nebgor Aparup Banerjee added a comment - works for me
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Whoever decided one week was worth 14 days had really one bad idea. Anyway, the nightmare is over, so thanks for your, once again, amazing contributions. Many, many thanks!

              Now... disconnect, relax and enjoy the next days, yay!

              Closing...ciao

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Whoever decided one week was worth 14 days had really one bad idea. Anyway, the nightmare is over, so thanks for your, once again, amazing contributions. Many, many thanks! Now... disconnect, relax and enjoy the next days, yay! Closing...ciao

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    9/Jan/12