wikimedia/mediawiki-extensions-VisualEditor
1
0
Fork 0
mirror of https://gerrit.wikimedia.org/r/mediawiki/extensions/VisualEditor synced 2024-11-15 10:35:48 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge "ve.dm.MWTemplateModel: Never remove empty required parameters"

Browse source
This commit is contained in:
jenkins-bot 2021-03-15 11:07:29 +00:00 committed by Gerrit Code Review
parent b0b35cd8ac 989792ac7f
commit 35a573f3c7
1 changed files with 9 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
modules/ve-mw/dm/models/ve.dm.MWTemplateModel.js
View file

@ -339,6 +339,7 @@ ve.dm.MWTemplateModel.prototype.serialize = function () {
origData = this.originalData || {},
origParams = origData.params || {},
template = { target: this.getTarget(), params: {} },
spec = this.getSpec(),
params = this.getParameters();
for ( name in params ) {
@ -346,8 +347,14 @@ ve.dm.MWTemplateModel.prototype.serialize = function () {
continue;
}
// Don't add empty parameters if they were not present before the edit (T101075)
if ( params[ name ].getValue() === '' && !Object.prototype.hasOwnProperty.call( origParams, name ) ) {
if (
// Don't add empty parameters (T101075)
params[ name ].getValue() === '' &&
// …unless they were present before the edit
!Object.prototype.hasOwnProperty.call( origParams, name ) &&
// …unless they are required (T276989)
!( spec.isParameterKnown( name ) && spec.isParameterRequired( name ) )
) {
continue;
}