Skip to content

Commit 60ce54d

Browse files
authored
Merge pull request metabrainz#2187 from reosarevok/MBS-11811
MBS-11811: Do not batch-change track data for collapsed media
2 parents a869aeb + 5e35e04 commit 60ce54d

File tree

3 files changed

+6
-3
lines changed

3 files changed

+6
-3
lines changed

root/release/edit/tracklist.tt

+1-1
Original file line numberDiff line numberDiff line change
@@ -498,7 +498,7 @@
498498
<!-- /ko -->
499499

500500
<!-- ko if: mediums() && mediums().length -->
501-
<div data-bind="guessCase: $root.guessCaseMediaNames.bind($data)">
501+
<div data-bind="guessCase: $root.guessCaseAllMedia.bind($data)">
502502
[% guesscase(show_icon=1) %]
503503
</div>
504504
<!-- /ko -->

root/static/scripts/release-editor/actions.js

+4-2
Original file line numberDiff line numberDiff line change
@@ -127,10 +127,12 @@ const actions = {
127127
}
128128
},
129129

130-
guessCaseMediaNames: function () {
130+
guessCaseAllMedia: function () {
131131
for (const medium of this.mediums.peek()) {
132132
releaseEditor.guessCaseMediumName(medium);
133-
releaseEditor.guessCaseTrackNames(medium);
133+
if (!medium.collapsed.peek()) {
134+
releaseEditor.guessCaseTrackNames(medium);
135+
}
134136
}
135137
},
136138

root/static/scripts/release-editor/bindingHandlers.js

+1
Original file line numberDiff line numberDiff line change
@@ -99,6 +99,7 @@ ko.bindingHandlers.artistCreditEditor = {
9999
const artistCredit = track.artistCredit.peek();
100100

101101
track.medium.release.mediums()
102+
.filter(m => !m.collapsed.peek())
102103
.flatMap(m => m.tracks())
103104
.forEach(function (t) {
104105
if (t === track) {

0 commit comments

Comments
 (0)