#1713 Multiple XML sitemap files

Author: RomanovM

src/Libraries/Nop.Services/Seo/ISitemapGenerator.cs

@@ -13,16 +13,18 @@ public partial interface ISitemapGenerator
         /// See http://en.wikipedia.org/wiki/Sitemaps for more information.
         /// </summary>
         /// <param name="urlHelper">URL helper</param>
+        /// <param name="id">Sitemap identifier</param>
         /// <returns>Sitemap.xml as string</returns>
-        string Generate(UrlHelper urlHelper);
+        string Generate(UrlHelper urlHelper, int? id);
 
         /// <summary>
         /// This will build an xml sitemap for better index with search engines.
         /// See http://en.wikipedia.org/wiki/Sitemaps for more information.
         /// </summary>
         /// <param name="urlHelper">URL helper</param>
+        /// <param name="id">Sitemap identifier</param>
         /// <param name="stream">Stream of sitemap.</param>
-        void Generate(UrlHelper urlHelper, Stream stream);
+        void Generate(UrlHelper urlHelper, Stream stream, int? id);
 
     }
 }

src/Libraries/Nop.Services/Seo/SitemapGenerator.cs

@@ -376,12 +376,12 @@ public virtual ActionResult Sitemap()
         [NopHttpsRequirement(SslRequirement.No)]
         //available even when a store is closed
         [StoreClosed(true)]
-        public virtual ActionResult SitemapXml()
+        public virtual ActionResult SitemapXml(int? id)
         {
             if (!_commonSettings.SitemapEnabled)
                 return RedirectToRoute("HomePage");
 
-            var siteMap = _commonModelFactory.PrepareSitemapXml(this.Url);
+            var siteMap = _commonModelFactory.PrepareSitemapXml(this.Url, id);
             return Content(siteMap, "text/xml");
         }
 

src/Presentation/Nop.Web/Controllers/CommonController.cs

@@ -497,13 +497,13 @@ public virtual SitemapModel PrepareSitemapModel()
             return cachedModel;
         }
 
-        public virtual string PrepareSitemapXml(UrlHelper url)
+        public virtual string PrepareSitemapXml(UrlHelper url, int? id)
         {
-            string cacheKey = string.Format(ModelCacheEventConsumer.SITEMAP_SEO_MODEL_KEY,
+            string cacheKey = string.Format(ModelCacheEventConsumer.SITEMAP_SEO_MODEL_KEY, id,
                 _workContext.WorkingLanguage.Id,
                 string.Join(",", _workContext.CurrentCustomer.GetCustomerRoleIds()),
                 _storeContext.CurrentStore.Id);
-            var siteMap = _cacheManager.Get(cacheKey, () => _sitemapGenerator.Generate(url));
+            var siteMap = _cacheManager.Get(cacheKey, () => _sitemapGenerator.Generate(url, id));
             return siteMap;
         }
 

src/Presentation/Nop.Web/Factories/CommonModelFactory.cs

@@ -29,7 +29,7 @@ ContactVendorModel PrepareContactVendorModel(ContactVendorModel model, Vendor ve
 
         SitemapModel PrepareSitemapModel();
 
-        string PrepareSitemapXml(UrlHelper url);
+        string PrepareSitemapXml(UrlHelper url, int? id);
 
         StoreThemeSelectorModel PrepareStoreThemeSelectorModel();
 

src/Presentation/Nop.Web/Factories/ICommonModelFactory.cs

@@ -750,11 +750,12 @@ public ModelCacheEventConsumer(CatalogSettings catalogSettings)
         /// Key for sitemap on the sitemap SEO page
         /// </summary>
         /// <remarks>
-        /// {0} : language id
-        /// {1} : roles of the current user
-        /// {2} : current store ID
+        /// {0} : sitemap identifier
+        /// {1} : language id
+        /// {2} : roles of the current user
+        /// {3} : current store ID
         /// </remarks>
-        public const string SITEMAP_SEO_MODEL_KEY = "Nop.pres.sitemap.seo-{0}-{1}-{2}";
+        public const string SITEMAP_SEO_MODEL_KEY = "Nop.pres.sitemap.seo-{0}-{1}-{2}-{3}";
         public const string SITEMAP_PATTERN_KEY = "Nop.pres.sitemap";
 
         /// <summary>

src/Presentation/Nop.Web/Infrastructure/Cache/ModelCacheEventConsumer.cs

@@ -702,6 +702,11 @@ public void RegisterRoutes(RouteCollection routes)
                             "sitemap.xml",
                             new { controller = "Common", action = "SitemapXml" },
                             new[] { "Nop.Web.Controllers" });
+            routes.MapLocalizedRoute("sitemap-indexed.xml",
+                            "sitemap-{Id}.xml",
+                            new { controller = "Common", action = "SitemapXml" },
+                            new { Id = @"\d+" },
+                            new[] { "Nop.Web.Controllers" });
 
             //store closed
             routes.MapLocalizedRoute("StoreClosed",

src/Presentation/Nop.Web/Infrastructure/RouteProvider.cs

@@ -96,7 +96,7 @@
     <urlCompression doStaticCompression="true" doDynamicCompression="true" />
     <validation validateIntegratedModeConfiguration="false" />
     <handlers>
-      <add name="SitemapXml" path="sitemap.xml" verb="*" type="System.Web.Routing.UrlRoutingModule" resourceType="Unspecified" preCondition="integratedMode" />
+      <add name="SitemapXml" path="sitemap*.xml" verb="*" type="System.Web.Routing.UrlRoutingModule" resourceType="Unspecified" preCondition="integratedMode" />
       <add name="RobotsTxt" path="robots.txt" verb="*" type="System.Web.Routing.UrlRoutingModule" resourceType="Unspecified" preCondition="integratedMode" />
       <add name="MiniProfiler" path="mini-profiler-resources/*" verb="*" type="System.Web.Routing.UrlRoutingModule" resourceType="Unspecified" preCondition="integratedMode" />
       <add name="DenyAccessToPluginDLLs" verb="*" path="*.dll" type="System.Web.HttpForbiddenHandler" />

src/Presentation/Nop.Web/Web.config

@@ -166,7 +166,7 @@ public void Common_routes()
             "~/contactvendor/1".ShouldMapTo<CommonController>(c => c.ContactVendor(1));
             "~/contactus".ShouldMapTo<CommonController>(c => c.ContactUs());
             "~/sitemap".ShouldMapTo<CommonController>(c => c.Sitemap());
-            "~/sitemap.xml".ShouldMapTo<CommonController>(c => c.SitemapXml());
+            "~/sitemap.xml".ShouldMapTo<CommonController>(c => c.SitemapXml(null));
         }
 
         [Test]