Fix Closure-compiler warnings, small code improvements

Aleksander Machniak

2013-12-18 538e64c572220bba43960a3073468c0ce3dc5f4c

 plugins/archive/archive.js

@@ -1,4 +1,4 @@
/*
/**
 * Archive plugin script
 * @version 2.1
 */
@@ -8,7 +8,7 @@
  if (!rcmail.env.uid && (!rcmail.message_list || !rcmail.message_list.get_selection().length))
    return;

  if (rcmail_is_archive()) {
  if (!rcmail_is_archive()) {
    if (!rcmail.env.archive_type) {
      // simply move to archive folder (if no partition type is set)
      rcmail.command('move', rcmail.env.archive_folder);
@@ -22,12 +22,9 @@

function rcmail_is_archive()
{
  if (!rcmail.env.archive_folder)
    return false;

  // check if current folder is an archive folder or one of its children
  if (rcmail.env.mailbox == rcmail.env.archive_folder
    || rcmail.env.mailbox.indexOf(rcmail.env.archive_folder + rcmail.env.delimiter) != 0
    || rcmail.env.mailbox.startsWith(rcmail.env.archive_folder + rcmail.env.delimiter)
  ) {
    return true;
  }
@@ -37,12 +34,12 @@
if (window.rcmail) {
  rcmail.addEventListener('init', function(evt) {
    // register command (directly enable in message view mode)
    rcmail.register_command('plugin.archive', rcmail_archive, rcmail.env.uid && rcmail_is_archive());
    rcmail.register_command('plugin.archive', rcmail_archive, rcmail.env.uid && !rcmail_is_archive());

    // add event-listener to message list
    if (rcmail.message_list)
      rcmail.message_list.addEventListener('select', function(list) {
        rcmail.enable_command('plugin.archive', list.get_selection().length > 0 && rcmail_is_archive());
        rcmail.enable_command('plugin.archive', list.get_selection().length > 0 && !rcmail_is_archive());
      });

    // set css style for archive folder
@@ -53,7 +50,7 @@
    // callback for server response
    rcmail.addEventListener('plugin.move2archive_response', function(result) {
      if (result.update)
        rcmail.command('checkmail');  // refresh list
        rcmail.command('list');  // refresh list
    });
  })
}