Skip to content

Commit

Permalink
Merge branch 'master' into entity-merging
Browse files Browse the repository at this point in the history
  • Loading branch information
MonkeyDo committed Feb 7, 2020
2 parents 9237e1b + 80ae865 commit 42e00cb
Showing 1 changed file with 7 additions and 8 deletions.
15 changes: 7 additions & 8 deletions src/server/routes/entity/entity.js
Original file line number Diff line number Diff line change
Expand Up @@ -929,14 +929,15 @@ async function saveEntitiesAndFinishRevision(
const parentsAddedPromise =
setParentRevisions(transacting, newRevision, _.uniq(parentRevisionIDs));

await Promise.all([
/** model.save returns a refreshed model */
const [savedEntities, ...others] = await Promise.all([
entitiesSavedPromise,
editorUpdatePromise,
parentsAddedPromise,
notePromise
]).catch(err => { throw err; });

return mainEntity;
return savedEntities.find(entityModel => entityModel.get('bbid') === mainEntity.get('bbid')) || mainEntity;
}

export function handleCreateOrEditEntity(
Expand Down Expand Up @@ -1042,13 +1043,11 @@ export function handleCreateOrEditEntity(
editorJSON.id, body.note
);

const refreshedEntity = await savedMainEntity.refresh({
require: false,
transacting,
withRelated: ['defaultAlias', 'aliasSet.aliases']
});
/** savedMainEntity is already updated, but without relations (we need the aliases for search reindexing) */
const savedEntityWithRelationships = await savedMainEntity.load(['aliasSet.aliases'],
{transacting});

return refreshedEntity.toJSON();
return savedEntityWithRelationships.toJSON();
}
catch (err) {
log.error(err);
Expand Down

0 comments on commit 42e00cb

Please sign in to comment.