Don't "abuse" setters for actually "unsetting" (closing) instances. Better use

[core.git] / inc / classes / main / file_directories / class_BaseAbstractFile.php

diff --git a/inc/classes/main/file_directories/class_BaseAbstractFile.php b/inc/classes/main/file_directories/class_BaseAbstractFile.php
index 382ea6fac356c42528bc0606fff36e4ca8dc0fa0..3f4869e416029dde018bcb0feeff333672be780f 100644 (file)
--- a/inc/classes/main/file_directories/class_BaseAbstractFile.php
+++ b/inc/classes/main/file_directories/class_BaseAbstractFile.php
@@ -133,10 +133,13 @@ class BaseAbstractFile extends BaseFrameworkSystem {
         *
         * @return      void
         */
-       public function closeFile () {
+       private function closeFile () {
                // Debug message
                /* NOISY-DEBUG: */ self::createDebugInstance(__CLASS__)->debugOutput(sprintf('[%s:%d]: fileName=%s - CALLED!', __METHOD__, __LINE__, $this->getFileName()));
 
+               // Close down pointer instance as well by unsetting it
+               $this->unsetPointerInstance();
+
                // Remove file name
                $this->setFileName('');