Skip to content
This repository has been archived by the owner on Sep 20, 2024. It is now read-only.

Photoshop: remove trailing underscore in subset name #5647

Merged
merged 1 commit into from
Sep 25, 2023

Conversation

kalisp
Copy link
Member

@kalisp kalisp commented Sep 25, 2023

Changelog Description

If {layer} placeholder is at the end of subset name template and not used (for example in auto_image where separating it by layer doesn't make any sense) trailing '_' was kept. This updates cleaning logic and extracts it as it might be similar in regular image instance.

Testing notes:

1.Configure project_settings/global/tools/creator/subset_name_profiles set something like {family}_{Task}_{Variant}_{layer}
2. try to publish from fresh new workfile

If {layer} placeholder is at the end of subset name template and not used (for example in auto_image where separating it by layer doesn't make any sense) trailing '_' was kept.
This updates cleaning logic and extracts it as it might be similar in regular `image` instance.
@ynbot
Copy link
Contributor

ynbot commented Sep 25, 2023

Task linked: OP-6874 Auto_image template name

@ynbot ynbot added size/XS Denotes a PR changes 0-99 lines, ignoring general files type: bug Something isn't working host: Photoshop labels Sep 25, 2023
@kalisp kalisp merged commit 1bd07bd into develop Sep 25, 2023
13 checks passed
@kalisp kalisp deleted the bugfix/OP-6874_Auto_image-template-name branch September 25, 2023 16:01
@ynbot ynbot added this to the next-patch milestone Sep 25, 2023
santi-svirsky pushed a commit to santi-svirsky/OpenPype that referenced this pull request Oct 8, 2023
If {layer} placeholder is at the end of subset name template and not used (for example in auto_image where separating it by layer doesn't make any sense) trailing '_' was kept.
This updates cleaning logic and extracts it as it might be similar in regular `image` instance.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
host: Photoshop size/XS Denotes a PR changes 0-99 lines, ignoring general files type: bug Something isn't working
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants