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

Undefined property: stdClass::$blockcolumnbgc in moodle20/theme/formal_white/db/install.php on line 62

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0.3
    • Fix Version/s: 2.0.3
    • Component/s: Themes
    • Labels:
      None
    • Testing Instructions:
      Hide

      to test my solution it shoud be uninstalled the theme
      then:

      process 1:
      step 1: install the old release of March 5 - upgrade
      step 2: and finally install the current release - upgrade

      process 2:
      step 1: install a release older than the one of March 5 - upgrade
      step 2: and finally install the current release - upgrade

      Every upgrade should be error/notice free.

      Show
      to test my solution it shoud be uninstalled the theme then: process 1: step 1: install the old release of March 5 - upgrade step 2: and finally install the current release - upgrade process 2: step 1: install a release older than the one of March 5 - upgrade step 2: and finally install the current release - upgrade Every upgrade should be error/notice free.
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Description

      As Eloy stated in MDL-27588
      I actually found that the error is due to a name of a variable that I modified during last two previous push.
      The problem comes from the missing version.php file in the previous two releases of the theme so I was forced to add a dirty workaround in formal_white/db/install.php

        Gliffy Diagrams

        1. upgrade.php
          2 kB
          Daniele Cordella

          Activity

          Hide
          daniss Daniele Cordella added a comment -

          this is fix is, IMHO, a very silly and quite useless.
          However, it was a bug and now it has been fixed.

          As usual: I apologize whether I still did not fill the tracker form correctly. Sam, please, let me know! Ciao.

          Show
          daniss Daniele Cordella added a comment - this is fix is, IMHO, a very silly and quite useless. However, it was a bug and now it has been fixed. As usual: I apologize whether I still did not fill the tracker form correctly. Sam, please, let me know! Ciao.
          Hide
          samhemelryk Sam Hemelryk added a comment -

          Thanks Daniele this has been integrated now.
          Cheers
          Sam

          Show
          samhemelryk Sam Hemelryk added a comment - Thanks Daniele this has been integrated now. Cheers Sam
          Hide
          daniss Daniele Cordella added a comment -

          Ciao Sam.
          To be 100% correct it should be added to the file version.php something like:
          $plugin->version = 2011061300;

          and should be added the file formal_white/db/upgrade.php attached to this issue

          Show
          daniss Daniele Cordella added a comment - Ciao Sam. To be 100% correct it should be added to the file version.php something like: $plugin->version = 2011061300; and should be added the file formal_white/db/upgrade.php attached to this issue
          Hide
          daniss Daniele Cordella added a comment -

          if you agree, I go to push this change to my github repo.
          Thanks for your involvement and patience.

          Show
          daniss Daniele Cordella added a comment - if you agree, I go to push this change to my github repo. Thanks for your involvement and patience.
          Hide
          daniss Daniele Cordella added a comment -

          I am sorry. As usual the first attachment was wrong. The current one is better.

          Show
          daniss Daniele Cordella added a comment - I am sorry. As usual the first attachment was wrong. The current one is better.
          Hide
          salvetore Michael de Raadt added a comment -

          Upgrade successful with no error messages.

          Show
          salvetore Michael de Raadt added a comment - Upgrade successful with no error messages.
          Hide
          daniss Daniele Cordella added a comment -

          Thanks Michael!

          Show
          daniss Daniele Cordella added a comment - Thanks Michael!
          Hide
          stronk7 Eloy Lafuente (stronk7) added a comment -

          Upstream, upstream, this is part of upstream, upstream... thanks!

          Show
          stronk7 Eloy Lafuente (stronk7) added a comment - Upstream, upstream, this is part of upstream, upstream... thanks!
          Hide
          daniss Daniele Cordella added a comment -

          THANKS THANKS THANKS THANKS THANKS THANKS THANKS THANKS ELOY
          if I were as good as 5% of how you are, I would be great!

          Show
          daniss Daniele Cordella added a comment - THANKS THANKS THANKS THANKS THANKS THANKS THANKS THANKS ELOY if I were as good as 5% of how you are, I would be great!

            People

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

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                5/May/11