Update UI to allow for non-overriding terms. Closes #2063
This commit is contained in:
parent
05839b1bbf
commit
f7a8085bab
@ -31,8 +31,10 @@
|
|||||||
"wikidata-schema/property-placeholder": "property",
|
"wikidata-schema/property-placeholder": "property",
|
||||||
"wikidata-schema/nb-references": " references",
|
"wikidata-schema/nb-references": " references",
|
||||||
"wikidata-schema/remove-column": "remove column",
|
"wikidata-schema/remove-column": "remove column",
|
||||||
"wikidata-schema/label": "Label",
|
"wikidata-schema/label-if-new": "New label",
|
||||||
"wikidata-schema/description": "Description",
|
"wikidata-schema/label-override": "Override label",
|
||||||
|
"wikidata-schema/description-if-new": "New description",
|
||||||
|
"wikidata-schema/description-override": "Override description",
|
||||||
"wikidata-schema/alias": "Alias",
|
"wikidata-schema/alias": "Alias",
|
||||||
"wikidata-schema/item-or-reconciled-column": "type item or drag reconciled column here",
|
"wikidata-schema/item-or-reconciled-column": "type item or drag reconciled column here",
|
||||||
"wikidata-schema/amount": "amount",
|
"wikidata-schema/amount": "amount",
|
||||||
|
@ -28,8 +28,10 @@
|
|||||||
"wikidata-schema/property-placeholder": "propriété",
|
"wikidata-schema/property-placeholder": "propriété",
|
||||||
"wikidata-schema/nb-references": " références",
|
"wikidata-schema/nb-references": " références",
|
||||||
"wikidata-schema/remove-column": "supprimer la colonne",
|
"wikidata-schema/remove-column": "supprimer la colonne",
|
||||||
"wikidata-schema/label": "Libellé",
|
"wikidata-schema/label-if-new": "Nouveau libellé",
|
||||||
"wikidata-schema/description": "Description",
|
"wikidata-schema/label-override": "Écraser le libellé",
|
||||||
|
"wikidata-schema/description-if-new": "Nouvelle description",
|
||||||
|
"wikidata-schema/description-override": "Écraser la description",
|
||||||
"wikidata-schema/alias": "Alias",
|
"wikidata-schema/alias": "Alias",
|
||||||
"wikidata-schema/item-or-reconciled-column": "entrer un élément ou déposer une colonne réconciliée ici",
|
"wikidata-schema/item-or-reconciled-column": "entrer un élément ou déposer une colonne réconciliée ici",
|
||||||
"wikidata-schema/amount": "quantité",
|
"wikidata-schema/amount": "quantité",
|
||||||
|
@ -441,11 +441,19 @@ SchemaAlignmentDialog._addNameDesc = function(item, json) {
|
|||||||
var type_input = $('<select></select>').appendTo(type_container);
|
var type_input = $('<select></select>').appendTo(type_container);
|
||||||
$('<option></option>')
|
$('<option></option>')
|
||||||
.attr('value', 'LABEL')
|
.attr('value', 'LABEL')
|
||||||
.text($.i18n('wikidata-schema/label'))
|
.text($.i18n('wikidata-schema/label-override'))
|
||||||
|
.appendTo(type_input);
|
||||||
|
$('<option></option>')
|
||||||
|
.attr('value', 'LABEL_IF_NEW')
|
||||||
|
.text($.i18n('wikidata-schema/label-if-new'))
|
||||||
.appendTo(type_input);
|
.appendTo(type_input);
|
||||||
$('<option></option>')
|
$('<option></option>')
|
||||||
.attr('value', 'DESCRIPTION')
|
.attr('value', 'DESCRIPTION')
|
||||||
.text($.i18n('wikidata-schema/description'))
|
.text($.i18n('wikidata-schema/description-override'))
|
||||||
|
.appendTo(type_input);
|
||||||
|
$('<option></option>')
|
||||||
|
.attr('value', 'DESCRIPTION_IF_NEW')
|
||||||
|
.text($.i18n('wikidata-schema/description-if-new'))
|
||||||
.appendTo(type_input);
|
.appendTo(type_input);
|
||||||
$('<option></option>')
|
$('<option></option>')
|
||||||
.attr('value', 'ALIAS')
|
.attr('value', 'ALIAS')
|
||||||
|
@ -37,13 +37,17 @@ EditRenderer._renderItem = function(json, container) {
|
|||||||
|
|
||||||
// Terms
|
// Terms
|
||||||
if ((json.labels && json.labels.length) ||
|
if ((json.labels && json.labels.length) ||
|
||||||
|
(json.labelsIfNew && json.labelsIfNew.length) ||
|
||||||
(json.descriptions && json.descriptions.length) ||
|
(json.descriptions && json.descriptions.length) ||
|
||||||
|
(json.descriptionsIfNew && json.descriptionsIfNew.length) ||
|
||||||
(json.addedAliases && json.addedAliases.length)) {
|
(json.addedAliases && json.addedAliases.length)) {
|
||||||
var termsContainer = $('<div></div>').addClass('wbs-namedesc-container')
|
var termsContainer = $('<div></div>').addClass('wbs-namedesc-container')
|
||||||
.appendTo(right);
|
.appendTo(right);
|
||||||
|
|
||||||
this._renderTermsList(json.labels, "label", termsContainer);
|
this._renderTermsList(json.labels, "label-override", termsContainer);
|
||||||
this._renderTermsList(json.descriptions, "description", termsContainer);
|
this._renderTermsList(json.labelsIfNew, "label-if-new", termsContainer);
|
||||||
|
this._renderTermsList(json.descriptions, "description-override", termsContainer);
|
||||||
|
this._renderTermsList(json.descriptionsIfNew, "description-if-new", termsContainer);
|
||||||
this._renderTermsList(json.addedAliases, "alias", termsContainer);
|
this._renderTermsList(json.addedAliases, "alias", termsContainer);
|
||||||
|
|
||||||
// Clear the float
|
// Clear the float
|
||||||
|
@ -242,8 +242,13 @@ public class ItemUpdate {
|
|||||||
*/
|
*/
|
||||||
@JsonIgnore
|
@JsonIgnore
|
||||||
public boolean isEmpty() {
|
public boolean isEmpty() {
|
||||||
return (addedStatements.isEmpty() && deletedStatements.isEmpty() && labels.isEmpty() && descriptions.isEmpty()
|
return (addedStatements.isEmpty() &&
|
||||||
&& aliases.isEmpty());
|
deletedStatements.isEmpty() &&
|
||||||
|
labels.isEmpty() &&
|
||||||
|
descriptions.isEmpty() &&
|
||||||
|
aliases.isEmpty() &&
|
||||||
|
labelsIfNew.isEmpty() &&
|
||||||
|
descriptionsIfNew.isEmpty());
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
@ -209,6 +209,7 @@ public class ItemUpdateTest {
|
|||||||
ItemUpdate merged = update1.merge(update2);
|
ItemUpdate merged = update1.merge(update2);
|
||||||
assertEquals(Collections.singleton(description1), merged.getDescriptionsIfNew());
|
assertEquals(Collections.singleton(description1), merged.getDescriptionsIfNew());
|
||||||
assertEquals(Collections.emptySet(), merged.getDescriptions());
|
assertEquals(Collections.emptySet(), merged.getDescriptions());
|
||||||
|
assertFalse(merged.isEmpty());
|
||||||
}
|
}
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
|
Loading…
Reference in New Issue
Block a user