Themes

Hi, 

Well after looking at the header.php I am not surprised its not doing what you want it to.

There are some better ways to code it if you use Moodles Coding Guidelines.

Can you email me the theme itself and I'll have a better chance of finding a solution.

email-addressremoved

Thanks

Mary

Hi,

Thanks for sending me the zip file of your theme.

I'll look at this later today and let you have some feedback...hopefully!

Cheers

Mary

OK...first bit of Feedback.

I don't see the required version.php for this theme, so how is this showing up in Moodle 3.2?  Technically it should not be working.

Also you have blocked some JavaScript files in the config.php, if this is to stop the files from working because you are using YUI JS in a js directory then this, again, is not really the Moodle way of doing things.

There are lots of places within the structure of your theme that could be causing it not to function as you want it to.

For a child of a child theme this is, sorry to say,  a really complicated set of bad coding.

You might as well have cloned EduHub theme and used that instead of creating a Frankenstein version of a theme!

Mary

Hi Vysakh,

Here is some more feedback for you concerning the files you need:

All the files you need are the ones listed below in this part of your themes config.php

$THEME->layouts = array(
// The site home page.
        'frontpage' => array(
        'file' => 'frontpage.php',
        'regions' => array('side-post'),
        'defaultregion' => 'side-post',
        'options' => array('nonavbar' => true),
    ),
    'mypublic' => array(
        'file' => 'profile.php',
        'regions' => array('side-post'),
        'defaultregion' => 'side-post',
        'options' => array('nonavbar' => true),
    ),
    'embedded' => array(
        'file' => 'embedded.php',
        'regions' => array()
    ),
    'standard' => array(
    'file' => 'general.php',
    'regions' => array('side-pre', 'side-post'),
    'defaultregion' => 'side-pre'
    ),
    'login' => array(
        'file' => 'login.php',
        'regions' => array(),
        'options' => array('langmenu' => true),
    ),
    'signup' => array(
        'file' => 'signup.php',
        'regions' => array(),
        'options' => array('langmenu' => true),
    ),

You can remove all of the other files unless they are needed in the coding like the header.php?

Further to this feedback:

The classes/core_renderer.php can be deleted as it is an exact copy of the eduhub core_renderer.php and so no need to have it in you child theme, as it is inherited from the parent EduHub theme

However this is not so for the following{

classes/course_renderer.php has been changed from the original eduhub/classes/course_renderer.php

so you need to change the following line:

From:

class theme_eduhub_core_course_renderer extends core_course_renderer {

To:

class theme_bentam_core_course_renderer extends core_course_renderer {

After doing all this, then you need to Purge all caches

your moodle site URL.. /admin/purgecaches.php

HOPE THIS HELPS?

Cheers

Mary