fix icon clearing bug in AIM 5.9

foodgroup/buddy.go

@@ -104,9 +104,6 @@ func getBuddyIconRefFromFeedbag(sess *state.Session, feedbagManager FeedbagManag
 		if bartType != wire.BARTTypesBuddyIcon {
 			continue
 		}
-		if bartInfo.HasClearIconHash() {
-			bartInfo.Flags = wire.BARTFlagsCustom | wire.BARTFlagsData
-		}
 		return &wire.BARTID{
 			Type: bartType,
 			BARTInfo: wire.BARTInfo{

foodgroup/feedbag_test.go

@@ -1057,7 +1057,7 @@ func TestFeedbagService_UpsertItem(t *testing.T) {
 											wire.NewTLV(wire.FeedbagAttributesBartInfo,
 												append(
 													[]byte{
-														wire.BARTFlagsCustom,
+														wire.BARTFlagsKnown,
 														uint8(len(wire.GetClearIconHash())),
 													},
 													wire.GetClearIconHash()...,
@@ -1085,7 +1085,7 @@ func TestFeedbagService_UpsertItem(t *testing.T) {
 										wire.BARTID{
 											Type: wire.BARTTypesBuddyIcon,
 											BARTInfo: wire.BARTInfo{
-												Flags: wire.BARTFlagsCustom | wire.BARTFlagsData,
+												Flags: wire.BARTFlagsKnown,
 												Hash:  wire.GetClearIconHash(),
 											},
 										},