Aleksander Machniak
2014-04-13 8b93fc92845d13e17795b9b3a52cdeb81e5340c5
Use POST for refresh/check-recent request

Conflicts:

program/steps/mail/check_recent.inc
2 files modified
12 ■■■■■ changed files
program/js/app.js 4 ●●●● patch | view | raw | blame | history
program/steps/mail/check_recent.inc 8 ●●●●● patch | view | raw | blame | history
program/js/app.js
@@ -2151,7 +2151,7 @@
    var lock = this.set_busy(true, 'checkingmail'),
      params = this.check_recent_params();
    this.http_request('check-recent', params, lock);
    this.http_post('check-recent', params, lock);
  };
  // list messages of a specific mailbox using filter
@@ -7330,7 +7330,7 @@
    this.env.lastrefresh = new Date();
    // plugins should bind to 'requestrefresh' event to add own params
    this.http_request('refresh', params, lock);
    this.http_post('refresh', params, lock);
  };
  // returns check-recent request parameters
program/steps/mail/check_recent.inc
@@ -21,7 +21,7 @@
// If there's no folder or messages list, there's nothing to update
// This can happen on 'refresh' request
if (empty($_REQUEST['_folderlist']) && empty($_REQUEST['_list'])) {
if (empty($_POST['_folderlist']) && empty($_POST['_list'])) {
    return;
}
@@ -73,14 +73,16 @@
            $_SESSION['search'] = $RCMAIL->storage->refresh_search();
        }
        if (!empty($_GET['_quota']))
        if (!empty($_POST['_quota'])) {
            $OUTPUT->command('set_quota', $RCMAIL->quota_content());
        }
        $OUTPUT->set_env('exists', $RCMAIL->storage->count($mbox_name, 'EXISTS'));
        // "No-list" mode, don't get messages
        if (empty($_GET['_list']))
        if (empty($_POST['_list'])) {
            continue;
        }
        // get overall message count; allow caching because rcube_storage::folder_status()
        // did a refresh but only in list mode