Skip to content

Commit

Permalink
Fix regenerate scoping (#2051)
Browse files Browse the repository at this point in the history
* New scripts are still considered owned by the owner even if they aren't created yet

Post #1930 #1851
  • Loading branch information
Martii authored Feb 24, 2025
1 parent 74d1846 commit 28f624b
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions views/includes/scripts/clipboard.html
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,7 @@
var rMin = /\.min\.js$/;

{{#isScriptViewSourcePage}}
{{#isOwner}}
var regenerateId = '#regenerate';
{{/isOwner}}
{{/isScriptViewSourcePage}}

var allowedIds = [
Expand All @@ -33,7 +31,6 @@
}

{{#isScriptViewSourcePage}}
{{#isOwner}}
// TODO: Should keep in sync with all peg.js upmixes eventually
function parseMeta(aString) {
aString = aString.toString();
Expand Down Expand Up @@ -91,7 +88,6 @@

return cleanName || aDefaultName;
};
{{/isOwner}}
{{/isScriptViewSourcePage}}

//
Expand All @@ -113,7 +109,6 @@
}

{{#isScriptViewSourcePage}}
{{#isOwner}}
editor = ace.edit("editor");

$(regenerateId).on('click', function () {
Expand Down Expand Up @@ -229,7 +224,6 @@

$('#alt-install-target-btn').attr('disabled', 'disabled');
});
{{/isOwner}}
{{/isScriptViewSourcePage}}

})();
Expand Down

0 comments on commit 28f624b

Please sign in to comment.