Fix so JSON.parse() errors on localStorage items are ignored (#1490249)

Aleksander Machniak

2015-02-06 56040b941c20d1d2ed2abf3f1993ea550bc13086

 program/js/app.js

@@ -1954,7 +1954,7 @@
    // attach events
    $.each(fn, function(i, f) {
      row[i].onclick = function(e) { f(e); return rcube_event.cancel(e); };
      if (bw.touch) {
      if (bw.touch && row[i].addEventListener) {
        row[i].addEventListener('touchend', function(e) {
          if (e.changedTouches.length == 1) {
            f(e);
@@ -2484,6 +2484,16 @@
        selection.push(selected[i]);

    this.message_list.selection = selection;

    // reset preview frame, if currently previewed message is not selected (has been removed)
    try {
      var win = this.get_frame_window(this.env.contentframe),
        id = win.rcmail.env.uid;

      if (id && $.inArray(id, selection) < 0)
        this.show_contentframe(false);
    }
    catch (e) {};
  };

  // expand all threads with unread children
@@ -3971,7 +3981,6 @@
    this.local_storage_remove_item('compose.index');
  };


  this.change_identity = function(obj, show_sig)
  {
    if (!obj || !obj.options)
@@ -4340,6 +4349,7 @@
      (this.env.search_request && (this.env.search_scope || 'base') != 'base');
  };

  // action executed after mail is sent
  this.sent_successfully = function(type, msg, folders)
  {
    this.display_message(msg, type);
@@ -4348,11 +4358,13 @@
    if (this.env.extwin) {
      this.lock_form(this.gui_objects.messageform);

      var rc = this.opener();
      var filter = {task: 'mail', action: ''},
        rc = this.opener(false, filter) || this.opener(true, filter);

      if (rc) {
        rc.display_message(msg, type);
        // refresh the folder where sent message was saved or replied message comes from
        if (folders && rc.env.task == 'mail' && rc.env.action == '' && $.inArray(rc.env.mailbox, folders) >= 0) {
        if (folders && $.inArray(rc.env.mailbox, folders) >= 0) {
          rc.command('checkmail');
        }
      }
@@ -5161,10 +5173,10 @@
        dateFormat: this.env.date_format,
        changeMonth: true,
        changeYear: true,
        yearRange: '-100:+10',
        yearRange: '-120:+10',
        showOtherMonths: true,
        selectOtherMonths: true,
        onSelect: function(dateText) { $(this).focus().val(dateText) }
        selectOtherMonths: true
//        onSelect: function(dateText) { $(this).focus().val(dateText); }
      });
      $('input.datepicker').datepicker();
    }
@@ -8147,15 +8159,16 @@
  // wrapper for localStorage.getItem(key)
  this.local_storage_get_item = function(key, deflt, encrypted)
  {
    var item;
    var item, result;

    // TODO: add encryption
    try {
      item = localStorage.getItem(this.get_local_storage_prefix() + key);
      result = JSON.parse(item);
    }
    catch (e) { }

    return item !== null ? JSON.parse(item) : (deflt || null);
    return result || deflt || null;
  };

  // wrapper for localStorage.setItem(key, data)