Show logs - minor bug

Discussion in 'Developers' Forum' started by mike_p, May 27, 2010.

  1. mike_p

    mike_p Member

    Version 3.0.2.1: I note that the show log pages in the monitor section don't have headings.

    Checking the source of show_log.php the title and description are set by
    Code:
    $app->tpl->setVar("title", $title);
    ...
    $app->tpl->setVar("description", $description);
    
    whereas show_data.php uses
    Code:
    $app->tpl->setVar("list_head_txt", $title);
    $app->tpl->setVar("list_desc_txt", $description);
    
    Before I submit a change to SVN does anyone know if removing the title and description vars and replacing them with the list_head_txt and list_desc_txt might cause a problem?
    Obviously I have tested on my server and found no errors, but ....?

    I want to squeeze this fix in before the next release (which looks almost ready to run?)
     
  2. till

    till Super Moderator Staff Member ISPConfig Developer

    If it works on your system, then feel free to replace the vars.
     
  3. mike_p

    mike_p Member

    Consider it done! :p
     

Share This Page