Skip to content
Snippets Groups Projects
Commit caad699a authored by Oliver Bartsch's avatar Oliver Bartsch
Browse files

[BUGFIX] Add uid field to fieldDefinitions in EXT:seo

EXT:seo defines two additional types for "Page Overview"
in the info module.

Since the special uid field, which has a special treatment
in PageInformationController, was previously missing, no
view and no edit options were displayed.

Besides the already existing special treatments, in master,
the field is also used to determine the row tag to be used
(either `<th>`or `<td>`). Therefore, the two EXT:seo types
previously always used `<th>`, which leads to all cells were
bold (in at least the chrome browser, due to user agent styles).

This is now fixed by adding the special uid field to the
mod.web_info.fieldDefinitions declaration in EXT:seo.

Resolves: #94291
Releases: master, 10.4
Change-Id: I5c20528008160a2c5d853b6d70cc84e3cf3ad89a
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/69418


Tested-by: default avatarcore-ci <typo3@b13.com>
Tested-by: default avatarChristian Kuhn <lolli@schwarzbu.ch>
Tested-by: default avatarJochen <rothjochen@gmail.com>
Tested-by: default avatarOliver Bartsch <bo@cedev.de>
Reviewed-by: default avatarChristian Kuhn <lolli@schwarzbu.ch>
Reviewed-by: default avatarJochen <rothjochen@gmail.com>
Reviewed-by: default avatarOliver Bartsch <bo@cedev.de>
parent 8140ec47
No related merge requests found
......@@ -32,11 +32,11 @@ unset($metaTagManagerRegistry);
mod.web_info.fieldDefinitions {
seo {
label = LLL:EXT:seo/Resources/Private/Language/locallang_webinfo.xlf:seo
fields = title,slug,seo_title,description,no_index,no_follow,canonical_link,sitemap_changefreq,sitemap_priority
fields = title,uid,slug,seo_title,description,no_index,no_follow,canonical_link,sitemap_changefreq,sitemap_priority
}
social_media {
label = LLL:EXT:seo/Resources/Private/Language/locallang_webinfo.xlf:social_media
fields = title,og_title,og_description,twitter_title,twitter_description
fields = title,uid,og_title,og_description,twitter_title,twitter_description
}
}
'));
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment