apify / actor-scraper

House of Apify Scrapers. Generic scraping actors with a simple UI to handle complex web crawling and scraping use cases.
https://docs.apify.com/scraping
116 stars 44 forks source link

Web Scraper - setValue doesn't put into named Key Value store (if provided) #111

Closed metalwarrior665 closed 3 years ago

metalwarrior665 commented 3 years ago

It puts everything to a default one, setValue is passed together with the store name.

mnmkng commented 3 years ago

Yeah, looks like an omission here: https://github.com/apify/actor-scraper/blob/5a2aeb393defdc8be99d947536e8ec90b52af1db/web-scraper/src/crawler_setup.js#L560

It should be this.keyValueStore or something, not Apify.

Might also affect other scrapers.

mnmkng commented 3 years ago

@B4nan pls also check this bug. Should be a trivial fix.

B4nan commented 3 years ago

closing, fixed in v2 via #143