Little bit of cleanup
This commit is contained in:
parent
3b5fa15388
commit
9e556cb162
2 changed files with 7 additions and 4 deletions
|
|
@ -338,7 +338,8 @@ public class ChapterController : BaseApiController
|
||||||
_unitOfWork
|
_unitOfWork
|
||||||
);
|
);
|
||||||
|
|
||||||
// TODO: If changes; remove ChapterPublisher from KPlusOverrides
|
// TODO: Only remove field if changes were made
|
||||||
|
chapter.KPlusOverrides.Remove(MetadataSettingField.ChapterPublisher);
|
||||||
// Update publishers
|
// Update publishers
|
||||||
await PersonHelper.UpdateChapterPeopleAsync(
|
await PersonHelper.UpdateChapterPeopleAsync(
|
||||||
chapter,
|
chapter,
|
||||||
|
|
|
||||||
|
|
@ -1131,8 +1131,10 @@ public class ExternalMetadataService : IExternalMetadataService
|
||||||
madeModification = UpdateChapterTitle(chapter, settings, potentialMatch.Title, series.Name) || madeModification;
|
madeModification = UpdateChapterTitle(chapter, settings, potentialMatch.Title, series.Name) || madeModification;
|
||||||
madeModification = UpdateChapterSummary(chapter, settings, potentialMatch.Summary) || madeModification;
|
madeModification = UpdateChapterSummary(chapter, settings, potentialMatch.Summary) || madeModification;
|
||||||
madeModification = UpdateChapterReleaseDate(chapter, settings, potentialMatch.ReleaseDate) || madeModification;
|
madeModification = UpdateChapterReleaseDate(chapter, settings, potentialMatch.ReleaseDate) || madeModification;
|
||||||
// TODO: Update to set KPlusOverride
|
|
||||||
madeModification = await UpdateChapterPublisher(chapter, settings, potentialMatch.Publisher) || madeModification;
|
var hasUpdatedPublisher = await UpdateChapterPublisher(chapter, settings, potentialMatch.Publisher);
|
||||||
|
if (hasUpdatedPublisher) chapter.AddKPlusOverride(MetadataSettingField.ChapterPublisher);
|
||||||
|
madeModification = hasUpdatedPublisher || madeModification;
|
||||||
|
|
||||||
madeModification = await UpdateChapterPeople(chapter, settings, PersonRole.CoverArtist, potentialMatch.Artists) || madeModification;
|
madeModification = await UpdateChapterPeople(chapter, settings, PersonRole.CoverArtist, potentialMatch.Artists) || madeModification;
|
||||||
madeModification = await UpdateChapterPeople(chapter, settings, PersonRole.Writer, potentialMatch.Writers) || madeModification;
|
madeModification = await UpdateChapterPeople(chapter, settings, PersonRole.Writer, potentialMatch.Writers) || madeModification;
|
||||||
|
|
@ -1522,7 +1524,7 @@ public class ExternalMetadataService : IExternalMetadataService
|
||||||
{
|
{
|
||||||
try
|
try
|
||||||
{
|
{
|
||||||
// Only choose a better image if we're overriding a user provided cover
|
// Only choose the better image if we're overriding a user provided cover
|
||||||
await _coverDbService.SetSeriesCoverByUrl(series, coverUrl, false, !series.Metadata.HasSetKPlusMetadata(MetadataSettingField.Covers));
|
await _coverDbService.SetSeriesCoverByUrl(series, coverUrl, false, !series.Metadata.HasSetKPlusMetadata(MetadataSettingField.Covers));
|
||||||
}
|
}
|
||||||
catch (Exception ex)
|
catch (Exception ex)
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue