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

Fatal error: Cannot access protected property xmldb_table::$name in lib/adminlib.php on line 455

          Details

          • Type: Bug
          • Status: Closed
          • Priority: Minor
          • Resolution: Fixed
          • Affects Version/s: 2.3
          • Fix Version/s: 2.3.1
          • Component/s: Database SQL/XMLDB
          • Labels:

            Description

            On trying to delete a third-party activity plugin, the following fatal error is displayed:

            Fatal error: Cannot access protected property xmldb_table::$name in lib/adminlib.php on line 455

            Suggested fix (lib/adminlib.php, line 455):

            • OLD: $table_names[] = strtolower($table->name);
            • NEW: $table_names[] = strtolower($table->getName());

              Gliffy Diagrams

                Attachments

                  Issue Links

                  is duplicated by

                  Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-34193 Can not delete data tables created from a module

                  • Minor - Minor loss of function where workaround is possible
                  • Closed

                  Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-34196 adminlib.php reports error when deleting plugins

                  • Minor - Minor loss of function where workaround is possible
                  • Closed

                    Activity

                    Ascending order - Click to sort in descending order
                    Hide
                    Permalink
                    skodak Petr Skoda added a comment -

                    Thanks a lot for the report with patch and sorry for the trouble.

                    Show
                    skodak Petr Skoda added a comment - Thanks a lot for the report with patch and sorry for the trouble.
                    Hide
                    Permalink
                    poltawski Dan Poltawski added a comment -

                    Integrated, thanks Gordon/Petr.

                    Show
                    poltawski Dan Poltawski added a comment - Integrated, thanks Gordon/Petr.
                    Hide
                    Permalink
                    abgreeve Adrian Greeve added a comment -

                    Tested in master to replicate the problem and then master integration. The Fatal error is no longer raised. No other problems encountered.
                    Test passed.

                    Show
                    abgreeve Adrian Greeve added a comment - Tested in master to replicate the problem and then master integration. The Fatal error is no longer raised. No other problems encountered. Test passed.
                    Hide
                    Permalink
                    samhemelryk Sam Hemelryk added a comment -

                    Congratulations your code is upstream - gold star for you!

                    This issue + 79 others made it in in time for the minor releases.
                    Thank you everyone involved for your exuberant efforts.

                    Show
                    samhemelryk Sam Hemelryk added a comment - Congratulations your code is upstream - gold star for you! This issue + 79 others made it in in time for the minor releases. Thank you everyone involved for your exuberant efforts.

                      People

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

                        Dates

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