Moodle
  1. Moodle
  2. MDL-27801

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

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor 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:
    • Rank:
      17666

      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

      1. upgrade.php
        2 kB
        Daniele Cordella

        Activity

        Hide
        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
        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
        Sam Hemelryk added a comment -

        Thanks Daniele this has been integrated now.
        Cheers
        Sam

        Show
        Sam Hemelryk added a comment - Thanks Daniele this has been integrated now. Cheers Sam
        Hide
        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
        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
        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
        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
        Daniele Cordella added a comment -

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

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

        Upgrade successful with no error messages.

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

        Thanks Michael!

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

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

        Show
        Eloy Lafuente (stronk7) added a comment - Upstream, upstream, this is part of upstream, upstream... thanks!
        Hide
        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
        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: