Roundcube Community Forum

Release Support => Requests => Topic started by: jasonq on July 05, 2009, 02:46:13 PM

Title: Stripslashes in Page title
Post by: jasonq on July 05, 2009, 02:46:13 PM

Obviously (from my earlier post) I am just starting to play around with round cube, considering deploying it on a number of my servers...so I may post a few more proposed patches...

I noticed that the page title does not strip the ' character, and since the installation that I'm playing around with has been named Jason Quick's webmail, that was annoying.

I found the set_pagetitle method in /program/include/rcube_json_output.php does not use stripcslashes or anything, so I modified it:


 /**
     * Issue command to set page title
     *
     * @param string New page title
     */
    public function set_pagetitle($title)
    {
        $name = $this->config->get('product_name');
        $this->command('set_pagetitle', JQ(empty($name) ? $title : stripcslashes($name).' :: '.$title));
    }



There may be a more elegant way to do this further up in the API, but this was a quick fix.

Thanks,
Jason Quick