jschaufuss/DiceCloud
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

re-enabled computation on client side for optimistic UI

Browse Source
This commit is contained in:
Stefan Zermatten
2020-05-30 23:56:55 +02:00
parent 4bdc254627
commit d24fb5661d
1 changed files with 1 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
app/imports/api/creature/computation/recomputeCreature.js
View File

@@ -72,13 +72,11 @@ const calculationPropertyTypes = [
* - Write the computed results back to the database * - Write the computed results back to the database
*/ */
export function recomputeCreatureById(creatureId){ export function recomputeCreatureById(creatureId){
// Skipping computation on the client can result in the server being made to
// do work that isn't possible, in exchange for dramatic performance gains
if (Meteor.isClient) return;
let props = getActiveProperties({ let props = getActiveProperties({
ancestorId: creatureId, ancestorId: creatureId,
filter: {type: {$in: calculationPropertyTypes}}, filter: {type: {$in: calculationPropertyTypes}},
includeUntoggled: true, includeUntoggled: true,
// TODO filter out expensive fields, particularly icon field
}); });
let computationMemo = new ComputationMemo(props); let computationMemo = new ComputationMemo(props);
computeMemo(computationMemo); computeMemo(computationMemo);