Project

General

Profile

Actions

Feature #57297

closed

Add UID to SysDomainCache

Added by Henrik Ziegenhain over 10 years ago. Updated about 7 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Frontend
Target version:
Start date:
2014-03-25
Due date:
% Done:

100%

Estimated time:
PHP Version:
5.3
Tags:
Complexity:
no-brainer
Sprint Focus:

Description

If you are building an extension based on Domain-Records (outputting Data based on a domain-record) it would be better to check against an Id instead of the domain name.

With TYPO3\CMS\Frontend\Controller\TypoScriptFrontendController->getSysDomainCache() you can fetch pid, domainName and the forced-option for domain-records.
It would be very easy to simply add the uid to the select statement and return it.

Having this uid in Frontend enables you to store the domain-record id in Database and output the data only if the ids from current domain-record and database field are matching.

I will make a patch the following days.

I think its nearly a no-brainer, but as it`s a feature the target version must be 6.3 imho.

Actions #1

Updated by Gerrit Code Review over 10 years 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/28793

Actions #2

Updated by Gerrit Code Review over 10 years ago

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/28794

Actions #3

Updated by Gerrit Code Review over 10 years 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/28794

Actions #4

Updated by Anonymous over 10 years ago

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

Updated by Riccardo De Contardi about 7 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF