Problem/Motivation

When configuring several sitemaps locations in "CDN via Sitemap" plugin, only the last sitemap seems to be used.
In fact, every call to SitemapParser::parseRecursive resets previous calculation.

Steps to reproduce

In /admin/config/development/warmer/settings, fill in several sitemaps locations.
In /admin/config/development/warmer, check "CDN via Sitemap" and submit. The success message wioll show the number of urls un the last sitemap.

Proposed resolution

Persist calculated urls in SitemapWarmer by adding an object property.

Remaining tasks

User interface changes

none

API changes

none

Data model changes

none

CommentFileSizeAuthor
#2 warmer-3370438-1.patch1.11 KBnollim

Comments

nollim created an issue. See original summary.

nollim’s picture

nollim commented
StatusFileSize
new warmer-3370438-1.patch1.11 KB
e0ipso’s picture

e0ipso
Primary language Catalan
Location Can Picafort
commented
Issue tags: +Needs tests

This looks great @nollim. Any chance we can add a test for this?

e0ipso’s picture

e0ipso
Primary language Catalan
Location Can Picafort
commented
Status: Active » Fixed

I have not tested this manually, but it seems a simple enough change and I assume it works for @nollim.

e0ipso’s picture

e0ipso
Primary language Catalan
Location Can Picafort
commented
Issue tags: -Needs tests
e0ipso’s picture

e0ipso
Primary language Catalan
Location Can Picafort
commented
//www.flaticon.com/free-icons/thank-you Thank you for your contribution! Your continued support to this project also helps me, as a maintainer.
There are multiple ways to show appreciation for the work I did in this project, those include:
  • Triaging issues, and adding more context to existing issues.
  • Writing documentation, or patches for this project.
  • Writing blog posts, speaking about it at conferences.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.