From 3d9e12baf43535c82e41771138307ceb61630309 Mon Sep 17 00:00:00 2001
From: CheeseSucker <cheesesucker@gmail.com>
Date: Fri, 14 Jun 2013 18:30:09 +0200
Subject: [PATCH] ENHANCEMENT: Allow programmers to set OwnerID for new files.

NOTE: This change should be reviewed to make sure it does not cause any side effects.

Example use case:
    An admin porting user images from an old website.
    The script would put the images in the assets folder,
    and then insert them into the database using Image::write()
    and data from the old database.

public function insertImage($data) {
    $image = new Image();
    $image->ParentID = $data->parentId;
    $image->Title = $data->title;
    $image->FileName = $data->filename;
    $image->OwnerID = $data->ownerId;
    $image->write();

    // In the current version, this results in all images
    // being owned by Member::currentUser() instead of
    // the expected $data->ownerId;
}
---
 filesystem/File.php | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/filesystem/File.php b/filesystem/File.php
index d970294e8..026a7bf63 100644
--- a/filesystem/File.php
+++ b/filesystem/File.php
@@ -466,7 +466,9 @@ class File extends DataObject {
 		parent::onBeforeWrite();
 
 		// Set default owner
-		if(!$this->ID) $this->OwnerID = (Member::currentUser() ? Member::currentUser()->ID : 0);
+		if(!$this->ID && !$this->OwnerID) {
+			$this->OwnerID = (Member::currentUser() ? Member::currentUser()->ID : 0);
+		}
 
 		// Set default name
 		if(!$this->getField('Name')) $this->Name = "new-" . strtolower($this->class);