U4-3838 - ContentService Copy uses IOHelper.MapPath but should use fs.GetFullPath

Created by Dirk 11 Dec 2013, 18:18:17 Updated by Dirk 28 Jun 2014, 15:36:09

Within Umbraco.Core.Services.ContentService: public IContent Copy(IContent content, int parentId, bool relateToOriginal, int userId = 0){ ... Instead of if (fs.FileExists(IOHelper.MapPath(property.Value.ToString()))) you should use if (fs.FileExists(fs.GetFullPath(property.Value.ToString()))).

Otherwise custom file system providers like the Azure Blob Storage Provider will fail to copy content nodes.

Comments

Sebastiaan Janssen 27 Jun 2014, 15:23:17

Commit made by '''Sebastiaan Janssen''' on ''2014-06-27T17:22:51+02:00'' https://github.com/umbraco/Umbraco-CMS/commit/5bee398d09adb0023d676496efc3933a381c8df6

#U4-3838 fixed Due in version 7.1.5

Fixes ContentService Copy uses IOHelper.MapPath but should use fs.GetFullPath and a few other instances where IOHelper was used


Dirk 28 Jun 2014, 15:36:09

Looks good H5YR!


Priority: Normal

Type: Bug

State: Fixed

Assignee:

Difficulty: Normal

Category:

Backwards Compatible: True

Fix Submitted: Inline code

Affected versions:

Due in version: 7.1.5

Sprint:

Story Points:

Cycle: