Project

General

Profile

Actions
Copy link

Bug #24679

closed

be_layout foreign_table_where

Added by Steffen Gebert over 13 years ago. Updated about 12 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Steffen Gebert
Category:
-
Target version:
-
Start date:
2011-01-20
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

If a General Record Storage Page (GRSP) is configured, which is very likely the case, when you use tt_news, then TYPO3 searches for be_layout records only on the GRSP page and the PAGE_TSCONFIG_ID.

If no GRSP is configured, it uses records from everywhere.

I suggest do always include records from the page, which properties are currently edited, to make the feature a bit more fool-proof.

'foreign_table_where' => 'AND ( ( ###PAGE_TSCONFIG_ID### = 0 AND ###STORAGE_PID### = 0 ) OR ( be_layouts.pid = ###PAGE_TSCONFIG_ID### OR be_layouts.pid = ###STORAGE_PID### ) ) AND be_layouts.hidden = 0',
'

(issue imported from #M17161)

Files

Download all files
17161.diff (1.92 KB) 17161.diff Administrator Admin, 2011-01-20 14:30
17161_v2.diff (1.99 KB) 17161_v2.diff Administrator Admin, 2011-01-20 14:34
17161_v3.diff (2.06 KB) 17161_v3.diff Administrator Admin, 2011-01-21 13:06
17161_v4.diff (2.06 KB) 17161_v4.diff Administrator Admin, 2011-01-21 14:06
17161_v5.diff (2.06 KB) 17161_v5.diff Administrator Admin, 2011-01-21 15:55
17161_v6.diff (1.66 KB) 17161_v6.diff Administrator Admin, 2011-01-21 16:02
Actions
Copy link
 #1

Updated by Jo Hasenau over 13 years ago

Actually this should have been the case anyway, so if it's not yet done, I guess I will have to fix it.

BTW: Currently it's not possible to create layout records with pid = 0 which is a bug we should fix as well - so IMHO this could be done in one go.

Any concerns?

Actions
Copy link
 #2

Updated by Steffen Gebert over 13 years ago

No

Actions
Copy link
 #3

Updated by Jo Hasenau over 13 years ago

Attached patch will add both, rootLevel and CURRENT_PID

Actions
Copy link
 #4

Updated by Jo Hasenau over 13 years ago

V2 will take PAGE_TSCONFIG_ID into account as well.

It will only give the user access to records of the current page, when there is no PAGE_TSCONFIG_ID restricting the access to another page.

Actions
Copy link
 #5

Updated by Jo Hasenau over 13 years ago

Just have to wait for Ingo renaming the table and fields from be_layout to backend_layout #24117

Actions
Copy link
 #6

Updated by Jo Hasenau over 13 years ago

Done with V3

Actions
Copy link
 #7

Updated by Jo Hasenau over 13 years ago

V3 did not apply to current trunk for some reason

Attached is V4

Actions
Copy link
 #8

Updated by Jo Hasenau over 13 years ago

rootLevel has to be -1 instead of 1 and CURRENT_PID actually should have been THIS_UID.

Attached patch fixes these bugs.

Actions
Copy link
 #9

Updated by Jo Hasenau over 13 years ago

It seems there is a bug in the concept of the rootLevel settings:

Notice: that the -1 value will still select foreign_table records for selectox boxes only from root (pid=0)

So it just doesn't work, which is why we have to remove it for now

Actions
Copy link
 #10

Updated by Jo Hasenau over 13 years ago

Attached V6 just changes the foreign_table_where

Actions
Copy link
 #11

Updated by Steffen Gebert over 13 years ago

rev. 10250

Actions
Copy link
 #12

Updated by Susanne Moog about 13 years ago

  • Target version deleted (4.5.0)
Actions
Copy link
 #13

Updated by Frederic Biermann about 12 years ago

I have the same issue with Typo3 4.5.x.
A customer have a page where he wants to add a General Record Storage Page.
When he do this he can't change the backend_layout anymore and it will be displayed that an invalid value is selected.

Actions
Copy link

Also available in: Atom PDF