Results 1 to 3 of 3

Thread: wrapper.php - is this buggy coded?

  1. #1
    Thread Starter
    trude76's Avatar
    Join Date
    Dec 2012
    Location
    Leipzig, Germany
    Posts
    785
    Thanks
    1
    Thanked 61 Times in 57 Posts

    wrapper.php - is this buggy coded?

    Hi,

    during testing and changing some files in latest dev version i wondered and stumbled a little by following code in wrapper.php

    Code:
    <body id="top" <?php
    	// TODO: use APP_VIEW instance from framework
    	$classes = '';
    	if (get_option('jr_show_sidebar')=='no') $classes .= 'wider ';
    	if (get_option('jr_child_theme')) $classes .= str_replace('.css','',get_option('jr_child_theme')).' ';
    	body_class( $classes );
    	?>>
    I do not know that much like an expert - but just by thinking based on my little coding knowledge I would guess it should be more like

    Code:
    <body id="top"> <?php
    	// TODO: use APP_VIEW instance from framework
    	$classes = '';
    	if (get_option('jr_show_sidebar')=='no') $classes .= 'wider ';
    	if (get_option('jr_child_theme')) $classes .= str_replace('.css','',get_option('jr_child_theme')).' ';
    	body_class( $classes );
    	?>
    Or am I go wrong in this case?

  2. #2
    Thread Starter
    trude76's Avatar
    Join Date
    Dec 2012
    Location
    Leipzig, Germany
    Posts
    785
    Thanks
    1
    Thanked 61 Times in 57 Posts
    You must be an AppThemes customer and logged in to view this response. Join today!

  3. #3
    jomarkosabel's Avatar
    Join Date
    Mar 2009
    Location
    Philippines
    Posts
    35,572
    Thanks
    158
    Thanked 2,836 Times in 2,714 Posts
    You must be an AppThemes customer and logged in to view this response. Join today!
    Please help our moderating team work more efficiently by not sending us support questions via PM. You can read more about how AppThemes support works here. However, you can send a PM to follow up and remind me if I missed your support request/thread.

    Thank you and have a nice day.

Thread Information

Users Browsing this Thread

There are currently 1 users browsing this thread. (0 members and 1 guests)

Similar Threads

  1. Icons not being Color Coded
    By jobspex in forum Mapsupreme JobRoller
    Replies: 5
    Last Post: March 12th, 2013, 12:27 PM
  2. [SOLVED] Buggy Clipper?
    By mahamadoca in forum Report Clipper Bugs
    Replies: 15
    Last Post: February 13th, 2012, 06:13 AM
  3. Free/Paid Buggy
    By 405media in forum Report ClassiPress Bugs
    Replies: 1
    Last Post: July 13th, 2010, 03:46 AM
  4. Editing Live Ad Buggy
    By johare in forum Report ClassiPress Bugs
    Replies: 2
    Last Post: April 6th, 2010, 05:19 PM