If you have alias for sitemaps you can't use it because index compose path manually instead routing system with Url class.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jansete created an issue. See original summary.

jansete’s picture

Status: Active » Needs review
FileSize
1.56 KB
jansete’s picture

FileSize
1.23 KB

Status: Needs review » Needs work

The last submitted patch, 3: 3051700-3.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

jansete’s picture

Status: Needs work » Needs review
FileSize
1.52 KB

Status: Needs review » Needs work

The last submitted patch, 5: 3051700-5.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

gbyte’s picture

Version: 8.x-3.1 » 8.x-3.x-dev
Status: Needs work » Needs review

Please create patches against 3.x, we can't fix a release retroactively.

gbyte’s picture

Status: Needs review » Needs work
jansete’s picture

Nice, now patch against dev branch.

jansete’s picture

Status: Needs work » Needs review
gbyte’s picture

Category: Bug report » Feature request

Thanks for the patch, I understand this is only to show the aliased path inside of the sitemap index, which is a feature request, not a bug.

  • gbyte.co committed 817aae8 on 8.x-3.x
    Issue #3051700 by jansete, gbyte.co: Use core Url class to generate...
gbyte’s picture

Status: Needs review » Fixed

Thanks.

Status: Fixed » Closed (fixed)

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