Wrong handling of slashes when loading on Mac #324

iLLiCiTiT · 2021-04-30T12:42:30Z

Changes

all formatted paths in get_representation_path are forced to replace \ with / on all systems except windows
- this is because os.path.normpath may not replace it automatically and MacOS can't handle it automatically
don't use hierarchy key from asset document but use joined parents key

iLLiCiTiT added 3 commits April 30, 2021 14:32

force to replace backslashes with forward slashes if not on windows

2e0fe2d

skip hierarchy key and use only parents

2d0c955

added backslash check to all 3 functions returning path

b5a55af

iLLiCiTiT self-assigned this Apr 30, 2021

iLLiCiTiT requested a review from mkolar April 30, 2021 12:42

iLLiCiTiT mentioned this pull request Apr 30, 2021

Wrong handling of slashes when loading on mac ynput/OpenPype#1411

Closed

iLLiCiTiT changed the title ~~Bugfix/1411 wrong handling of slashes when loading on mac~~ Wrong handling of slashes when loading on ma Apr 30, 2021

iLLiCiTiT changed the title ~~Wrong handling of slashes when loading on ma~~ Wrong handling of slashes when loading on Mac Apr 30, 2021

iLLiCiTiT added the bug Something isn't working label Apr 30, 2021

mkolar approved these changes May 4, 2021

View reviewed changes

mkolar merged commit cfd4191 into develop May 4, 2021

mkolar deleted the bugfix/1411-wrong-handling-of-slashes-when-loading-on-mac branch May 4, 2021 14:57