Fixed: Multiple warnings for episode combined into one tooltip

Fixes #3890
pull/3902/head
Mark McDowall 4 years ago
parent ac75a31641
commit 0cc06fcba8

@ -17,6 +17,20 @@ function getAlternateTitles(seasonNumber, sceneSeasonNumber, alternateTitles) {
}); });
} }
function getWarningMessage(unverifiedSceneNumbering, seriesType, absoluteEpisodeNumber) {
const messages = [];
if (unverifiedSceneNumbering) {
messages.push('Scene number hasn\'t been verified yet');
}
if (seriesType === 'anime' && !absoluteEpisodeNumber) {
messages.push('Episode does not have an absolute episode number');
}
return messages.join('\n');
}
function EpisodeNumber(props) { function EpisodeNumber(props) {
const { const {
seasonNumber, seasonNumber,
@ -38,6 +52,8 @@ function EpisodeNumber(props) {
(seriesType === 'anime' && sceneAbsoluteEpisodeNumber !== undefined) || (seriesType === 'anime' && sceneAbsoluteEpisodeNumber !== undefined) ||
!!alternateTitles.length; !!alternateTitles.length;
const warningMessage = getWarningMessage(unverifiedSceneNumbering, seriesType, absoluteEpisodeNumber);
return ( return (
<span> <span>
{ {
@ -94,24 +110,16 @@ function EpisodeNumber(props) {
} }
{ {
unverifiedSceneNumbering && warningMessage ?
<Icon <Icon
className={styles.warning} className={styles.warning}
name={icons.WARNING} name={icons.WARNING}
kind={kinds.WARNING} kind={kinds.WARNING}
title="Scene number hasn't been verified yet" title={warningMessage}
/> /> :
null
} }
{
seriesType === 'anime' && !absoluteEpisodeNumber &&
<Icon
className={styles.warning}
name={icons.WARNING}
kind={kinds.WARNING}
title="Episode does not have an absolute episode number"
/>
}
</span> </span>
); );
} }

Loading…
Cancel
Save