Collection of small fixes

[pmolodo]

Description (this won't be part of the changelog)

This branches off of https://github.com/AnimalLogic/AL_USDMaya/pull/12, so may want to check that PR first...

Changelog

Changed

• Some c++11 style foreach loops changed to iterate over refs, instead of values, to avoid copy
• Replaced removeVariantFromPath in ProxyShapeUI with standard SdfPath:: StripAllVariantSelections
• Removed an unused opIt var from TransformTranslator
• ProxyShapeSelection - avoided re-assignment of helper.m_paths = m_selectedPaths on every loop
• Removed unused if-branch in ProxyDrawOverride::userSelect

Fixed

• Fixes bug where, in UFE mode, if had a USD hierarchy without any Xform nodes, it would select the root anytime you clicked on anything
• Made sure SelectionEnded event was triggered in toggle mode, even if no changes made

Checklist (Please do not remove this line)

• [X] Make sure the Title and Description of the PR make sense and provide sufficient context for your work
• [X] Do any added files have the correct AL Apache Licence Header?
• [X] Are there Doxygen comments in the headers?
• [X] Are any new features, behaviour changes documented in the .md format documentation?
• [ ] Have you added, updated tests to cover new features and behaviour changes?
• [X] Have you filled out at least one changelog entry?

[b0ssi]

Hi @elrond79 - we decided to merge #133 and this PR separately, so we will probably drop #143 (thanks for preparing that one though!). Would you mind resolving the conflict in here again so we can merge this on its own? Cheers

[pmolodo]

Hey - I just pushed the merge commit from #143 onto this branch, so it's all here for tracking... is that what you wanted? Or did you mean you wanted me to do a rebase?

[fabal]

Hi @elrond79 , yes if you could rebase (or squash) your work against develop that would be much appreciated (we're ready to merge it).

[pmolodo]

Ok, rebased!