Cleaner way of handling setting of current stream into the same mime …

…part

slightly optimize that case.
Also, cleanly/correctly close the existing stream. Shouldn't make a
difference since PHP should be doing GC automatically, but best to be
explicit.

framework/Mime/lib/Horde/Mime/Part.php

@@ -423,6 +423,10 @@ public function getName($default = false)
      */
     public function setContents($contents, $options = array())
     {
+        if (is_resource($contents) && ($contents === $this->_contents)) {
+            return;
+        }
+
         if (empty($options['encoding'])) {
             $options['encoding'] = $this->_transferEncoding;
         }
@@ -431,6 +435,9 @@ public function setContents($contents, $options = array())
             ? $this->_writeStream($contents)
             : $contents;
 
+        /* Properly close the existing stream. */
+        $this->clearContents();
+
         $this->setTransferEncoding($options['encoding']);
         $this->_contents = $this->_transferDecode($fp, $options['encoding']);
     }