Bug #39967
closedmod.SHARED.colPos_list is not taken into account any more
100%
Description
Since the merge of the https://review.typo3.org/13510
mod.SHARED.colPos_list is not taken into account any more as a hardcoded backend layout is used with all columns.
Updated by Jigal van Hemert over 12 years ago
I'd like to drop this setting completely. There are a few solutions to help integrators during the upgrade:
- make an upgrade wizard which tries to find these settings and creates backend layouts; this is rather complicated as the setting might apply to a subtree of the site
- make an upgrade wizard which tries to detect the setting and explains how to solve it manually
- only document it
Updated by Michael Stucki over 12 years ago
- Status changed from New to Accepted
- Complexity set to easy
I agree that an upgrade wizard could help to replace the old setting entirely. Anyone up for taking over this change?
Updated by Oliver Hader almost 12 years ago
Modified for Introduction Package with a new backend layout, see https://review.typo3.org/#/c/16793/
Updated by Chris topher over 11 years ago
With #47071 we have updated the documentation of mod.SHARED.colPos_list so that the documentation says mod.SHARED.colPos_list has been removed.
The only thing that still needs to be done here is:
Remove mod.SHARED.colPos_list from the code completely.
Updated by Jo Hasenau almost 11 years ago
Well - as usual - it is not that easy, since colPos_list and backend layouts are completely different things, with completely different tasks.
While backend layouts provide the structure as such, that will be available for a certain site, colPos_list is meant to restrict the access to these columns i.e. based on certain conditions.
AFAIK this has been working even together with backend layouts in TYPO3 4.x so removing it silently with 6.0 instead of fixing the bug will now result in additional problems for people who are upgrading from 4.5 LTS to 6.2 LTS.
IMHO we must revert the removal at least for 6.2, but actually this should be fixed in earlier versions as well.
Updated by Helmut Hummel almost 11 years ago
- Target version changed from 6.0.0 to 6.2.0
- TYPO3 Version changed from 6.0 to 6.2
- Is Regression set to No
I agree that we should provide a compatibility layer for that at least for 6.2
Updated by Jo Hasenau almost 11 years ago
There is no need for a compatibility layer, since this has been a working feature, that was broken with the patch you mentioned in the description.
Removing it without deprecation is against our CGL, so it currently could not be removed before TYPO3 6.4 or 7.0
Actually it should not be removed, since it is used for another purpose than just a "poor man's backend layout".
Updated by Patrick Broens almost 11 years ago
Agree with Joey,
The mentioned patch should be reverted and the issue should be fixed for all versions.
When having a backend layout, you sometimes want to prevent access to a certain column for some users/user groups. At one of my clients (a university) we are using the setting mod.SHARED.colPos_list together with backend layouts, and it works.
As mentioned by Jo colPos_list and backend layouts are completely different things. The first is for access, the second for structure.
Updated by Peter Niederlag almost 11 years ago
+1 for reverting, backend layouts does in no way replace the functionality of mod.SHARED.colPos_list
Updated by Helmut Hummel over 10 years ago
Patrick Broens wrote:
As mentioned by Jo colPos_list and backend layouts are completely different things. The first is for access, the second for structure.
Not really true. The order of colPos_list also mattered and was used to identify the order of columns.
Updated by Helmut Hummel over 10 years ago
Peter Niederlag wrote:
+1 for reverting
reverting what? the layout change?
The question is: How to we deal with the situation. I see following possibilities:
- Restore mod.SHARED.colPos_list only when no backend layouts are present
- Restore mod.SHARED.colPos_list, but only ignore the order when backend layouts are present (using the info to hide containers with not matching ids)
- Add a new option that adds the possibility to remove containers by colpos from backend layouts
- Provide an Upgradewizard (or separate script) adding backend layouts if colPos_list is present (traversing the tree)
Updated by Peter Niederlag over 10 years ago
Prio I¶
- (Re-)Introduce mod.SHARED.colPos_list or introduce a new setting (I don't care to much on the name)
- Only purpose of the setting is to influence wether certain columns in dropdown-selections and the page-module are to be shown
- The order of the columns should not be within the scope of this setting (I am fine if it is for core)
Nice to have¶
- It might be nice to have both whitelisting as well as blacklisting options (something like colPos_showOnly and colPos_hide)
- It might be nice to have influence on the way columns are hidden (something like colPos_hiddenLayout = hidden|readonly)
Updated by Ernesto Baschny over 10 years ago
- Category set to Miscellaneous
- Target version changed from 6.2.0 to next-patchlevel
This is indeed a feature upgraders from 4.x are expecting which was removed in 6.0. As we don't have time until 6.2.0 now (to be released in two days), I would opt on an "easy working solution" that we can include as a very special case after 6.2.0 (in 6.2.1 for example).
It should allow admins to do what the old setting did in regards of "hiding" undesired columns. The "order" is not the scope here, this is already handled by the backend layouts.
I would not put energy in Upgrade Wizards, as this can be documented and those who need it can think about how to use the new possibilities while upgrading.
Any candidate to work on a solution for this?
Updated by Gerrit Code Review over 10 years ago
- Status changed from Accepted 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 http://review.typo3.org/31783
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 http://review.typo3.org/31783
Updated by Gerrit Code Review over 10 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/31783
Updated by Gerrit Code Review over 10 years ago
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/31783
Updated by Gerrit Code Review over 10 years ago
Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/31783
Updated by Gerrit Code Review over 10 years ago
Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/31783
Updated by Gerrit Code Review about 10 years ago
Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/32615
Updated by Jigal van Hemert about 10 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 241f4d0e8bc6568e6d48b084291afe23f5e203cd.
Updated by Gerrit Code Review about 10 years ago
- Status changed from Resolved 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 http://review.typo3.org/32697
Updated by Gerrit Code Review about 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 http://review.typo3.org/32697
Updated by Gerrit Code Review about 10 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/32697
Updated by Gerrit Code Review about 10 years ago
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/32697
Updated by Helmut Hummel about 10 years ago
- Status changed from Under Review to Resolved
Applied in changeset 20de0fc46b15f783b1884b9d7823617b838c9407.
Updated by Gerrit Code Review about 10 years ago
- Status changed from Resolved 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 http://review.typo3.org/32712
Updated by Gerrit Code Review about 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 http://review.typo3.org/32712
Updated by Gerrit Code Review about 10 years ago
Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/32742
Updated by Gerrit Code Review about 10 years ago
Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/32743
Updated by Jigal van Hemert about 10 years ago
- Status changed from Under Review to Resolved
Applied in changeset 6a38987010ed732a4f25587fab52a39c336526d5.