Foundry376/Mailspring
1
1
Fork 0
mirror of https://github.com/Foundry376/Mailspring.git synced 2025-01-25 01:20:24 +08:00
Code Issues Projects Releases Packages Wiki Activity

fix(metadata): Correctly call modelify for latest models when setting metadata

Browse source
This commit is contained in:
Juan Tejada 2016-02-19 11:52:49 -08:00
parent b65c4104b4
commit aedc09b2b3
1 changed files with 5 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/flux/stores/metadata-store.es6
View file

@ -13,12 +13,16 @@ class MetadataStore extends NylasStore {
_setMetadata(modelOrModels, pluginId, metadataValue) { _setMetadata(modelOrModels, pluginId, metadataValue) {
const models = (modelOrModels instanceof Array) ? modelOrModels : [modelOrModels]; const models = (modelOrModels instanceof Array) ? modelOrModels : [modelOrModels];
const modelClass = models[0].constructor
if (!models.every(m => m.constructor === modelClass)) {
throw new Error('Actions.setMetadata - All models provided must be of the same type')
}
DatabaseStore.inTransaction((t)=> { DatabaseStore.inTransaction((t)=> {
// Get the latest version of the models from the datbaase before applying // Get the latest version of the models from the datbaase before applying
// metadata in case other plugins also saved metadata, and we don't want // metadata in case other plugins also saved metadata, and we don't want
// to overwrite it // to overwrite it
return ( return (
t.modelify(_.pluck(models, 'clientId')) t.modelify(modelClass, _.pluck(models, 'clientId'))
.then((latestModels)=> { .then((latestModels)=> {
const updatedModels = latestModels.map(m => m.applyPluginMetadata(pluginId, metadataValue)); const updatedModels = latestModels.map(m => m.applyPluginMetadata(pluginId, metadataValue));
return ( return (