Skip to content

Commit

Permalink
Revert "Handle existing votes where MaxValue is not 10 (#988)" (#989)
Browse files Browse the repository at this point in the history 
Because server already sends value out of 10

This reverts commit 90f46eb.
  • Loading branch information
@harshithmohan
harshithmohan authored Aug 18, 2024
1 parent 90f46eb commit 7d65179
Showing 1 changed file with 70 additions and 78 deletions.
148 changes: 70 additions & 78 deletions src/components/Collection/SeriesUserStats.tsx
View file
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import React, { useMemo } from 'react';
import React from 'react';

import SeriesRating from '@/components/Collection/Series/SeriesRating';
import { formatThousand } from '@/core/util';
Expand All @@ -9,89 +9,81 @@ type SeriesInfoProps = {
series: SeriesType;
};

const SeriesUserStats = React.memo(({ series }: SeriesInfoProps) => {
const userRating = useMemo(() => {
if (!series.UserRating) return 0;
const multiplier = 10 / series.UserRating.MaxValue;
return series.UserRating.Value * multiplier;
}, [series.UserRating]);

return (
<div className="flex flex-col gap-y-2">
<div className="flex justify-between">
<div className="font-semibold">File Count</div>
<div className="flex flex-row gap-x-1 font-normal">
<span>{formatThousand(series.Sizes.Local.Episodes)}</span>
<span>
{series.Sizes.Local.Episodes !== 1 ? 'Episodes' : 'Episode'}
</span>
{series.Sizes.Local.Specials > 0 && (
<>
<span>|</span>
<span>{formatThousand(series.Sizes.Local.Specials)}</span>
<span>
{series.Sizes.Local.Specials !== 1 ? 'Specials' : 'Special'}
</span>
</>
)}
</div>
const SeriesUserStats = React.memo(({ series }: SeriesInfoProps) => (
<div className="flex flex-col gap-y-2">
<div className="flex justify-between">
<div className="font-semibold">File Count</div>
<div className="flex flex-row gap-x-1 font-normal">
<span>{formatThousand(series.Sizes.Local.Episodes)}</span>
<span>
{series.Sizes.Local.Episodes !== 1 ? 'Episodes' : 'Episode'}
</span>
{series.Sizes.Local.Specials > 0 && (
<>
<span>|</span>
<span>{formatThousand(series.Sizes.Local.Specials)}</span>
<span>
{series.Sizes.Local.Specials !== 1 ? 'Specials' : 'Special'}
</span>
</>
)}
</div>
</div>

<div className="flex justify-between">
<div className="font-semibold">Watched</div>
<div className="flex flex-row gap-x-1 font-normal">
<span>{formatThousand(series.Sizes.Watched.Episodes)}</span>
<span>
{series.Sizes.Watched.Episodes !== 1 ? 'Episodes' : 'Episode'}
</span>
{series.Sizes.Total.Specials > 0 && series.Sizes.Watched.Specials > 0 && (
<>
<span>|</span>
<span>{formatThousand(series.Sizes.Watched.Specials)}</span>
<span>
{series.Sizes.Watched.Specials !== 1 ? 'Specials' : 'Special'}
</span>
</>
)}
</div>
<div className="flex justify-between">
<div className="font-semibold">Watched</div>
<div className="flex flex-row gap-x-1 font-normal">
<span>{formatThousand(series.Sizes.Watched.Episodes)}</span>
<span>
{series.Sizes.Watched.Episodes !== 1 ? 'Episodes' : 'Episode'}
</span>
{series.Sizes.Total.Specials > 0 && series.Sizes.Watched.Specials > 0 && (
<>
<span>|</span>
<span>{formatThousand(series.Sizes.Watched.Specials)}</span>
<span>
{series.Sizes.Watched.Specials !== 1 ? 'Specials' : 'Special'}
</span>
</>
)}
</div>
</div>

<div className="flex justify-between">
<div className="font-semibold">Missing</div>
{(series.Sizes.Missing.Episodes > 0 || series.Sizes.Missing.Specials > 0)
? (
<div className="flex flex-row gap-x-1 font-normal">
{series.Sizes.Missing.Episodes > 0 && (
<>
<span>{formatThousand(series.Sizes.Missing.Episodes)}</span>
<span>
{series.Sizes.Missing.Episodes !== 1 ? 'Episodes' : 'Episode'}
</span>
</>
)}
{series.Sizes.Missing.Episodes > 0 && series.Sizes.Missing.Specials > 0 && <span>|</span>}
{series.Sizes.Missing.Specials > 0 && (
<>
<span>{formatThousand(series.Sizes.Missing.Specials)}</span>
<span>
{series.Sizes.Missing.Specials !== 1 ? 'Specials' : 'Special'}
</span>
</>
)}
</div>
)
: <div>None, Nice Work!</div>}
</div>
<div className="flex justify-between">
<div className="font-semibold">Missing</div>
{(series.Sizes.Missing.Episodes > 0 || series.Sizes.Missing.Specials > 0)
? (
<div className="flex flex-row gap-x-1 font-normal">
{series.Sizes.Missing.Episodes > 0 && (
<>
<span>{formatThousand(series.Sizes.Missing.Episodes)}</span>
<span>
{series.Sizes.Missing.Episodes !== 1 ? 'Episodes' : 'Episode'}
</span>
</>
)}
{series.Sizes.Missing.Episodes > 0 && series.Sizes.Missing.Specials > 0 && <span>|</span>}
{series.Sizes.Missing.Specials > 0 && (
<>
<span>{formatThousand(series.Sizes.Missing.Specials)}</span>
<span>
{series.Sizes.Missing.Specials !== 1 ? 'Specials' : 'Special'}
</span>
</>
)}
</div>
)
: <div>None, Nice Work!</div>}
</div>

<div className="flex items-center justify-between">
<div className="font-semibold ">
Series Rating&nbsp;
{series.UserRating?.Type === 'Temporary' && '(Temp)'}
</div>
<SeriesRating seriesId={series.IDs.ID} ratingValue={userRating} />
<div className="flex items-center justify-between">
<div className="font-semibold ">
Series Rating&nbsp;
{series.UserRating?.Type === 'Temporary' && '(Temp)'}
</div>
<SeriesRating seriesId={series.IDs.ID} ratingValue={series.UserRating?.Value ?? 0} />
</div>
);
});
</div>
));

export default SeriesUserStats;

0 comments on commit 7d65179

Please sign in to comment.