Project

General

Profile

Actions

Feature #103493

closed

Show button to edit full record in linkvalidator (additonally?)

Added by Sybille Peters 8 months ago. Updated 5 months ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Linkvalidator
Target version:
-
Start date:
2024-03-27
Due date:
% Done:

100%

Estimated time:
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

By default, a form showing only the field with the broken link is opened, if clicking the "pencil" button in the Link Validator report.

If checking sys_redirect.target as well, I noticed that opening the form this way is not helpful, because some context is missing: we see only the target, but not the source_path and the rest of the fields.

e.g. TSconfig:

mod.linkvalidator.searchFields.sys_redirect = target

In this particular case, the default behaviour is unhelpful.

Originally, the behaviour was that the entire record was edited. This, however also proved as unhelpful, because sometime the broken link was a bit hidden, or it was in a different tab.

Implementation options

  1. (Idealistic) would be if the full record was opened, but the tab where the broken link is contained is opened by default, and if necessary there is scrolling so the field is in focus. Additionally, it might be helpful if this field (or all fields with broken links) would be marked visibly. (However, marking visibly should be different from what is currently used in case of evaluation).
  2. (pragmatic) show both buttons but make it configurable, e.g.

Files

edit_redirect_target.png (15.8 KB) edit_redirect_target.png Sybille Peters, 2024-06-21 16:20

Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Bug #104261: "Linkvalidator" module: Link in record title does not work anymoreClosedJasmina Ließmann2024-06-29

Actions
Actions #1

Updated by Gerrit Code Review 8 months ago

  • Status changed from New to Under Review

Patch set 1 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/83614

Actions #2

Updated by Gerrit Code Review 5 months ago

Patch set 2 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/83614

Actions #3

Updated by Gerrit Code Review 5 months ago

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

Actions #4

Updated by Sybille Peters 5 months ago

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

Updated by Sybille Peters 5 months ago

Actions #6

Updated by Jasmina Ließmann 5 months ago

  • Related to Bug #104261: "Linkvalidator" module: Link in record title does not work anymore added
Actions #7

Updated by Benni Mack 5 months ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF