Skip to content
This repository was archived by the owner on May 24, 2024. It is now read-only.

Commit 4c8ffd1

Browse files
committed
clean up
Signed-off-by: Ayman <[email protected]>
1 parent 3526c96 commit 4c8ffd1

File tree

1 file changed

+14
-7
lines changed

1 file changed

+14
-7
lines changed

cmd/confluence/confluence.go

Lines changed: 14 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,7 @@ const (
4949
ConfluenceAddHistoryCreatedByRole = false
5050
// ConfluenceAddHistoryLastUpdatedByRole - should we add contributor for history->lastUpdatedBy page version edit?
5151
ConfluenceAddHistoryLastUpdatedByRole = false
52+
contentHashField = "contentHash"
5253
)
5354

5455
var (
@@ -1351,7 +1352,9 @@ func (j *DSConfluence) ConfluenceEnrichItems(ctx *shared.Ctx, thrN int, items []
13511352
j.log.WithFields(logrus.Fields{"operation": "ConfluenceEnrichItems"}).Errorf("preventUpdateDuplication error: %+v", err)
13521353
return
13531354
}
1354-
d = append(d, cacheData...)
1355+
if len(cacheData) > 0 {
1356+
d = append(d, cacheData...)
1357+
}
13551358
if len(updates) > 0 {
13561359
ev, _ := updates[0].(insightsConf.ContentUpdatedEvent)
13571360
err = j.Publisher.PushEvents(ev.Event(), insightsStr, ConfluenceDataSource, contentsStr, envStr, updates)
@@ -1565,8 +1568,10 @@ func (j *DSConfluence) cachedCreatedContent(v []interface{}) ([]map[string]inter
15651568
}
15661569
contentHash := fmt.Sprintf("%x", sha256.Sum256(b))
15671570
cacheData = append(cacheData, map[string]interface{}{
1568-
"id": id,
1569-
"data": contentHash,
1571+
"id": id,
1572+
"data": map[string]interface{}{
1573+
contentHashField: contentHash,
1574+
},
15701575
})
15711576
}
15721577
return cacheData, nil
@@ -1600,13 +1605,15 @@ func (j *DSConfluence) preventUpdateDuplication(v []interface{}) ([]interface{},
16001605
if err != nil {
16011606
return updatedVals, cacheData, nil
16021607
}
1603-
cachedHash := ""
1608+
cachedHash := make(map[string]interface{})
16041609
err = json.Unmarshal(byt, &cachedHash)
1605-
if contentHash != cachedHash {
1610+
if contentHash != cachedHash["contentHash"] {
16061611
updatedVals = append(updatedVals, val)
16071612
cacheData = append(cacheData, map[string]interface{}{
1608-
"id": cacheID,
1609-
"data": contentHash,
1613+
"id": cacheID,
1614+
"data": map[string]interface{}{
1615+
contentHashField: contentHash,
1616+
},
16101617
})
16111618
}
16121619
}

0 commit comments

Comments
 (0)