From 586d146fdbf086d58abf8308cc8ad3ddeca42eff Mon Sep 17 00:00:00 2001 From: gitgiggety <79809426+gitgiggety@users.noreply.github.com> Date: Fri, 21 May 2021 04:32:28 +0200 Subject: [PATCH] Apply all post processors to performer (#1387) * Apply all post processors to performer Scraping a performer by fragment doesn't correctly work with tags. When tags are returned to the scraper then all are recognized as new. This is due to the post process method not being applied while it should be, as is done when scraping a performer by URL. --- pkg/scraper/scrapers.go | 8 +++++--- ui/v2.5/src/components/Changelog/versions/v080.md | 3 +++ 2 files changed, 8 insertions(+), 3 deletions(-) diff --git a/pkg/scraper/scrapers.go b/pkg/scraper/scrapers.go index f82dbd223..a6ad82bf9 100644 --- a/pkg/scraper/scrapers.go +++ b/pkg/scraper/scrapers.go @@ -197,9 +197,11 @@ func (c Cache) ScrapePerformer(scraperID string, scrapedPerformer models.Scraped return nil, err } - // post-process - set the image if applicable - if err := setPerformerImage(ret, c.globalConfig); err != nil { - logger.Warnf("Could not set image using URL %s: %s", *ret.Image, err.Error()) + if ret != nil { + err = c.postScrapePerformer(ret) + if err != nil { + return nil, err + } } return ret, nil diff --git a/ui/v2.5/src/components/Changelog/versions/v080.md b/ui/v2.5/src/components/Changelog/versions/v080.md index d83e44312..4116b435f 100644 --- a/ui/v2.5/src/components/Changelog/versions/v080.md +++ b/ui/v2.5/src/components/Changelog/versions/v080.md @@ -5,3 +5,6 @@ * Add `subtractDays` post-process scraper action. ([#1399](https://github.com/stashapp/stash/pull/1399)) * Skip scanning directories if path matches image and video exclude patterns. ([#1382](https://github.com/stashapp/stash/pull/1382)) * Add button to remove studio stash ID. ([#1378](https://github.com/stashapp/stash/pull/1378)) + +### 🐛 Bug fixes +* Fix post-processing not running when scraping by performer fragment. ([#1387](https://github.com/stashapp/stash/pull/1387))