Skip to content

[!!!][TASK] Remove the unused GlossaryBackendUtility - #111

Open
calien666 wants to merge 1 commit into
glossary-api-v3from
task/glossary-maintenance
Open

calien666 wants to merge 1 commit into
glossary-api-v3from
task/glossary-maintenance

Conversation

@calien666

@calien666 calien666 commented Oct 4, 2026 •

Copy link
Copy Markdown
Member

Builds on #52 and targets its branch.

GlossaryBackendUtility is removed together with the page TSconfig template path and its only partial. It filtered the languages of the DeepL translation dropdown in glossary folders, but its caller stayed in deepltranslate-core when the glossary functionality moved into this extension and was removed there. A Breaking changelog entry documents the removal.

Design notes

  • The removal is Breaking in a minor version, as the class was public. Nothing in this extension or in deepltranslate-core calls it any longer.
  • The removal does not go to 5, which only takes patch releases.

Updated on 2026-10-06

@github-actions

github-actions Bot commented Oct 4, 2026 •

Copy link
Copy Markdown

Documentation rendering

You can find files attached to the below linked Workflow Run URL (Logs).

Please note that files only stay for around 5 days!

Name Link
Commit 963e6cb
Logs https://github.com/web-vision/deepltranslate-glossary/actions/runs/37540109214
Documentation https://github.com/web-vision/deepltranslate-glossary/actions/runs/37540109214/artifacts/11447179154

sbuerk added a commit that referenced this pull request Oct 6, 2026
The -t option was documented as usable only with composerInstall
suites, which this extension does not provide. It selects the core
for composerUpdate and the matching configuration for phpstan. The
version hint of TYPO3 v14 is dropped: it went stale with every raise
of the core requirement.

The same correction as on branch 5, taken out of #111, which waits
for the Glossary API v3 (#52).
The utility filtered the languages of the DeepL translation dropdown
in glossary folders. Its caller stayed in deepltranslate-core when the
glossary functionality moved into this extension and was removed
there, and the dropdown is no longer rendered in glossary folders at
all.

The backend template path registered in page TSconfig only provided a
partial no template renders, so it is removed as well.
@sbuerk
sbuerk force-pushed the task/glossary-maintenance branch from 3f50a96 to 963e6cb Compare October 6, 2026 22:21
@sbuerk sbuerk changed the title [TASK] Correct the runTests.sh help and remove the unused GlossaryBackendUtility [!!!][TASK] Remove the unused GlossaryBackendUtility Oct 6, 2026

@sbuerk sbuerk left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reduced to the removal of GlossaryBackendUtility: the correction of the runTests.sh help is on main with #116 (and on 5 with #113). Rebased onto #52, one commit with the subject of the pull request.

Verified: green at its own commit on TYPO3 13 (cgl, phpstan, unit, functional, checkRst) and as part of the stack on 13 and 14. Nothing in this extension or in deepltranslate-core calls the class.

Open for the final review: the removal is Breaking in a minor version, as the class was public.

@sbuerk
sbuerk added this pull request to stack #124 October 6, 2026 22:32
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.

2 participants