Skip to content

Conversation

@rr-it
Copy link
Contributor

@rr-it rr-it commented Jul 25, 2025

This excludes the luxletter link, log and queue tables from the reference index.
Thereby the size of the reference index is massively reduced.

For this patch to become effective it is necessary to manually run:

  • SQL:
    DELETE FROM `sys_refindex`
    WHERE `tablename` IN (
      'tx_luxletter_domain_model_link',
      'tx_luxletter_domain_model_log',
      'tx_luxletter_domain_model_queue'
    );
  • vendor/bin/typo3 referenceindex:update

This fixes #130

@einpraegsam
Copy link
Collaborator

Thx for digging into it. 2 things just come into my mind:

  1. I would like to not use final for class. That is fine for project development but not for a community extension that could be extended from anyone
  2. I would also like to use the same code style as it is already implemented in LUX - see https://github.com/in2code-de/lux/blob/develop/Classes/TCA/PreventReferenceIndex.php
    Because both extensions are similar created I would like to have a similar codebase

@rr-it rr-it force-pushed the feature/exclude-tables-refindex branch from c187385 to 8ffd0d7 Compare July 28, 2025 10:50
@rr-it rr-it force-pushed the feature/exclude-tables-refindex branch from 8ffd0d7 to e101e48 Compare July 28, 2025 11:00
@rr-it
Copy link
Contributor Author

rr-it commented Jul 28, 2025

Refactored to use LUX code style

@einpraegsam einpraegsam merged commit 8e0f2c4 into in2code-de:develop Aug 1, 2025
9 checks passed
@einpraegsam
Copy link
Collaborator

Thx for your work

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

DB table sys_refindex gets upsized with entries from tx_luxletter_domain_model_*

2 participants