* fix(metadata): persist series show_status from provider metadata
Plugin-reported series status (proto status field 31) was mapped into
MetadataResult.ShowStatus but dropped by both metadataResultToItem and
itemToMetadataResult, so media_items.show_status stayed empty for every
movie and series - only the manga enrichment path ever wrote it. This
left the Show Status card overlay permanently blank for series.
- carry ShowStatus through both converters; series values normalize to
a canonical lowercase domain (returning/ended/cancelled/in_production/
upcoming) so TMDB "Returning Series"/"Canceled" and TVDB
"Continuing"/"Upcoming" converge on one spelling
- pass non-series values through verbatim so the manga status domain
("Ongoing", ...) can never be mangled by a generic refresh round-trip
- round-tripping the existing item's status also stops refreshes from
wiping a previously persisted value via show_status = EXCLUDED.show_status
- extend the web overlay formatter with continuing/upcoming/planned
TMDB/TVDB plugins need follow-up changes to actually emit the status
field; prepped separately in their repos.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
* fix(web): overlay badge layout, ordering, and wordmark rendering
Fixes from a full card-overlay audit (every issue verified by DOM
geometry measurement before/after):
- render each card edge as one flex row holding both corner stacks so
opposing badges share the width (min-w-0 + truncate) instead of
overlapping on narrow cards; long labels ellipsize instead of
wrapping over the opposite corner
- honor prefs.order via orderedOverlaysForPosition — the renderer
previously ignored the stored order entirely
- cap corners at 3 badges so maxed-out configs can't collide with the
opposite vertical corner
- lift bottom-right badges above the card menu button, which is always
visible on touch devices and occluded them
- suppress the text label when a wordmark icon (HDR10/ATMOS/AV1/HDR)
already spells it — pill/vibrant presets rendered "HDR10 HDR10" —
and widen the wordmark viewBoxes, which clipped their own text;
drop the never-used iconOnly flag the wordmark rule supersedes
- standalone resolution badge now uses prettyResolution ("4K", not
"2160P"), matching the combined badge
- manga cards skip generic overlays (their status/count chips own both
top corners) and the two chips now share a row and truncate instead
of overlapping each other
- useOverlayPrefs returns null while loading so cards no longer flash
default badges before the user's config or admin kill switch arrives
- settings rows for Resolution/HDR now say why they're hidden while
the combined badge is enabled
- add a CardOverlays test suite covering every registered overlay,
ordering, suppression, wordmarks, the corner cap, and menu clearance
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
---------
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
36 lines
1.3 KiB
Go
36 lines
1.3 KiB
Go
package metadata
|
|
|
|
import "strings"
|
|
|
|
// NormalizeShowStatus maps provider-reported series lifecycle statuses onto
|
|
// the canonical domain persisted in media_items.show_status for series:
|
|
// "returning", "ended", "cancelled", "in_production", "upcoming", or "".
|
|
// Providers spell these differently (TMDB "Returning Series" / "Canceled",
|
|
// TVDB "Continuing" / "Upcoming"), so persistence converges on one spelling
|
|
// clients can rely on. Unrecognized values pass through trimmed and
|
|
// lowercased rather than being dropped, so a new provider status still
|
|
// reaches clients (which fall back to displaying the raw value).
|
|
//
|
|
// This is series-domain only. Manga statuses use a different value domain
|
|
// ("Ongoing", "Completed", ...) normalized by the manga enrichment pipeline;
|
|
// callers must not route manga values through this function.
|
|
func NormalizeShowStatus(raw string) string {
|
|
cleaned := strings.ToLower(strings.TrimSpace(raw))
|
|
switch cleaned {
|
|
case "":
|
|
return ""
|
|
case "returning", "returning series", "continuing":
|
|
return "returning"
|
|
case "ended":
|
|
return "ended"
|
|
case "cancelled", "canceled":
|
|
return "cancelled"
|
|
case "in production", "in_production", "pilot":
|
|
return "in_production"
|
|
case "upcoming", "planned":
|
|
return "upcoming"
|
|
default:
|
|
return cleaned
|
|
}
|
|
}
|