Bug #80800: Too many records on sys_file_reference table breaks TYPO3 when saving IRRE records - TYPO3 Core - TYPO3 Forge

Custom queries

Accessibility
Easy tasks
Forge Triage
JavaScript issues
JavaScript Issues (w/o Epics/Stories)
My open issues
No feedback within 90 days
Open Bugs
Open issues of 10
Recently modified
Regressions
Stabilization Issues
Usability or UX

Watchers (3)

Eric no-lastname-given
José Ricardo
Tobias Schmidt

Actions

Copy link

Bug #80800

closed

Too many records on sys_file_reference table breaks TYPO3 when saving IRRE records

Added by José Ricardo over 7 years ago. Updated about 6 years ago.

Status:

Closed

Priority:

Must have

Assignee:

Category:

File Abstraction Layer (FAL)

Target version:

Start date:

2017-04-10

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

PHP Version:

Tags:

Complexity:

Is Regression:

Sprint Focus:

On Location Sprint

Description

When saving a record (eg: tx_news_domain_model_news) and uploading images on field with IRRE of FAL records (eg: fal_media) the database will generate lots of UPDATES on the sys_file_reference table, breaking it when this table is too big (250k+ records).

The reason for that is that the resorting method of TYPO3's Data Handler (sysext/core/Classes/DataHandling/DataHandler.php) verifies if there is a "sortby" configuration on the table's TCA, which, in the case of sys_file_reference, is the field "sorting". The resorting method is useful to recalculate the order of the record on a PID using the "sorting" field.

However, unless i'm wrong, there are no places on TYPO3 CMS where sys_file_reference records are listed, so this configuration is useless for this table. Setting it to empty string solves the problem and apparently has no side effects.

Version: TYPO3 7.6.16

Related issues 3 (1 open — 2 closed)

Related to TYPO3 Core - Bug #80875: Multiple problems with relation handler if too many records exist on a page

Accepted

Sascha Egerer

2017-06-12

Actions

Related to TYPO3 Core - Task #48098: Reversed order in sorting field in sys_file_reference compared to e.g. tt_content

Rejected

2013-05-10

Actions

Related to TYPO3 Core - Bug #83160: Remove unused column sys_file_reference.sorting

Closed

Stephan Großberndt

2017-11-29

Actions

Issue # Delay: days Cancel

History
Notes
Property changes
Associated revisions

Actions

Copy link

Updated by José Ricardo over 7 years ago

Description updated (diff)

Actions

Copy link

Updated by José Ricardo over 7 years ago

Description updated (diff)

Actions

Copy link

Updated by Thomas Hohn over 7 years ago

Status changed from New to Needs Feedback

Please provide information about TYPO3 version - if it 8.7 - did you follow all the migrations steps in the install tool?

Actions

Copy link

Updated by José Ricardo over 7 years ago

Description updated (diff)

Actions

Copy link

Updated by José Ricardo over 7 years ago

Sorry, forgot it. It's 7.6.16 (i've updated the description as well).

Actions

Copy link

Updated by Thomas Hohn over 7 years ago

Had similar issues on a installation. Our fix was rebuild the ref_index since it contained a lot of crashed records. And
then we started to use https://github.com/NamelessCoder/asynchronous_reference_indexing since we had
similar issues.

Actions

Copy link

Updated by José Ricardo over 7 years ago

Gonna look the plugin, but the question is why sortby is necessary for sys_file_reference?

Actions

Copy link

Updated by Sascha Egerer over 7 years ago

Related to Bug #80875: Multiple problems with relation handler if too many records exist on a page added

Actions

Copy link

Updated by Alexander Opitz about 7 years ago

Status changed from Needs Feedback to New

Actions

Copy link

#10

Updated by Stephan Großberndt about 7 years ago

Related to Task #48098: Reversed order in sorting field in sys_file_reference compared to e.g. tt_content added

Actions

Copy link

#11

Updated by Stephan Großberndt about 7 years ago

https://docs.typo3.org/typo3cms/TCAReference/ColumnsConfig/Type/Inline.html#foreign-sortby

Actions

Copy link

#12

Updated by Stephan Großberndt about 7 years ago

Status changed from New to In Progress
Priority changed from Should have to Must have
TYPO3 Version changed from 7 to 9

Actions

Copy link

#13

Updated by Gerrit Code Review about 7 years ago

Status changed from In Progress 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/54342

Actions

Copy link

#14

Updated by Gerrit Code Review about 7 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/54342

Actions

Copy link

#15

Updated by Gerrit Code Review about 7 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/54342

Actions

Copy link

#16

Updated by Gerrit Code Review about 7 years ago

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

Actions

Copy link

#17

Updated by Gerrit Code Review about 7 years ago

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

Actions

Copy link

#18

Updated by Gerrit Code Review about 7 years ago

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

Actions

Copy link

#19

Updated by Gerrit Code Review about 7 years ago

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

Actions

Copy link

#20

Updated by Gerrit Code Review about 7 years ago

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

Actions

Copy link

#21

Updated by Gerrit Code Review about 7 years ago

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

Actions

Copy link

#22

Updated by Gerrit Code Review about 7 years ago

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

Actions

Copy link

#23

Updated by Gerrit Code Review about 7 years ago

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

Actions

Copy link

#24

Updated by Susanne Moog almost 7 years ago

Sprint Focus set to On Location Sprint

Actions

Copy link

#25

Updated by Gerrit Code Review almost 7 years ago

Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54840

Actions

Copy link

#26