benjaminkott / benjaminkott/bootstrap_package

palette fields missing for tx_bootstrappackage_tab_item's media file references

Open
#1,611 0 comments 1 reaction 0 assignees View on GitHub
bug
Dominant language
PHP
Stars
355
Forks
212
Avg merge
1h 4m
Merged PRs (30d)
31

Description

# Bug Report

## Prerequisites

* [x] Can you reproduce the problem on TYPO3 v11.5 LTS
* [x] Can you reproduce the problem on TYPO3 v12.4 LTS
* [x] Can you reproduce the problem on TYPO3 v13.4 LTS
* [x] Did you [perform a cursory search](https://github.com/issues?q=is%3Aissue+user%3Abenjaminkott+repo%3Abenjaminkott%2Fbootstrap_package)
to see if your bug or enhancement is already reported?

## Description

When adding a PDF file to `tx_bootstrappackage_tab_item`'s media field the `link` field is not showing up so it's impossible to add a link an PDF file.

The issue seems to be in `Configuration/TCA/tx_bootstrappackage_tab_item.php`, ['columns']['media']['overrideChildTca'] only checking for types `\TYPO3\CMS\Core\Resource\File::FILETYPE_IMAGE` and `\TYPO3\CMS\Core\Resource\File::FILETYPE_VIDEO`.

Adding
```
\TYPO3\CMS\Core\Resource\File::FILETYPE_APPLICATION => [
'showitem' => '
--palette--;;extendedBasicOverlayPalette,
--palette--;;filePalette',
],
```
fixed the issue for me. Consider adding `\TYPO3\CMS\Core\Resource\File::FILETYPE_TEXT` or `\TYPO3\CMS\Core\Resource\File::FILETYPE_UNKNOWN` so these can be linked too.

## Steps to Reproduce

1. Create a `Tab` content element.
2. Add a new tab item.
3. Add a PDF file to the `media` field.

### Expected behavior

`link` field showing.

### Actual behavior

No `link` field showing.

### Screenshots

Image

## Versions

15.0.4

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.