#7793 closed (fixed)
Sitemaps generator does not care about 50,000 URL limit
Reported by: | Julian Bez | Owned by: | nobody |
---|---|---|---|
Component: | Contrib apps | Version: | dev |
Severity: | Keywords: | sitemaps | |
Cc: | Triage Stage: | Ready for checkin | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
As stated in the protocol, each sitemap must have no more than 50,000 URLs.
http://sitemaps.org/protocol.php#index
The current sitemaps generator does not care about the limit and is not suited for larger sets of data.
The patch provided adds the paginator to sitemaps and creates a sitemap for every 50,000 URLs.
Attachments (3)
Change History (10)
by , 16 years ago
Attachment: | sitemaps-pagination.diff added |
---|
comment:1 by , 16 years ago
Has patch: | set |
---|
comment:2 by , 16 years ago
milestone: | → 1.0 beta |
---|---|
Triage Stage: | Unreviewed → Ready for checkin |
comment:3 by , 16 years ago
I don't really have any way of testing this, but I've read it through with some care and it looks to be doing the right thing. The docs change might need to be added to a little in the future, since it doesn't really make sense to me and I know what the patch is meant to be doing. So attempts to fix that (in another ticket) will be welcomed.
comment:4 by , 16 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:5 by , 15 years ago
I am just trying to paginate the different sections to a lower limit, I tried to define the pagination limit to 1000 (otherwise the size of the view is too big to be in my memcached, memcached-limit: 1mb) to get more pages, but the index get's messed up and just list always the same "sitemap.xml" not the correct sections pages, should we reopen this ticket or is just my configuration problem ?
comment:6 by , 15 years ago
sorry for my last message, I found the problem was related to cache_view and urlresolve, I will send a message to the mailing list
Patch that adds pagination