Thomas Bruederli
2012-07-24 2d2af72c22e7b3aca2f90f10bcda1c73767d9656
Correctly quote localized labels when used in javascript variables (#1488567)
3 files modified
13 ■■■■ changed files
CHANGELOG 1 ●●●● patch | view | raw | blame | history
program/include/rcube_template.php 10 ●●●●● patch | view | raw | blame | history
skins/larry/includes/footer.html 2 ●●● patch | view | raw | blame | history
CHANGELOG
@@ -1,6 +1,7 @@
CHANGELOG Roundcube Webmail
===========================
- Correctly escape localized labels in javascript variable (#1488567)
- Update Net_SMTP/Auth_SASL packages to fix Digest-MD5/Cram-MD5 authentication (#1488571)
- Don't add attachments content into reply/forward/draft message body (#1488557)
- Fix 'no connection' errors on page unloads (#1488547)
program/include/rcube_template.php
@@ -711,7 +711,15 @@
                    $vars = $attrib + array('product' => $this->config['product_name']);
                    unset($vars['name'], $vars['command']);
                    $label = rcube_label($attrib + array('vars' => $vars));
                    return !$attrib['noshow'] ? (get_boolean((string)$attrib['html']) ? $label : Q($label)) : '';
                    $quoting = !empty($attrib['quoting']) ? strtolower($attrib['quoting']) : (get_boolean((string)$attrib['html']) ? 'no' : '');
                    switch ($quoting) {
                        case 'no':
                        case 'raw': break;
                        case 'javascript':
                        case 'js': $label = JQ($label); break;
                        default:   $label = Q($label); break;
                    }
                    return !$attrib['noshow'] ? $label : '';
                }
                break;
skins/larry/includes/footer.html
@@ -3,7 +3,7 @@
// UI startup
var UI = new rcube_mail_ui();
$(document).ready(function(){
    UI.set('errortitle', '<roundcube:label name="errortitle" />');
    UI.set('errortitle', '<roundcube:label name="errortitle" quoting="javascript" />');
    UI.init();
});