0x2620/openmedialibrary
1
0
Fork 0
Code Issues 131 Pull requests Releases Wiki Activity

info view: make unique on split; compare correctly on edit

Browse source
This commit is contained in:
rlx 2016-01-14 14:43:04 +05:30
parent d54f524953
commit d3de45766d
1 changed files with 21 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

33
static/js/infoView.js
View file

@ -285,10 +285,14 @@ oml.ui.infoView = function(externalData, isMixed) {
&& Ox.decodeHTMLEntities(value).split(separator).some(function(value) {
return Ox.endsWith(value, ' (Ed.)');
});
return Ox.decodeHTMLEntities(value).split(separator).map(function(value) {
return Ox.encodeHTMLEntities(value).replace(/ \(Ed\.\)/, '')
+ (isEditor ? ' (Ed.)' : '');
});
return Ox.unique(
Ox.decodeHTMLEntities(value).split(separator).map(function(value) {
value = Ox.encodeHTMLEntities(value);
return isEditor
? value.replace(/ \(Ed\.\)$/, '') + ' (Ed.)'
: value;
})
);
}
function toggleCoverSize(ratio) {
@ -470,7 +474,11 @@ oml.ui.infoView = function(externalData, isMixed) {
)),
tooltip: isEditable ? oml.getEditTooltip() : '',
value: Ox.encodeHTMLEntities(
(data.author || []).join(separator)
(data.author || []).map(function(value, index) {
return index < data.author.length - 1
? value.replace(/ \(Ed\.\)$/, '')
: value;
}).join(separator)
)
})
.css({
@ -727,13 +735,14 @@ oml.ui.infoView = function(externalData, isMixed) {
});
function editMetadata(key, value) {
if (value != data[key]) {
var edit = {id: !isMultiple ? data.id : ui.listSelection};
edit[key] = key == 'author'
? splitValue(value || [], 'author')
: Ox.contains(arrayKeys, key)
? value ? value.split(separator) : []
: value;
var edit = Ox.extend(
{id: !isMultiple ? data.id : ui.listSelection},
key,
Ox.contains(arrayKeys, key)
? splitValue(value || '', key)
: value
);
if (!Ox.isEqual(edit[key], data[key])) {
data[key] = edit[key];
oml.api.edit(edit, function(result) {
if (!isMultiple || ui.updateResults) {