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

          Attachments

            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