Skip to content
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

MBS-9675, MBS-9676: Fix work attributes localization issues in the work creation dialog #675

Merged
merged 2 commits into from May 30, 2018

Conversation

yvanzo
Copy link
Contributor

@yvanzo yvanzo commented May 29, 2018

Fix both bugs MBS-9675 and MBS-9676 for good

Cause was translation domain attributes was not loaded from the work creation dialog (opened from the release relationship editor), whereas it is required for adding new works when no existing work is found.

It reverts change from PR #673, and load gettext domain “attributes” for work creation dialog specifically.

which did not work as work creation dialog is sandboxed in iframe.
SET gettext_domains = [];
END;
IF dialog_template == "work/edit_form.tt";
SET gettext_domains.push('attributes');
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think SET doesn't make sense there since it's not an assignment. (SET is optional for assignments, even.)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oops, removed in 43f032e

in work creation dialog specifically.
@yvanzo yvanzo merged commit 21c751d into metabrainz:master May 30, 2018
@yvanzo yvanzo deleted the mbs-9676-mbs-9675 branch July 26, 2018 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants