Feature #34204
Global configuration of tag storage pid prevents multisite installations
| Status: | Resolved | Start date: | 2012-02-23 | |
|---|---|---|---|---|
| Priority: | Should have | Due date: | ||
| Assignee: | - | % Done: | 100% |
|
| Category: | - | |||
| Target version: | 2.0.0 | |||
| TYPO3 Version: | 4.5 | Has patch: | No | |
| PHP Version: | Complexity: | |||
| Votes: | 1 (View) |
Description
Storage Pid should be configurable on TS basis instead of global Ext setting. ATM it is not possible to have separate tags for multiple sites in one pagetree.
Associated revisions
[FEATURE] Make uid of tag saving page configurable
Currently the page where tags are saved was
configured in EM's settings which is not good
if you need multiple pid support in one backend.
Now it is possible to define pid in Page TsConfig with
tx_news.tagPid = 123
Change-Id: I13ade791acc46f1d4683386d69ffb1349be35d08
Resolves: #34204
Reviewed-on: https://review.typo3.org/17460
Reviewed-by: Georg Ringer
Tested-by: Georg Ringer
History
Updated by Georg Ringer about 1 year ago
- Status changed from New to Accepted
when doing that I thought more about value objects but yeah it makes sense.
however configuration needs then to be done in TsConfig to be able to add new tags
Updated by Steffen Müller about 1 year ago
A value objects is usually bound to its domain. (Evans: "Is address a value object? Who's asking?") I understand a multisite installation as a multidomain installation, so separating tags makes sense to me.
So is there still no way to avoid setting storagePid in TSconfig? It breaks TYPO3 convention. Since the number of extbased BE modules with TypoScript configuration grows, chances might be better these days.
Updated by Georg Ringer about 1 year ago
see Tx_News_Hooks_SuggestReceiverCall->getTagUid which saves new tags. I need the pid there.
as TsConfig is usually used for configuration in tceforms in backend, IMO it is not bad at all to use TsConfig as well but of course you can also try to parse TS of current page and to it by using TypoScript?
FE doesn't need the PID at all currently as there is no TagController->listAction(), so tags are always fetched from a news record by given uid
Updated by Georg Ringer 4 months ago
- Status changed from Accepted to Resolved
- Target version set to 2.0.0
- % Done changed from 0 to 100
resolved with https://review.typo3.org/#/c/17460/