Skip to content
This repository was archived by the owner on Jan 30, 2025. It is now read-only.

Fix sticker display bug #2175

Merged
merged 3 commits into from
Apr 24, 2018
Merged

Fix sticker display bug #2175

merged 3 commits into from
Apr 24, 2018

Conversation

Tyuoli
Copy link
Contributor

@Tyuoli Tyuoli commented Apr 23, 2018

No description provided.

@Tyuoli Tyuoli requested review from bmarty and manuroe April 23, 2018 12:50
} else {
downloadId = mMediasCache.downloadIdFromUrl(((ImageMessage) message).getUrl());
}
} else if (!event.getType().equals(Event.EVENT_TYPE_STICKER) && (message instanceof ImageMessage)) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove test on Event type

}
} else if (!event.getType().equals(Event.EVENT_TYPE_STICKER) && (message instanceof ImageMessage)) {
downloadId = mMediasCache.downloadIdFromUrl(((ImageMessage) message).getUrl());
} else if (event.getType().equals(Event.EVENT_TYPE_STICKER) && (message instanceof StickerMessage)) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove test on event type and test if message is a StickerMessage before ImageMessage

@bmarty
Copy link
Member

bmarty commented Apr 24, 2018

Ok, please squash your commit before submitting your PR

@Tyuoli Tyuoli merged commit a2d125e into develop Apr 24, 2018
@Tyuoli Tyuoli deleted the fix_sticker_bug branch April 24, 2018 15:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants