Bug #86602

Ext: seo sitemap for records broken

Added by Susanne Moog 11 months ago. Updated 4 months ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
-
Target version:
Start date:
2018-10-09
Due date:
% Done:

100%

TYPO3 Version:
9
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

The sitemap for records does not use the fieldToParameterMap fields data to generate links.

Steps to reproduce:

TypoScript example:

plugin.tx_seo {
  config {
    xmlSitemap {
      sitemaps {
        example {
          provider = TYPO3\CMS\Seo\XmlSitemap\RecordsXmlSitemapDataProvider
          config {
            table = tx_example_domain_model_records
            sortField = sorting
            lastModifiedField = tstamp
            pid = 10
            url {
              pageId = 123
              fieldToParameterMap {
                uid = tx_example_detail[id]
              }

              useCacheHash = 1
            }
          }
        }
      }
    }
  }
}

  1. Add TS from above
  2. Call frontend sitemap with ?type=1533906435
  3. Go to the example sitemap
  4. Check links: In the resulting urls the "tx_example_detail[id]=<uid>" parameter is missing.

Related issues

Duplicated by TYPO3 Core - Bug #86598: TypoScript for fieldToParameterMap does not work (wrong assignment Key => Value) Closed 2018-10-08
Duplicated by TYPO3 Core - Bug #86716: RecordsXmlSitemapDataProvider not working Closed 2018-10-23

Associated revisions

Revision 70e31eea (diff)
Added by Susanne Moog 11 months ago

[BUGFIX] Records sitemap uses wrong data array

The records sitemap has an additional level of nesting for
data from database but was trying to access database data
on the first level resulting in missing URL parameters in
generated URLs.

This array access has been fixed.

Resolves: #86602
Releases: master
Change-Id: I0e5e368f177ad2fc4358b8ef8927e220536fa59f
Reviewed-on: https://review.typo3.org/58598
Reviewed-by: Richard Haeser <>
Tested-by: Richard Haeser <>
Tested-by: TYPO3com <>
Reviewed-by: Markus Klein <>
Tested-by: Markus Klein <>

History

#1 Updated by Gerrit Code Review 11 months ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/58598

#2 Updated by Wittkiel Gruppe 11 months ago

See also issue #86598, can also be closed after this fix.

#3 Updated by Georg Ringer 11 months ago

  • Duplicated by Bug #86598: TypoScript for fieldToParameterMap does not work (wrong assignment Key => Value) added

#4 Updated by Gerrit Code Review 11 months ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/58598

#5 Updated by Susanne Moog 11 months ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#6 Updated by Georg Ringer 10 months ago

  • Duplicated by Bug #86716: RecordsXmlSitemapDataProvider not working added

#7 Updated by Benni Mack 4 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF