Bug #72171

Since the authMode_enforce strict is set editors can't edit content elements if the list_type is 0 (which is the default value)

Added by Marco Huber almost 4 years ago. Updated 12 months ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
-
Target version:
-
Start date:
2015-12-11
Due date:
% Done:

100%

TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

Since the authMode_enforce strict is set editors can't edit content elements if the list_type is 0. But 0 is the default value.

So we have to change typo3/sysext/frontend/ext_tables.sql and perhaps also offer a migration script which updates the database?

This was introduced in:
https://git.typo3.org/Packages/TYPO3.CMS.git/commitdiff/3276f07f39dcaa12458a67469b0c256bf77e89ed
https://review.typo3.org/#/c/45165/
https://forge.typo3.org/issues/32209


Related issues

Related to TYPO3 Core - Bug #32209: Be user with explicit deny can edit the content plugin fields Closed 2011-11-30

Associated revisions

Revision dda2856f (diff)
Added by Marco Huber almost 4 years ago

[BUGFIX] Set default value for list_type

Set the default value for list_type to an empty string in the
database to match the default value in the TCA.

Resolves: #72171
Related: #32209
Releases: master
Change-Id: I46ed9935571e3b3d8dbfb6bec232ffd8f63a35f0
Reviewed-on: https://review.typo3.org/45208
Reviewed-by: Markus Klein <>
Tested-by: Markus Klein <>
Reviewed-by: Daniel Goerz <>
Tested-by: Daniel Goerz <>
Reviewed-by: Christian Kuhn <>
Tested-by: Christian Kuhn <>

History

#1 Updated by Marco Huber almost 4 years ago

But 0 is the default value.

I mean, 0 is the default value in the database. In the TCA the default value is an empty string. But there could be some cases, where the default value from the TCA is ignored (f.e. our importer :-)).

#2 Updated by Gerrit Code Review almost 4 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/45208

#3 Updated by Marco Huber almost 4 years ago

Or is this one or https://forge.typo3.org/issues/32209 a breaking change?

#4 Updated by Gerrit Code Review almost 4 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/45208

#5 Updated by Gerrit Code Review almost 4 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/45208

#6 Updated by Marco Huber almost 4 years ago

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

#7 Updated by Christoph Lehmann over 3 years ago

In 6.2 it isn't fixed. Why?

#8 Updated by Wouter Wolters over 3 years ago

please read the comments for that in https://review.typo3.org/45208

#9 Updated by Benni Mack 12 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF