Uploaded image for project: 'Plugins'
  1. Plugins
  2. CONTRIB-6003

Eliminate stuff being dependent of $CFG->branch

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Minor Minor
    • 3.1
    • 2.9.3, 3.0, 3.1
    • Module: Survey Pro
    • None
    • MOODLE_29_STABLE, MOODLE_30_STABLE, MOODLE_31_STABLE
    • MOODLE_31_STABLE

      It has been detected that, in order to keep the master branch of the module working for all Moodle versions since, say, 2.7. Some conditional code, based on $CFG->branch has been added here and there.

      $ grep -r 'CFG.*branch' *
      lib.php:if ($CFG->branch == '26') {
      mod_form.php:        if ($CFG->branch < '29') {
      settings.php:    if ($CFG->branch < '29') {
      

      That's simply not acceptable and proper branches should be used for every line of code.

      So this is about to:

      1) Decide what goes to the MOODLE_XX_STABLE branches, so people using them can do it up to now.
      2) In master (3.1), kill all the conditional coding above. And its consequences. Completely.
      3) Decide which politic is followed to backport future issues the the MOODLE_XX_STABLES (nothing is backported, everything is backported, only serious bugs are...).

      Ciao

            daniss Daniele Cordella
            stronk7 Eloy Lafuente (stronk7)
            Eloy Lafuente (stronk7) Eloy Lafuente (stronk7)
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved:

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