Reduced MAL diff complexity
This commit is contained in:
parent
fec35059ae
commit
f691d04432
168
pages/editor/diffs.go
Normal file
168
pages/editor/diffs.go
Normal file
@ -0,0 +1,168 @@
|
|||||||
|
package editor
|
||||||
|
|
||||||
|
import (
|
||||||
|
"github.com/animenotifier/arn"
|
||||||
|
"github.com/animenotifier/mal"
|
||||||
|
"github.com/animenotifier/notify.moe/utils"
|
||||||
|
"github.com/animenotifier/notify.moe/utils/animediff"
|
||||||
|
)
|
||||||
|
|
||||||
|
// diff titles
|
||||||
|
func diffTitles(anime *arn.Anime, malAnime *mal.Anime) []animediff.Difference {
|
||||||
|
var differences []animediff.Difference
|
||||||
|
|
||||||
|
// Canonical title
|
||||||
|
if anime.Title.Canonical != malAnime.Title {
|
||||||
|
hash := utils.HashString(malAnime.Title)
|
||||||
|
|
||||||
|
if !arn.IsAnimeDifferenceIgnored(anime.ID, "mal", malAnime.ID, "CanonicalTitle", hash) {
|
||||||
|
differences = append(differences, &animediff.CanonicalTitle{
|
||||||
|
TitleA: anime.Title.Canonical,
|
||||||
|
TitleB: malAnime.Title,
|
||||||
|
NumericHash: hash,
|
||||||
|
})
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
// Japanese title
|
||||||
|
if anime.Title.Japanese != malAnime.JapaneseTitle {
|
||||||
|
hash := utils.HashString(malAnime.JapaneseTitle)
|
||||||
|
|
||||||
|
if !arn.IsAnimeDifferenceIgnored(anime.ID, "mal", malAnime.ID, "JapaneseTitle", hash) {
|
||||||
|
differences = append(differences, &animediff.JapaneseTitle{
|
||||||
|
TitleA: anime.Title.Japanese,
|
||||||
|
TitleB: malAnime.JapaneseTitle,
|
||||||
|
NumericHash: hash,
|
||||||
|
})
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
// Romaji title
|
||||||
|
if anime.Title.Romaji != malAnime.Title {
|
||||||
|
hash := utils.HashString(malAnime.Title)
|
||||||
|
|
||||||
|
if !arn.IsAnimeDifferenceIgnored(anime.ID, "mal", malAnime.ID, "RomajiTitle", hash) {
|
||||||
|
differences = append(differences, &animediff.RomajiTitle{
|
||||||
|
TitleA: anime.Title.Romaji,
|
||||||
|
TitleB: malAnime.Title,
|
||||||
|
NumericHash: hash,
|
||||||
|
})
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
return differences
|
||||||
|
}
|
||||||
|
|
||||||
|
// diff dates
|
||||||
|
func diffDates(anime *arn.Anime, malAnime *mal.Anime) []animediff.Difference {
|
||||||
|
var differences []animediff.Difference
|
||||||
|
|
||||||
|
// Airing start date
|
||||||
|
if anime.StartDate != malAnime.StartDate && malAnime.StartDate != "" {
|
||||||
|
hash := utils.HashString(malAnime.StartDate)
|
||||||
|
|
||||||
|
if !arn.IsAnimeDifferenceIgnored(anime.ID, "mal", malAnime.ID, "StartDate", hash) {
|
||||||
|
differences = append(differences, &animediff.StartDate{
|
||||||
|
DateA: anime.StartDate,
|
||||||
|
DateB: malAnime.StartDate,
|
||||||
|
NumericHash: hash,
|
||||||
|
})
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
// Airing end date
|
||||||
|
if anime.EndDate != malAnime.EndDate && malAnime.EndDate != "" {
|
||||||
|
hash := utils.HashString(malAnime.EndDate)
|
||||||
|
|
||||||
|
if !arn.IsAnimeDifferenceIgnored(anime.ID, "mal", malAnime.ID, "EndDate", hash) {
|
||||||
|
differences = append(differences, &animediff.EndDate{
|
||||||
|
DateA: anime.EndDate,
|
||||||
|
DateB: malAnime.EndDate,
|
||||||
|
NumericHash: hash,
|
||||||
|
})
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
return differences
|
||||||
|
}
|
||||||
|
|
||||||
|
// diff episodes
|
||||||
|
func diffEpisodes(anime *arn.Anime, malAnime *mal.Anime) []animediff.Difference {
|
||||||
|
var differences []animediff.Difference
|
||||||
|
|
||||||
|
// EpisodeCount
|
||||||
|
if malAnime.EpisodeCount != 0 && anime.EpisodeCount != malAnime.EpisodeCount {
|
||||||
|
hash := uint64(malAnime.EpisodeCount)
|
||||||
|
|
||||||
|
if !arn.IsAnimeDifferenceIgnored(anime.ID, "mal", malAnime.ID, "EpisodeCount", hash) {
|
||||||
|
differences = append(differences, &animediff.EpisodeCount{
|
||||||
|
EpisodesA: anime.EpisodeCount,
|
||||||
|
EpisodesB: malAnime.EpisodeCount,
|
||||||
|
NumericHash: hash,
|
||||||
|
})
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
return differences
|
||||||
|
}
|
||||||
|
|
||||||
|
// diff status
|
||||||
|
func diffStatus(anime *arn.Anime, malAnime *mal.Anime) []animediff.Difference {
|
||||||
|
var differences []animediff.Difference
|
||||||
|
|
||||||
|
// Status
|
||||||
|
if anime.Status != malAnime.Status {
|
||||||
|
hash := utils.HashString(malAnime.Status)
|
||||||
|
|
||||||
|
if !arn.IsAnimeDifferenceIgnored(anime.ID, "mal", malAnime.ID, "Status", hash) {
|
||||||
|
differences = append(differences, &animediff.Status{
|
||||||
|
StatusA: anime.Status,
|
||||||
|
StatusB: malAnime.Status,
|
||||||
|
NumericHash: hash,
|
||||||
|
})
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
return differences
|
||||||
|
}
|
||||||
|
|
||||||
|
// diff synopsis
|
||||||
|
func diffSynopsis(anime *arn.Anime, malAnime *mal.Anime) []animediff.Difference {
|
||||||
|
var differences []animediff.Difference
|
||||||
|
|
||||||
|
// Synopsis
|
||||||
|
if len(anime.Summary) < 300 && len(anime.Summary)+50 < len(malAnime.Synopsis) {
|
||||||
|
hash := utils.HashString(malAnime.Synopsis)
|
||||||
|
|
||||||
|
if !arn.IsAnimeDifferenceIgnored(anime.ID, "mal", malAnime.ID, "Synopsis", hash) {
|
||||||
|
differences = append(differences, &animediff.Synopsis{
|
||||||
|
SynopsisA: anime.Summary,
|
||||||
|
SynopsisB: malAnime.Synopsis,
|
||||||
|
NumericHash: hash,
|
||||||
|
})
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
return differences
|
||||||
|
}
|
||||||
|
|
||||||
|
// diff genres
|
||||||
|
func diffGenres(anime *arn.Anime, malAnime *mal.Anime) []animediff.Difference {
|
||||||
|
var differences []animediff.Difference
|
||||||
|
|
||||||
|
// Compare genres
|
||||||
|
hashA := utils.HashStringsNoOrder(anime.Genres)
|
||||||
|
hashB := utils.HashStringsNoOrder(malAnime.Genres)
|
||||||
|
|
||||||
|
if hashA != hashB {
|
||||||
|
if !arn.IsAnimeDifferenceIgnored(anime.ID, "mal", malAnime.ID, "Genres", hashB) {
|
||||||
|
differences = append(differences, &animediff.Genres{
|
||||||
|
GenresA: anime.Genres,
|
||||||
|
GenresB: malAnime.Genres,
|
||||||
|
NumericHash: hashB,
|
||||||
|
})
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
return differences
|
||||||
|
}
|
||||||
@ -14,6 +14,9 @@ import (
|
|||||||
|
|
||||||
const maxCompareMALEntries = 15
|
const maxCompareMALEntries = 15
|
||||||
|
|
||||||
|
// diffFunction is the signature of a diff function.
|
||||||
|
type diffFunction func(*arn.Anime, *mal.Anime) []animediff.Difference
|
||||||
|
|
||||||
// CompareMAL ...
|
// CompareMAL ...
|
||||||
func CompareMAL(ctx *aero.Context) string {
|
func CompareMAL(ctx *aero.Context) string {
|
||||||
user := utils.GetUser(ctx)
|
user := utils.GetUser(ctx)
|
||||||
@ -121,122 +124,19 @@ func compare(animes []*arn.Anime) []*utils.MALComparison {
|
|||||||
func diff(anime *arn.Anime, malAnime *mal.Anime) []animediff.Difference {
|
func diff(anime *arn.Anime, malAnime *mal.Anime) []animediff.Difference {
|
||||||
var differences []animediff.Difference
|
var differences []animediff.Difference
|
||||||
|
|
||||||
// Canonical title
|
// We'll use the following diffs
|
||||||
if anime.Title.Canonical != malAnime.Title {
|
diffFunctions := []diffFunction{
|
||||||
hash := utils.HashString(malAnime.Title)
|
diffTitles,
|
||||||
|
diffDates,
|
||||||
if !arn.IsAnimeDifferenceIgnored(anime.ID, "mal", malAnime.ID, "CanonicalTitle", hash) {
|
diffEpisodes,
|
||||||
differences = append(differences, &animediff.CanonicalTitle{
|
diffStatus,
|
||||||
TitleA: anime.Title.Canonical,
|
diffSynopsis,
|
||||||
TitleB: malAnime.Title,
|
diffGenres,
|
||||||
NumericHash: hash,
|
|
||||||
})
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
// Japanese title
|
for _, diffFunction := range diffFunctions {
|
||||||
if anime.Title.Japanese != malAnime.JapaneseTitle {
|
diffs := diffFunction(anime, malAnime)
|
||||||
hash := utils.HashString(malAnime.JapaneseTitle)
|
differences = append(differences, diffs...)
|
||||||
|
|
||||||
if !arn.IsAnimeDifferenceIgnored(anime.ID, "mal", malAnime.ID, "JapaneseTitle", hash) {
|
|
||||||
differences = append(differences, &animediff.JapaneseTitle{
|
|
||||||
TitleA: anime.Title.Japanese,
|
|
||||||
TitleB: malAnime.JapaneseTitle,
|
|
||||||
NumericHash: hash,
|
|
||||||
})
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
// Romaji title
|
|
||||||
if anime.Title.Romaji != malAnime.Title {
|
|
||||||
hash := utils.HashString(malAnime.Title)
|
|
||||||
|
|
||||||
if !arn.IsAnimeDifferenceIgnored(anime.ID, "mal", malAnime.ID, "RomajiTitle", hash) {
|
|
||||||
differences = append(differences, &animediff.RomajiTitle{
|
|
||||||
TitleA: anime.Title.Romaji,
|
|
||||||
TitleB: malAnime.Title,
|
|
||||||
NumericHash: hash,
|
|
||||||
})
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
// Airing start date
|
|
||||||
if anime.StartDate != malAnime.StartDate && malAnime.StartDate != "" {
|
|
||||||
hash := utils.HashString(malAnime.StartDate)
|
|
||||||
|
|
||||||
if !arn.IsAnimeDifferenceIgnored(anime.ID, "mal", malAnime.ID, "StartDate", hash) {
|
|
||||||
differences = append(differences, &animediff.StartDate{
|
|
||||||
DateA: anime.StartDate,
|
|
||||||
DateB: malAnime.StartDate,
|
|
||||||
NumericHash: hash,
|
|
||||||
})
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
// Airing end date
|
|
||||||
if anime.EndDate != malAnime.EndDate && malAnime.EndDate != "" {
|
|
||||||
hash := utils.HashString(malAnime.EndDate)
|
|
||||||
|
|
||||||
if !arn.IsAnimeDifferenceIgnored(anime.ID, "mal", malAnime.ID, "EndDate", hash) {
|
|
||||||
differences = append(differences, &animediff.EndDate{
|
|
||||||
DateA: anime.EndDate,
|
|
||||||
DateB: malAnime.EndDate,
|
|
||||||
NumericHash: hash,
|
|
||||||
})
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
// Status
|
|
||||||
if anime.Status != malAnime.Status {
|
|
||||||
hash := utils.HashString(malAnime.Status)
|
|
||||||
|
|
||||||
if !arn.IsAnimeDifferenceIgnored(anime.ID, "mal", malAnime.ID, "Status", hash) {
|
|
||||||
differences = append(differences, &animediff.Status{
|
|
||||||
StatusA: anime.Status,
|
|
||||||
StatusB: malAnime.Status,
|
|
||||||
NumericHash: hash,
|
|
||||||
})
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
// EpisodeCount
|
|
||||||
if malAnime.EpisodeCount != 0 && anime.EpisodeCount != malAnime.EpisodeCount {
|
|
||||||
hash := uint64(malAnime.EpisodeCount)
|
|
||||||
|
|
||||||
if !arn.IsAnimeDifferenceIgnored(anime.ID, "mal", malAnime.ID, "EpisodeCount", hash) {
|
|
||||||
differences = append(differences, &animediff.EpisodeCount{
|
|
||||||
EpisodesA: anime.EpisodeCount,
|
|
||||||
EpisodesB: malAnime.EpisodeCount,
|
|
||||||
NumericHash: hash,
|
|
||||||
})
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
// Synopsis
|
|
||||||
if len(anime.Summary) < 300 && len(anime.Summary)+50 < len(malAnime.Synopsis) {
|
|
||||||
hash := utils.HashString(malAnime.Synopsis)
|
|
||||||
|
|
||||||
if !arn.IsAnimeDifferenceIgnored(anime.ID, "mal", malAnime.ID, "Synopsis", hash) {
|
|
||||||
differences = append(differences, &animediff.Synopsis{
|
|
||||||
SynopsisA: anime.Summary,
|
|
||||||
SynopsisB: malAnime.Synopsis,
|
|
||||||
NumericHash: hash,
|
|
||||||
})
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
// Compare genres
|
|
||||||
hashA := utils.HashStringsNoOrder(anime.Genres)
|
|
||||||
hashB := utils.HashStringsNoOrder(malAnime.Genres)
|
|
||||||
|
|
||||||
if hashA != hashB {
|
|
||||||
if !arn.IsAnimeDifferenceIgnored(anime.ID, "mal", malAnime.ID, "Genres", hashB) {
|
|
||||||
differences = append(differences, &animediff.Genres{
|
|
||||||
GenresA: anime.Genres,
|
|
||||||
GenresB: malAnime.Genres,
|
|
||||||
NumericHash: hashB,
|
|
||||||
})
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
return differences
|
return differences
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user