Skip to content

Commit e91a4d9

Browse files
authored
Merge pull request #778 from bserdar/untouched-hook
Untouched documents should not be queued for hook
2 parents 86f03fe + 4bba964 commit e91a4d9

File tree

1 file changed

+10
-8
lines changed

1 file changed

+10
-8
lines changed

crud/src/main/java/com/redhat/lightblue/hooks/HookManager.java

Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -235,14 +235,16 @@ private void addDocument(List<HookAndDocs> hooks,DocCtx doc) {
235235
if(!doc.hasErrors()) {
236236
for(HookAndDocs hook:hooks) {
237237
boolean queue=false;
238-
switch(doc.getCRUDOperationPerformed()) {
239-
case INSERT: queue=hook.hook.isInsert();break;
240-
case UPDATE: queue=hook.hook.isUpdate();break;
241-
case DELETE: queue=hook.hook.isDelete();break;
242-
case FIND: queue=hook.hook.isFind();break;
243-
}
244-
if(queue) {
245-
hook.docList.add(new HookDocInfo(doc));
238+
if(doc.getCRUDOperationPerformed()!=null) {
239+
switch(doc.getCRUDOperationPerformed()) {
240+
case INSERT: queue=hook.hook.isInsert();break;
241+
case UPDATE: queue=hook.hook.isUpdate();break;
242+
case DELETE: queue=hook.hook.isDelete();break;
243+
case FIND: queue=hook.hook.isFind();break;
244+
}
245+
if(queue) {
246+
hook.docList.add(new HookDocInfo(doc));
247+
}
246248
}
247249
}
248250
}

0 commit comments

Comments
 (0)