Fixed Samba Move Object

[duygupotur]

This pr fixes #194

[mertcelen]

We need this pr to merged ASAP.

[stefanmcshane]

A few minor changes to be made before approval
Tag me and I will help get this approved for you asap

[mertcelen]

@stefanmcshane changes applied :)

[stefanmcshane]

@mertcelen Still a few failing tests - all PRs are blocked until the tests pass

[mertcelen]

@stefanmcshane you are absolutely right to ask for tests but if you check closer, there seems to be a bug with older go version tests itself. If I'm not wrong, it has nothing to do with our change. I'll gladly help if that's the case but since you flagged them as legacy and google obviously dropped support for it https://golang.org/doc/devel/release.html#policy we may as well just delete those.

Your call.

[stefanmcshane]

@mertcelen Whilst I wouldn't use Go's release history as an argument as the scopes are very different, I would agree that we should review the supported legacy versions going forward. This would need to be done as a separate PR, before merging this change.

@johnweldon Opinions on this? We would be potentially breaking backwards compatibility for any Go versions <= 1.6 (released 2016, support dropped 2017).
However, I would suggest dropping support for 1.5 and 1.6 in a new PR, tagging as a hotfix release (v3.2.4), then merging this PR.
EDIT: Issue raised: #287, with PR in #288

Aside: This would also provide a strong segue for tidying go modules up further, by dropping support for non-module installation, and making the new default branch "v3". If you agree to the above, I'll open a new issue for discussion on go module related updates.

[stefanmcshane]

@mertcelen @ZDuygu This has been tagged v3.2.4 and released. Thank you for your contributions and patience!