Osclass forums
Support forums => General help => Topic started by: jack daniels on March 31, 2010, 10:42:11 pm
-
I have built a simple CSS dropdown horizontal navigation, user scrolls over categories and the dropdown shows the respective sub-cats, (nice for webmasters with a smaller number of sub-cats)
Works great on the homepage, but with it being horizontal im needing it to be either hardcoded into the header or pulled into the header as a serverside include from a separate file nav.php for example so it is included on every page..
Hardcoding into the header or using a serverside include both throw out these errors
Notice: Undefined variable: categories in C:\wamp\www\oscom\oc-content\themes\simply\header.php on line 124
Warning: Invalid argument supplied for foreach() in C:\wamp\www\oscom\oc-content\themes\simply\header.php on line 124
this even happens if i cut and copy the dropdown code from home.php and put it at the very end of the header.php code, any ideas?
thanks
-
It seems that variable categories is missing.
My first guess is that $categories var is accessible only after header is generated. Fast hack will be to make following action to oc-includes/osclass/themes.php:
Replace this function:
function osc_renderHeader($headerConf = array()) {
global $preferences;
$themePath = THEMES_PATH . $preferences['theme'] . '/header.php';
if(file_exists($themePath))
require $themePath;
}
WITH
function osc_renderHeader($headerConf = array()) {
global $preferences;
global $categories;
$themePath = THEMES_PATH . $preferences['theme'] . '/header.php';
if(file_exists($themePath))
require $themePath;
}
this should make a trick.
-
Thanks for taking the time Max P, that works a treat... almost
The good thing is i can now use a serverside include and i can also add the code to the bottom of the header and it works, the downside is it only works when on the themes home page when i navigate away (item.php) for example it then returns the
Warning: Invalid argument supplied for foreach() in C:\wamp\www\oscom\oc-content\themes\simply\header.php on line 124
which refers to this line
<?php foreach($categories as $c): ?>
-
This is because item.php has $categories = Category::newInstance()->toTree(); inside post action. Try to move it to the top after line 17: osc_dbAutoConnect(); :)
Hope this helps.
I think we will have to do some changes to the code to make it work. I bet you are not the only one who wants to have categories available in header!
-
My mistake Max P, item.php works fine now without any changes, it is category.php where the error shows
Horizontal would be a nice option for a site with fewer sub-cats, possibly further down the development line have a checkbox in ACP for either Horizontal or Vertical (vertical is better for more sub-cats with-out them flying of screen)
-
You can try to add $categories = Category::newInstance()->toTree(); to category.php and it should work fine after that.
-
Success! Thankyou very much for your input
Would not work from category.php, however i dropped the line of code into my modified header.php above <?php foreach($categories as $c): ?> and all is good and working as it should do,
I will upload a demo of it to my server for you to look at later next week (Easter holidays and things coming up) in the meantime i will check for cross-browser compliancy and SEO a few things in the themes code.
A point i would like to make is because this theme use's a horizontal menu the homepage is bare of content, but im sure this will improve as the project develops..
-
Just in case anybody else is interested i also had to add this line into ../user.php as the registration page would not show the menu
"require_once 'osclass/model/Category.php';"