Small improvements in APPEND command handling

Aleksander Machniak

2012-08-31 b5652641be7ebf4f1b7f115333989d2c971275ea

 program/include/rcube_imap_generic.php

@@ -2377,7 +2377,7 @@
        return $this->handlePartBody($mailbox, $id, $is_uid, $part);
    }

    function handlePartBody($mailbox, $id, $is_uid=false, $part='', $encoding=NULL, $print=NULL, $file=NULL)
    function handlePartBody($mailbox, $id, $is_uid=false, $part='', $encoding=NULL, $print=NULL, $file=NULL, $formatted=true)
    {
        if (!$this->select($mailbox)) {
            return false;
@@ -2494,7 +2494,7 @@
                        continue;
                    $line = convert_uudecode($line);
                // default
                } else {
                } else if ($formatted) {
                    $line = rtrim($line, "\t\r\n\0\x0B") . "\n";
                }

@@ -2538,7 +2538,7 @@
    {
        unset($this->data['APPENDUID']);

        if (!$mailbox) {
        if ($mailbox === null || $mailbox === '') {
            return false;
        }

@@ -2603,7 +2603,7 @@
    {
        unset($this->data['APPENDUID']);

        if (!$mailbox) {
        if ($mailbox === null || $mailbox === '') {
            return false;
        }

@@ -2612,6 +2612,7 @@
        if (file_exists(realpath($path))) {
            $in_fp = fopen($path, 'r');
        }

        if (!$in_fp) {
            $this->setError(self::ERROR_UNKNOWN, "Couldn't open $path for reading");
            return false;
@@ -3202,10 +3203,10 @@
     */
    static function getStructurePartData($structure, $part)
    {
       $part_a = self::getStructurePartArray($structure, $part);
       $data   = array();
        $part_a = self::getStructurePartArray($structure, $part);
        $data   = array();

       if (empty($part_a)) {
        if (empty($part_a)) {
            return $data;
        }

@@ -3238,13 +3239,13 @@

    static function getStructurePartArray($a, $part)
    {
       if (!is_array($a)) {
        if (!is_array($a)) {
            return false;
        }

        if (empty($part)) {
          return $a;
       }
            return $a;
        }

        $ctype = is_string($a[0]) && is_string($a[1]) ? $a[0] . '/' . $a[1] : '';

@@ -3252,20 +3253,17 @@
            $a = $a[8];
        }

       if (strpos($part, '.') > 0) {
          $orig_part = $part;
          $pos       = strpos($part, '.');
          $rest      = substr($orig_part, $pos+1);
          $part      = substr($orig_part, 0, $pos);
        if (strpos($part, '.') > 0) {
            $orig_part = $part;
            $pos       = strpos($part, '.');
            $rest      = substr($orig_part, $pos+1);
            $part      = substr($orig_part, 0, $pos);

          return self::getStructurePartArray($a[$part-1], $rest);
       }
            return self::getStructurePartArray($a[$part-1], $rest);
        }
        else if ($part > 0) {
          if (is_array($a[$part-1]))
                return $a[$part-1];
          else
                return $a;
       }
            return (is_array($a[$part-1])) ? $a[$part-1] : $a;
        }
    }

    /**