New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Copy-Item used as transfer between databases should include blob data for media items #782

Closed
michaellwest opened this Issue Jan 8, 2017 · 6 comments

Comments

Projects
None yet
3 participants
@michaellwest
Member

michaellwest commented Jan 8, 2017

Discovered that the following command will not transfer blob data.

Copy-Item -Path "master:/media library/Images/SPE" -Destination "web:/media library/Images/" -TransferOptions 0 -Recurse
orce

Result:
image

@michaellwest

This comment has been minimized.

Member

michaellwest commented Jan 8, 2017

@MichaelBosch Thank you for bringing this issue to our attention. Turns out the functionality was completely missing from the Copy-Item command. I've now been able to address it.

@michaellwest

This comment has been minimized.

Member

michaellwest commented Jan 8, 2017

Fixed.

@MichaelBosch

This comment has been minimized.

MichaelBosch commented Jan 9, 2017

Thanks for the quick feedback and fix!

@AdamNaj AdamNaj added this to the 4.3.1 milestone Jan 10, 2017

@AdamNaj

This comment has been minimized.

Member

AdamNaj commented Jan 21, 2017

What's the verdict @michaellwest? Are we compatible with all Sitecore versions now?

@michaellwest

This comment has been minimized.

Member

michaellwest commented Jan 22, 2017

Yes. I'm happy to announce that the Copy-Item unit test made the verification pain free.

Validated on SC 7.0 rev 140408

  1. Installed SPE 4.3 (had to disable Cognifide.PowerShell.Events.config)
  2. Ran unit test which failed on the blob check
  3. Copied debug binary for 7.0
  4. Ran unit test which passed on the blob check
@AdamNaj

This comment has been minimized.

Member

AdamNaj commented Jan 31, 2017

I would like to avoid bugs being part of the release-highlight. Changing the qualification to improvement.

@AdamNaj AdamNaj closed this Jan 31, 2017

@AdamNaj AdamNaj added improvement and removed bug labels Jan 31, 2017

@AdamNaj AdamNaj changed the title from Copy-Item does not transfer blob data for media items to Copy-Item used as transfer between databases should include blob data for media items Jan 31, 2017

@michaellwest michaellwest removed their assignment May 23, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment