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

Default profile page uses page-layout "mydashboard" instead of "mypublic"

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.7.2, 2.8
    • Fix Version/s: 2.7.4, 2.8.1
    • Component/s: Themes
    • Labels:
    • Testing Instructions:
      Hide

      Test:

      Add a new region "side-center" to a customised mypublic layout"

      In theme/mytheme/config.php:

      // My public page.
      		 
          'mypublic' => array(
      		 
              'file' => 'profile.php',
      		 
              'regions' => array('side-pre', 'side-post', 'side-center', 'side-admin'),
      		 
              'defaultregion' => 'side-pre',
      		 
          ),

      In theme/mytheme/layout/profile.php:

      echo $OUTPUT->course_content_header();
      		 
      echo $OUTPUT->main_content();
      		 
      //Add the new content area.
      		 
      echo $OUTPUT->blocks('side-center');
      		 
      echo $OUTPUT->course_content_footer();

      2. Edit the default layout: Site Adminisitration -> Appearance -> Default profile page

      Expected result: It should be possible add blocks to the side-center region

      Current result: The region does not appear (unless you also add it to the mydashboard layout)

      Show
      Test: Add a new region "side-center" to a customised mypublic layout" In theme/mytheme/config.php: // My public page. 'mypublic' => array ( 'file' => 'profile.php' , 'regions' => array ( 'side-pre' , 'side-post' , 'side-center' , 'side-admin' ), 'defaultregion' => 'side-pre' , ), In theme/mytheme/layout/profile.php: echo $OUTPUT ->course_content_header(); echo $OUTPUT ->main_content(); //Add the new content area. echo $OUTPUT ->blocks( 'side-center' ); echo $OUTPUT ->course_content_footer(); 2. Edit the default layout: Site Adminisitration -> Appearance -> Default profile page Expected result: It should be possible add blocks to the side-center region Current result: The region does not appear (unless you also add it to the mydashboard layout)
    • Workaround:
      Hide

      Fix: Change the page-layout from "mydashboard" to "mypublic"

      Line 41:

      admin_externalpage_setup('profilepage', '', null, '', array('pagelayout' => 'mypublic'));

      Show
      Fix: Change the page-layout from "mydashboard" to "mypublic" Line 41: admin_externalpage_setup( 'profilepage' , '' , null, '' , array ( 'pagelayout' => 'mypublic' ));
    • Affected Branches:
      MOODLE_27_STABLE, MOODLE_28_STABLE
    • Fixed Branches:
      MOODLE_27_STABLE, MOODLE_28_STABLE
    • Pull from Repository:
      git@github.com:ninelanterns/moodle.git
    • Pull Master Branch:
      MDL-47391_master
    • Pull Master Diff URL:

      Description

      When editing the default profile page /user/profilesys.php the layout is set to "mydashboard" instead of "mypublic" such that if you add extra block areas in the theme you cannot add blocks to these as default.

        Attachments

          Issue Links

          is a regression caused by

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-30132 Profile page layout defaults to 'mydashboard' when it should be using 'mypublic'

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

            Activity

              People

              Assignee:
              jballard James Ballard
              Reporter:
              jballard James Ballard
              Peer reviewer:
              Mary Evans
              Integrator:
              Sam Hemelryk
              Tester:
              Frédéric Massart
              Participants:
              Component watchers:
              Bas Brands
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                13/Nov/14