loleaflet/dist/toolbar/toolbar.js |    6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

New commits:
commit 870b46be78a3fd09384d7e8e3e74d02cfe66614e
Author: Pranav Kant <pran...@collabora.com>
Date:   Fri May 13 00:00:49 2016 +0530

    bccu#1756: Check for empty input
    
    In case of presentation we are getting an empty value in response
    to .uno:CharFontName. While this looks like a bug in LOKit, no harm
    in putting a check for empty values here.
    
    In ES5 argument to Object.keys must be an object otherwise it
    throws an exception.
    
    Change-Id: I5c2f7963544fda55bab3f314f1b098b3c9e3bd0b

diff --git a/loleaflet/dist/toolbar/toolbar.js 
b/loleaflet/dist/toolbar/toolbar.js
index e074f7b..4b516d3 100644
--- a/loleaflet/dist/toolbar/toolbar.js
+++ b/loleaflet/dist/toolbar/toolbar.js
@@ -786,7 +786,11 @@ map.on('updatetoolbarcommandvalues', function (e) {
                $(".styles-select").on('select2:select', onStyleSelect);
        }
        else if (e.commandName === '.uno:CharFontName') {
-               data = data.concat(Object.keys(e.commandValues));
+               // Old browsers like IE11 et al don't like Object.keys with
+               // empty arguments
+               if (typeof e.commandValues === 'object') {
+                       data = data.concat(Object.keys(e.commandValues));
+               }
                $(".fonts-select").select2({
                        data: data,
                        placeholder: _("Font")
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to